| Message ID | 1481662664-18986-1-git-send-email-paulo.r.zanoni@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > We've been ignoring the poor bugzilla reporters that say PSR causes > system lockups and all other sorts of problems. The earliest bug > report is from April, so I think we can use the "revert the offending > commit if no fixes are presented within 8 months" rule here. > > Fixes: 9b58e352b463 ("drm/i915: Enable PSR by default on Haswell and Broadwell.") > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97602 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97515 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96736 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96704 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96569 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95176 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94985 > Cc: <stable@vger.kernel.org> # v4.6+ > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > Cc: Jim Bride <jim.bride@linux.intel.com> > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> > --- > drivers/gpu/drm/i915/intel_psr.c | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_psr.c b/drivers/gpu/drm/i915/intel_psr.c > index d5f8d03..6aca8ff 100644 > --- a/drivers/gpu/drm/i915/intel_psr.c > +++ b/drivers/gpu/drm/i915/intel_psr.c > @@ -823,13 +823,9 @@ void intel_psr_init(struct drm_i915_private *dev_priv) > dev_priv->psr_mmio_base = IS_HASWELL(dev_priv) ? > HSW_EDP_PSR_BASE : BDW_EDP_PSR_BASE; > > - /* Per platform default */ > - if (i915.enable_psr == -1) { > - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > - i915.enable_psr = 1; > - else > - i915.enable_psr = 0; > - } > + /* Per platform default: all disabled. */ > + if (i915.enable_psr == -1) > + i915.enable_psr = 0; > > /* Set link_standby x link_off defaults */ > if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
> > Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > > > > On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > > We've been ignoring the poor bugzilla reporters that say PSR causes > > system lockups and all other sorts of problems. The earliest bug > > report is from April, so I think we can use the "revert the offending > > commit if no fixes are presented within 8 months" rule here. > > Would it means that, since we cannot fix and disable it by default then PSR is going to be deprecated ? I recall that on reported bugs we pay attention whether or not default parameter values is used or not to proceed on fix. -Yann > > Fixes: 9b58e352b463 ("drm/i915: Enable PSR by default on Haswell and > > Broadwell.") > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97602 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97515 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96736 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96704 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96569 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95176 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94985 > > Cc: <stable@vger.kernel.org> # v4.6+ > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > > Cc: Jim Bride <jim.bride@linux.intel.com> > > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> > > --- > > drivers/gpu/drm/i915/intel_psr.c | 10 +++------- > > 1 file changed, 3 insertions(+), 7 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/intel_psr.c > > b/drivers/gpu/drm/i915/intel_psr.c > > index d5f8d03..6aca8ff 100644 > > --- a/drivers/gpu/drm/i915/intel_psr.c > > +++ b/drivers/gpu/drm/i915/intel_psr.c > > @@ -823,13 +823,9 @@ void intel_psr_init(struct drm_i915_private > *dev_priv) > > dev_priv->psr_mmio_base = IS_HASWELL(dev_priv) ? > > HSW_EDP_PSR_BASE : BDW_EDP_PSR_BASE; > > > > - /* Per platform default */ > > - if (i915.enable_psr == -1) { > > - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > > - i915.enable_psr = 1; > > - else > > - i915.enable_psr = 0; > > - } > > + /* Per platform default: all disabled. */ > > + if (i915.enable_psr == -1) > > + i915.enable_psr = 0; > > > > /* Set link_standby x link_off defaults */ > > if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Em Qua, 2016-12-14 às 08:00 +0000, Argotti, Yann escreveu: > > > > > > Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > > > > > > > > On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > > > > > > We've been ignoring the poor bugzilla reporters that say PSR > > > causes > > > system lockups and all other sorts of problems. The earliest bug > > > report is from April, so I think we can use the "revert the > > > offending > > > commit if no fixes are presented within 8 months" rule here. > > > > Would it means that, since we cannot fix and disable it by default > then PSR is going to be deprecated ? I recall that on reported bugs > we pay attention whether or not default parameter values is used or > not to proceed on fix. The plan should be to re-enable it at some point, but I'm not working on that. What I did was just fix the regression so that users have usable machines while we try to fix the feature in the appropriate way. I'll let Rodrigo and Jim provide better details on their current plans regarding PSR. > -Yann > > > > > > > > > Fixes: 9b58e352b463 ("drm/i915: Enable PSR by default on Haswell > > > and > > > Broadwell.") > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97602 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97515 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96736 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96704 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96569 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95176 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94985 > > > Cc: <stable@vger.kernel.org> # v4.6+ > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > > > Cc: Jim Bride <jim.bride@linux.intel.com> > > > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> > > > --- > > > drivers/gpu/drm/i915/intel_psr.c | 10 +++------- > > > 1 file changed, 3 insertions(+), 7 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/i915/intel_psr.c > > > b/drivers/gpu/drm/i915/intel_psr.c > > > index d5f8d03..6aca8ff 100644 > > > --- a/drivers/gpu/drm/i915/intel_psr.c > > > +++ b/drivers/gpu/drm/i915/intel_psr.c > > > @@ -823,13 +823,9 @@ void intel_psr_init(struct drm_i915_private > > *dev_priv) > > > > > > dev_priv->psr_mmio_base = IS_HASWELL(dev_priv) ? > > > HSW_EDP_PSR_BASE : BDW_EDP_PSR_BASE; > > > > > > - /* Per platform default */ > > > - if (i915.enable_psr == -1) { > > > - if (IS_HASWELL(dev_priv) || > > > IS_BROADWELL(dev_priv)) > > > - i915.enable_psr = 1; > > > - else > > > - i915.enable_psr = 0; > > > - } > > > + /* Per platform default: all disabled. */ > > > + if (i915.enable_psr == -1) > > > + i915.enable_psr = 0; > > > > > > /* Set link_standby x link_off defaults */ > > > if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Tue, 13 Dec 2016, Paulo Zanoni <paulo.r.zanoni@intel.com> wrote: > We've been ignoring the poor bugzilla reporters that say PSR causes > system lockups and all other sorts of problems. The earliest bug > report is from April, so I think we can use the "revert the offending > commit if no fixes are presented within 8 months" rule here. Ugh. Should be more like 2 weeks or so. We suck. :( Acked-by: Jani Nikula <jani.nikula@intel.com> PS. You're using a version of git that screws up # comments at the end of Cc: lines, so I presume this didn't make it to stable list. It's enough to have the Cc: in the commit when it gets applied though. > > Fixes: 9b58e352b463 ("drm/i915: Enable PSR by default on Haswell and Broadwell.") > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97602 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97515 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96736 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96704 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96569 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95176 > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94985 > Cc: <stable@vger.kernel.org> # v4.6+ > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > Cc: Jim Bride <jim.bride@linux.intel.com> > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> > --- > drivers/gpu/drm/i915/intel_psr.c | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_psr.c b/drivers/gpu/drm/i915/intel_psr.c > index d5f8d03..6aca8ff 100644 > --- a/drivers/gpu/drm/i915/intel_psr.c > +++ b/drivers/gpu/drm/i915/intel_psr.c > @@ -823,13 +823,9 @@ void intel_psr_init(struct drm_i915_private *dev_priv) > dev_priv->psr_mmio_base = IS_HASWELL(dev_priv) ? > HSW_EDP_PSR_BASE : BDW_EDP_PSR_BASE; > > - /* Per platform default */ > - if (i915.enable_psr == -1) { > - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > - i915.enable_psr = 1; > - else > - i915.enable_psr = 0; > - } > + /* Per platform default: all disabled. */ > + if (i915.enable_psr == -1) > + i915.enable_psr = 0; > > /* Set link_standby x link_off defaults */ > if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
On Wed, 2016-12-14 at 08:00 +0000, Argotti, Yann wrote: > > > > Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > > > > > > > > On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > > > We've been ignoring the poor bugzilla reporters that say PSR causes > > > system lockups and all other sorts of problems. The earliest bug > > > report is from April, so I think we can use the "revert the offending > > > commit if no fixes are presented within 8 months" rule here. > > > > Would it means that, since we cannot fix and disable it by default then PSR is going to be deprecated ? No, it is not getting deprecated. But with bug list increasing let's do what is more safe for end users while we continue working to get this in a enough stable level again. > I recall that on reported bugs we pay attention whether or not default parameter values is used or not to proceed on fix. This is definitely a good approach to define the priorities of the bugs, but not to close or ignore them completely. Here what happens is that we end up ignoring the bugs even with parameter enabled so it is better do disable it for now. Thanks, Rodrigo. > -Yann > > > > Fixes: 9b58e352b463 ("drm/i915: Enable PSR by default on Haswell and > > > Broadwell.") > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97602 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97515 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96736 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96704 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96569 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95176 > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94985 > > > Cc: <stable@vger.kernel.org> # v4.6+ > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > > > Cc: Jim Bride <jim.bride@linux.intel.com> > > > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> > > > --- > > > drivers/gpu/drm/i915/intel_psr.c | 10 +++------- > > > 1 file changed, 3 insertions(+), 7 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/i915/intel_psr.c > > > b/drivers/gpu/drm/i915/intel_psr.c > > > index d5f8d03..6aca8ff 100644 > > > --- a/drivers/gpu/drm/i915/intel_psr.c > > > +++ b/drivers/gpu/drm/i915/intel_psr.c > > > @@ -823,13 +823,9 @@ void intel_psr_init(struct drm_i915_private > > *dev_priv) > > > dev_priv->psr_mmio_base = IS_HASWELL(dev_priv) ? > > > HSW_EDP_PSR_BASE : BDW_EDP_PSR_BASE; > > > > > > - /* Per platform default */ > > > - if (i915.enable_psr == -1) { > > > - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > > > - i915.enable_psr = 1; > > > - else > > > - i915.enable_psr = 0; > > > - } > > > + /* Per platform default: all disabled. */ > > > + if (i915.enable_psr == -1) > > > + i915.enable_psr = 0; > > > > > > /* Set link_standby x link_off defaults */ > > > if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> > On Wed, 2016-12-14 at 08:00 +0000, Argotti, Yann wrote: > > > > > > Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > > > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > > > > > > > > > > > > On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > > > > We've been ignoring the poor bugzilla reporters that say PSR > > > > causes system lockups and all other sorts of problems. The > > > > earliest bug report is from April, so I think we can use the > > > > "revert the offending commit if no fixes are presented within 8 > months" rule here. > > > > > > Would it means that, since we cannot fix and disable it by default then > PSR is going to be deprecated ? > > No, it is not getting deprecated. > > But with bug list increasing let's do what is more safe for end users > while we continue working to get this in a enough stable level again. > > > I recall that on reported bugs we pay attention whether or not default > parameter values is used or not to proceed on fix. > > This is definitely a good approach to define the priorities of the bugs, > but not to close or ignore them completely. Here what happens is that we > end up ignoring the bugs even with parameter enabled so it is better do > disable it for now. > > Thanks, > Rodrigo. Thanks for the confirmation. I think this is fair approach. Let me know whenever some patches are available to re-test PSR > > > -Yann > > > > > > Fixes: 9b58e352b463 ("drm/i915: Enable PSR by default on Haswell > > > > and > > > > Broadwell.") > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97602 > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97515 > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96736 > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96704 > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96569 > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95176 > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94985 > > > > Cc: <stable@vger.kernel.org> # v4.6+ > > > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > > > > Cc: Jim Bride <jim.bride@linux.intel.com> > > > > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> > > > > --- > > > > drivers/gpu/drm/i915/intel_psr.c | 10 +++------- > > > > 1 file changed, 3 insertions(+), 7 deletions(-) > > > > > > > > diff --git a/drivers/gpu/drm/i915/intel_psr.c > > > > b/drivers/gpu/drm/i915/intel_psr.c > > > > index d5f8d03..6aca8ff 100644 > > > > --- a/drivers/gpu/drm/i915/intel_psr.c > > > > +++ b/drivers/gpu/drm/i915/intel_psr.c > > > > @@ -823,13 +823,9 @@ void intel_psr_init(struct drm_i915_private > > > *dev_priv) > > > > dev_priv->psr_mmio_base = IS_HASWELL(dev_priv) ? > > > > HSW_EDP_PSR_BASE : BDW_EDP_PSR_BASE; > > > > > > > > - /* Per platform default */ > > > > - if (i915.enable_psr == -1) { > > > > - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > > > > - i915.enable_psr = 1; > > > > - else > > > > - i915.enable_psr = 0; > > > > - } > > > > + /* Per platform default: all disabled. */ > > > > + if (i915.enable_psr == -1) > > > > + i915.enable_psr = 0; > > > > > > > > /* Set link_standby x link_off defaults */ > > > > if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > > > > > > _______________________________________________ > > > Intel-gfx mailing list > > > Intel-gfx@lists.freedesktop.org > > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Em Ter, 2016-12-13 às 21:22 +0000, Vivi, Rodrigo escreveu: > Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > > Patch pushed. Thanks everybody. I'll just go ahead and close the bugzilla tickets. Even though we want to fix PSR for real and those bugs could be useful as a documentation of what's wrong, I think it's better to just close them since they're actually fixed now. We can still go back to them later since they're documented in the git commit message. If anybody disagrees, please feel free to reopen them. > > On Tue, 2016-12-13 at 18:57 -0200, Paulo Zanoni wrote: > > > > We've been ignoring the poor bugzilla reporters that say PSR causes > > system lockups and all other sorts of problems. The earliest bug > > report is from April, so I think we can use the "revert the > > offending > > commit if no fixes are presented within 8 months" rule here. > > > > Fixes: 9b58e352b463 ("drm/i915: Enable PSR by default on Haswell > > and Broadwell.") > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97602 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97515 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96736 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96704 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96569 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95176 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94985 > > Cc: <stable@vger.kernel.org> # v4.6+ > > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> > > Cc: Jim Bride <jim.bride@linux.intel.com> > > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> > > --- > > drivers/gpu/drm/i915/intel_psr.c | 10 +++------- > > 1 file changed, 3 insertions(+), 7 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/intel_psr.c > > b/drivers/gpu/drm/i915/intel_psr.c > > index d5f8d03..6aca8ff 100644 > > --- a/drivers/gpu/drm/i915/intel_psr.c > > +++ b/drivers/gpu/drm/i915/intel_psr.c > > @@ -823,13 +823,9 @@ void intel_psr_init(struct drm_i915_private > > *dev_priv) > > dev_priv->psr_mmio_base = IS_HASWELL(dev_priv) ? > > HSW_EDP_PSR_BASE : BDW_EDP_PSR_BASE; > > > > - /* Per platform default */ > > - if (i915.enable_psr == -1) { > > - if (IS_HASWELL(dev_priv) || > > IS_BROADWELL(dev_priv)) > > - i915.enable_psr = 1; > > - else > > - i915.enable_psr = 0; > > - } > > + /* Per platform default: all disabled. */ > > + if (i915.enable_psr == -1) > > + i915.enable_psr = 0; > > > > /* Set link_standby x link_off defaults */ > > if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) >
diff --git a/drivers/gpu/drm/i915/intel_psr.c b/drivers/gpu/drm/i915/intel_psr.c index d5f8d03..6aca8ff 100644 --- a/drivers/gpu/drm/i915/intel_psr.c +++ b/drivers/gpu/drm/i915/intel_psr.c @@ -823,13 +823,9 @@ void intel_psr_init(struct drm_i915_private *dev_priv) dev_priv->psr_mmio_base = IS_HASWELL(dev_priv) ? HSW_EDP_PSR_BASE : BDW_EDP_PSR_BASE; - /* Per platform default */ - if (i915.enable_psr == -1) { - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) - i915.enable_psr = 1; - else - i915.enable_psr = 0; - } + /* Per platform default: all disabled. */ + if (i915.enable_psr == -1) + i915.enable_psr = 0; /* Set link_standby x link_off defaults */ if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
We've been ignoring the poor bugzilla reporters that say PSR causes system lockups and all other sorts of problems. The earliest bug report is from April, so I think we can use the "revert the offending commit if no fixes are presented within 8 months" rule here. Fixes: 9b58e352b463 ("drm/i915: Enable PSR by default on Haswell and Broadwell.") Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97602 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97515 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96736 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96704 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96569 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95176 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94985 Cc: <stable@vger.kernel.org> # v4.6+ Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: Jim Bride <jim.bride@linux.intel.com> Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> --- drivers/gpu/drm/i915/intel_psr.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-)