OSDN Git Service

drm/i915: Don't fiddle with rps/rc6 across GPU reset
authorChris Wilson <chris@chris-wilson.co.uk>
Tue, 10 Apr 2018 13:33:54 +0000 (14:33 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Tue, 10 Apr 2018 16:14:16 +0000 (17:14 +0100)
commit3834dc1f0e57891a398b2cabd40c60fa9595cd7c
tree586ad88d2e34a06c5070768673a52aad7a1ed41f
parentd52ad9cb9d6d3b696d6b7ad20a381a8f5520ea03
drm/i915: Don't fiddle with rps/rc6 across GPU reset

Resetting the GPU doesn't affect the RPS/RC6 state, so we can stop
forcibly reloading the registers.

Ville suggested this many moons ago, I said at that time that sanitizing
was no harm and meant that our bookkeeping was kept consistent with the
HW. However, in a forthcoming series, we want to split rps/rc6 GT
powermanagement and one of the key simplifications is the control of
when we enable it. Performing a crude sanitize in the middle of
i915_gem_reset() is then a huge wart.

Suggested-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Sagar Arun Kamble <sagar.a.kamble@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180410133354.13425-1-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_gem.c