[Intel-gfx] [PATCH] drm/i915: Show FBC worker status in debugfs

[Chris Wilson]

Quoting Paulo Zanoni (2017-12-21 17:20:50)
> Em Qua, 2017-12-20 às 20:58 +0000, Chris Wilson escreveu:
> > Include the pending update from the FBC worker in i915_fbc_status.
> > 
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Cc: Paulo Zanoni <paulo.r.zanoni at intel.com>
> > ---
> >  drivers/gpu/drm/i915/i915_debugfs.c | 13 +++++++++----
> >  1 file changed, 9 insertions(+), 4 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c
> > b/drivers/gpu/drm/i915/i915_debugfs.c
> > index c4780f085428..931037a458be 100644
> > --- a/drivers/gpu/drm/i915/i915_debugfs.c
> > +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> > @@ -1581,18 +1581,23 @@ static int i915_frontbuffer_tracking(struct
> > seq_file *m, void *unused)
> >  static int i915_fbc_status(struct seq_file *m, void *unused)
> >  {
> >       struct drm_i915_private *dev_priv = node_to_i915(m-
> > >private);
> > +     struct intel_fbc *fbc = &dev_priv->fbc;
> >  
> >       if (!HAS_FBC(dev_priv))
> >               return -ENODEV;
> >  
> >       intel_runtime_pm_get(dev_priv);
> > -     mutex_lock(&dev_priv->fbc.lock);
> > +     mutex_lock(&fbc->lock);
> >  
> >       if (intel_fbc_is_active(dev_priv))
> >               seq_puts(m, "FBC enabled\n");
> >       else
> > -             seq_printf(m, "FBC disabled: %s\n",
> > -                        dev_priv->fbc.no_fbc_reason);
> > +             seq_printf(m, "FBC disabled: %s\n", fbc-
> > >no_fbc_reason);
> > +
> > +     if (fbc->work.scheduled)
> > +             seq_printf(m, "FBC worker scheduled on vblank %d, 
> 
> vblank %u since this is u32?
> 
> Reviewed-by: Paulo Zanoni <paulo.r.zanoni at intel.com>

Deal. Thanks,
-Chris