Re: [Intel-gfx] [PATCH 23/26] drm/i915: use a single intel_fbc_work struct

2015-10-28 Thread ch...@chris-wilson.co.uk
On Wed, Oct 28, 2015 at 05:24:18PM +, Zanoni, Paulo R wrote: > Em Ter, 2015-10-27 às 20:29 +, Chris Wilson escreveu: > > On Tue, Oct 27, 2015 at 02:50:25PM -0200, Paulo Zanoni wrote: > > > diff --git a/drivers/gpu/drm/i915/i915_drv.h > > > b/drivers/gpu/drm/i915/i915_drv.h > > > index a9434

Re: [Intel-gfx] [PATCH 23/26] drm/i915: use a single intel_fbc_work struct

2015-10-28 Thread Zanoni, Paulo R
Em Ter, 2015-10-27 às 20:29 +, Chris Wilson escreveu: > On Tue, Oct 27, 2015 at 02:50:25PM -0200, Paulo Zanoni wrote: > > diff --git a/drivers/gpu/drm/i915/i915_drv.h > > b/drivers/gpu/drm/i915/i915_drv.h > > index a9434d1..fdbe068 100644 > > --- a/drivers/gpu/drm/i915/i915_drv.h > > +++ b/driv

Re: [Intel-gfx] [PATCH 23/26] drm/i915: use a single intel_fbc_work struct

2015-10-27 Thread Chris Wilson
On Tue, Oct 27, 2015 at 02:50:25PM -0200, Paulo Zanoni wrote: > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index a9434d1..fdbe068 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -917,9 +917,11 @@ struct i915_fbc { >

[Intel-gfx] [PATCH 23/26] drm/i915: use a single intel_fbc_work struct

2015-10-27 Thread Paulo Zanoni
This was already on my TODO list, and was requested both by Chris and Ville, for different reasons. The advantages are avoiding a frequent malloc/free pair, and the locality of having the work structure embedded in dev_priv. The maximum used memory is also smaller since previously we could have mul