[Intel-gfx] [PATCH 4/7] drm/i915: Use set_rps to enable RPS

Chris Wilson chris at chris-wilson.co.uk
Mon Feb 20 14:38:33 UTC 2017


On Mon, Feb 20, 2017 at 04:29:16PM +0200, Mika Kuoppala wrote:
> Chris Wilson <chris at chris-wilson.co.uk> writes:
> 
> > Defer actual enabling of RPS to the set rps routine, called upon
> > enabling and so we only start RPS when all thresholds have been set.
> >
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Cc: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> > Cc: stable at vger.kernel.org
> 
> As discussed in irc, we will need a followup cleanup as the
> function names deviate from the actual content.

They do still enable rps, just via set_rps_thresholds. We could go
further and move that to a common point, e.g.

if (gen >= 9)
	gen9_setup_rps();
...
else
	gen6_setup_rps();

enable_rps(); -> (the current reset_rps dance, but use intel_set_rps).
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list