> -----Original Message-----
> From: Ville Syrjälä <[email protected]>
> Sent: Thursday, July 16, 2020 6:19 PM
> To: Shankar, Uma <[email protected]>
> Cc: [email protected]
> Subject: Re: [Intel-gfx] [PATCH] drm/i915/display/fbc: Disable fbc by default
> on
> TGL
>
> On Thu, Jul 16, 2020 at 03:38:03PM +0300, Ville Syrjälä wrote:
> > On Thu, Jul 16, 2020 at 02:25:40PM +0530, Uma Shankar wrote:
> > > Fbc is causing random underruns in CI execution on TGL platforms.
> > > Disabling the same while the problem is being debugged and analyzed.
> > >
> > > Cc: Stanislav Lisovskiy <[email protected]>
> > > Cc: Ville Syrjälä <[email protected]>
> > > Signed-off-by: Uma Shankar <[email protected]>
> >
> > Acked-by: Ville Syrjälä <[email protected]>
>
> Scratch that...
>
> >
> > > ---
> > > drivers/gpu/drm/i915/display/intel_fbc.c | 7 +++++++
> > > 1 file changed, 7 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/i915/display/intel_fbc.c
> > > b/drivers/gpu/drm/i915/display/intel_fbc.c
> > > index 3a4f980788a6..1d6370b29b27 100644
> > > --- a/drivers/gpu/drm/i915/display/intel_fbc.c
> > > +++ b/drivers/gpu/drm/i915/display/intel_fbc.c
> > > @@ -1420,6 +1420,13 @@ void intel_fbc_handle_fifo_underrun_irq(struct
> drm_i915_private *dev_priv)
> > > */
> > > static int intel_sanitize_fbc_option(struct drm_i915_private
> > > *dev_priv) {
> > > + /*
> > > + * Fbc is causing random underruns in CI execution on TGL platforms.
> > > + * Disabling the same while the problem is being debugged and analyzed.
> > > + */
> > > + if (IS_TIGERLAKE(dev_priv))
> > > + return 0;
>
> ... looks lke this should be done *after* the modparam check below.
> Otherwise we can't enable fbc for testing via the modparam.
Oh yeah, updated the change and sent a v2 with fix. Thanks Ville.
Regards,
Uma Shankar
> > > +
> > > if (dev_priv->params.enable_fbc >= 0)
> > > return !!dev_priv->params.enable_fbc;
> > >
> > > --
> > > 2.22.0
> >
> > --
> > Ville Syrjälä
> > Intel
> > _______________________________________________
> > Intel-gfx mailing list
> > [email protected]
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
> --
> Ville Syrjälä
> Intel
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
