[Intel-gfx] [PATCH 2/7] drm/i915: Fix computation of last_adjustment for RPS autotuning

Chris Wilson chris at chris-wilson.co.uk
Wed Mar 18 02:05:55 PDT 2015


On Wed, Mar 18, 2015 at 12:26:49PM +0530, Deepak S wrote:
> On Friday 06 March 2015 08:36 PM, Chris Wilson wrote:
> I think we should modify adj in GEN6_PM_RP_UP_EI_EXPIRED?
> if not not we might request higher freq since we add adj to new_delay before request freq.

Oh yeah. That branch didn't exist when I first wrote the patch, and
didn't exist at the end of the series so I never saw it!

For this point in the patch series, yes, we do need to set adj=0.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list