From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rodrigo Vivi Subject: Re: [Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Enable PSR2 by default" Date: Thu, 11 Jul 2019 04:02:01 -0700 Message-ID: <20190711110201.GD9599@intel.com> References: <20190711092254.1719-1-chris@chris-wilson.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <20190711092254.1719-1-chris@chris-wilson.co.uk> Sender: stable-owner@vger.kernel.org To: Chris Wilson Cc: intel-gfx@lists.freedesktop.org, Dhinakaran Pandiyan , stable@vger.kernel.org List-Id: intel-gfx@lists.freedesktop.org On Thu, Jul 11, 2019 at 10:22:54AM +0100, Chris Wilson wrote: > Multiple users are reporting black screens upon boot, after resume, or > frozen after a short period of idleness. A black screen on boot is a > critical issue so disable psr2 again until resolved. > > This reverts commit 8f6e87d6d561f10cfa48a687345512419839b6d8. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=111088 I agree it is critical, but unfortunately this revert won't solve the issue. [ 1.954886] [drm:intel_psr_init_dpcd [i915]] eDP panel supports PSR version 1 [ 2.003686] [drm:intel_psr_enable_locked [i915]] Enabling PSR1 Users are claiming the regression is only on 5.2 with 5.1 working well and PSR1 is enabled by default since 5.0. A bisect would be good, but it seems a hard issue to reproduce as well what makes things more difficult. > Fixes: 8f6e87d6d561 ("drm/i915: Enable PSR2 by default") > Signed-off-by: Chris Wilson > Cc: Rodrigo Vivi > Cc: Dhinakaran Pandiyan > Cc: José Roberto de Souza > Cc: Jani Nikula > Cc: Joonas Lahtinen > Cc: stable@vger.kernel.org #v5.2 > --- > drivers/gpu/drm/i915/display/intel_psr.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_psr.c b/drivers/gpu/drm/i915/display/intel_psr.c > index 69d908e6a050..ddde4da2de33 100644 > --- a/drivers/gpu/drm/i915/display/intel_psr.c > +++ b/drivers/gpu/drm/i915/display/intel_psr.c > @@ -83,6 +83,9 @@ static bool intel_psr2_enabled(struct drm_i915_private *dev_priv, > case I915_PSR_DEBUG_DISABLE: > case I915_PSR_DEBUG_FORCE_PSR1: > return false; > + case I915_PSR_DEBUG_DEFAULT: > + if (i915_modparams.enable_psr <= 0) > + return false; > default: > return crtc_state->has_psr2; > } > -- > 2.22.0 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx