On Tue, May 28, 2013 at 04:06:07PM +0100, Damien Lespiau wrote:
> On Sat, Apr 27, 2013 at 05:59:27PM -0700, Ben Widawsky wrote:
> > From: "Xiang, Haihao" <haihao.xiang at intel.com>
> >
> > v2 (Ben): s/hsw/hws
> >
> > Signed-off-by: Xiang, Haihao <haihao.xiang at intel.com>
> > [Order changed, and modified by]
> > Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
> > ---
> > drivers/gpu/drm/i915/i915_debugfs.c | 13 +++++++++++++
> > 1 file changed, 13 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> > index a55630a..71fb7aa 100644
> > --- a/drivers/gpu/drm/i915/i915_debugfs.c
> > +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> > @@ -379,6 +379,17 @@ static int i915_gem_request_info(struct seq_file *m, void *data)
> > }
> > count++;
> > }
> > + if (!list_empty(&dev_priv->ring[VECS].request_list)) {
> > + seq_printf(m, "VEBOX requests:\n");
> > + list_for_each_entry(gem_request,
> > + &dev_priv->ring[VECS].request_list,
> > + list) {
> > + seq_printf(m, " %d @ %d\n",
> > + gem_request->seqno,
> > + (int) (jiffies - gem_request->emitted_jiffies));
> > + }
> > + count++;
> > + }
> > mutex_unlock(&dev->struct_mutex);
>
> That block doesn't seem necessary as the code above this chuck cycles
> over all the rings?
Indeed. This is a rebase relic. This hunk is removed.
>
> >
> > if (count == 0)
> > @@ -570,6 +581,7 @@ static const char *ring_str(int ring)
> > case RCS: return "render";
> > case VCS: return "bsd";
> > case BCS: return "blt";
> > + case VECS: return "vebox";
> > default: return "";
> > }
> > }
> > @@ -2099,6 +2111,7 @@ static struct drm_info_list i915_debugfs_list[] = {
> > {"i915_gem_hws", i915_hws_info, 0, (void *)RCS},
> > {"i915_gem_hws_blt", i915_hws_info, 0, (void *)BCS},
> > {"i915_gem_hws_bsd", i915_hws_info, 0, (void *)VCS},
> > + {"i915_gem_hws_vebox", i915_hws_info, 0, (void *)VECS},
> > {"i915_rstdby_delays", i915_rstdby_delays, 0},
> > {"i915_cur_delayinfo", i915_cur_delayinfo, 0},
> > {"i915_delayfreq_table", i915_delayfreq_table, 0},
> > --
> > 1.8.2.1
> >
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Ben Widawsky, Intel Open Source Technology Center
