Re: [PATCH v7 07/11] drm/i915: Make ring buffer size of a LRC context configurable

[Joonas Lahtinen <joonas.lahtinen@linux.intel.com>]

On ke, 2016-06-08 at 08:08 +0100, Chris Wilson wrote:
> On Tue, Jun 07, 2016 at 11:18:43AM -0400, Zhi Wang wrote:
> > 
> > This patch introduces an option for configuring the ring buffer size
> > of a LRC context after the context creation.
> > 
> > Signed-off-by: Zhi Wang <zhi.a.wang@intel.com>
> > ---
> >  drivers/gpu/drm/i915/i915_drv.h         | 1 +
> >  drivers/gpu/drm/i915/i915_gem_context.c | 1 +
> >  drivers/gpu/drm/i915/intel_lrc.c        | 3 ++-
> >  3 files changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> > index 33147b1..50b3ea7 100644
> > --- a/drivers/gpu/drm/i915/i915_drv.h
> > +++ b/drivers/gpu/drm/i915/i915_drv.h
> > @@ -878,6 +878,7 @@ struct i915_gem_context {
> >  		int pin_count;
> >  		bool initialised;
> >  	} engine[I915_NUM_ENGINES];
> > +	u32 lrc_ring_buffer_size;
> u32 ring_size;
> 

My Reviewed-by: can be kept with that change.

Regards, Joonas

> There is no reason why we need to limit it to lrc other than immediate
> convenience.
> -Chris
> 
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx