summaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorKatarzyna Dec <katarzyna.dec@intel.com>2017-08-31 09:40:09 +0200
committerArkadiusz Hiler <arkadiusz.hiler@intel.com>2017-08-31 15:34:50 +0300
commit5ce65a9a51f17e0183e3e4f8943981ee7b96cadd (patch)
treefaa9251ce9ad446741ec6edd587384aef39da73d /scripts
parentc2159678d283fea5615ec8e846a51cf4954ac82d (diff)
pm_rps: Changes in waitboost scenario
CI is observing sporadical failures in pm_rps subtests. There are a couple of reasons. One of them is the fact that on gen6, gen7 and gen7.5, max frequency (as in the HW limit) is not set to RP0, but the value obtaind from PCODE (which may be different from RP0). Thus the test is operating under wrong assumptions (SOFTMAX == RP0 == BOOST which is simply not the case). Let's compare current frequency with BOOST frequency rather than SOFTMAX to get the test behaviour under control. In boost_freq function I set MAX freq to medium freqency, which ensures that we for sure reach BOOST frequency. This could help with failures with boost frequency failing to drop down. Additionally GPU reset needs to be modified so we are not dependent on kernel's low priority retire worker. Reset method was replaced by igt_force_gpu_reset() and in reset testcase we make sure that we can recover from hang. v2: Commit message, simplified waiting for boost to finish, drop noisy whitespace cleanup. v3: Removed reading from i915_rps_boost_info debugfs because it not the same on every kernel. Removed function waiting for boost. Instead of that I made sure we will reach in boost by setting MAX freq to fmid. v4: Moved proposal with making test drm master to other patch v5: Used igt_force_gpu_reset() to reset GPU. Modified "reset" testcase. v6: Comments changes and update. v7: Fixing typos v8: Fixing another typos in comments and commit msg Cc: Chris Wilson <chris@chris-wilson.co.uk> Cc: Jeff Mcgee <jeff.mcgee@intel.com> Cc: Petri Latvala <petri.latvala@intel.com> Cc: Jani Saarinen <jani.saarinen@intel.com> Cc: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@intel.com> Reviewed-by: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@intel.com> Acked-by: Daniel Vetter <daniel.vetter@intel.com> Signed-off-by: Katarzyna Dec <katarzyna.dec@intel.com>
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions