[Intel-gfx] [PATCH 08/13] drm/i915; Pull some watermarks state into a separate structure

Chris Wilson chris at chris-wilson.co.uk
Tue Aug 6 21:58:40 CEST 2013


On Tue, Aug 06, 2013 at 10:24:07PM +0300, ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> There is a bunch of global state that needs to be considered when
> checking watermarks for validity. Move most of that to a new
> structure intel_wm_config, to avoid having to pass around so
> many variables.
> 
> One notable thing left out is the DDB partitioning information,
> since we often anyway need to check the same watermarks against
> both 1/2 and 5/6 DDB partitioning layouts.
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

Switching sprites_enabled from a count threw me for a moment there, but
the conversion looks sane.
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list