diff options
| author | Imre Deak <imre.deak@intel.com> | 2014-12-04 17:05:25 +0200 |
|---|---|---|
| committer | Imre Deak <imre.deak@intel.com> | 2014-12-08 12:36:54 +0200 |
| commit | f70afed4c7abd95fc3250141a1e36587e31f7c87 (patch) | |
| tree | 6d58c4b544cd59366a27ec043e82fd220f4c28f6 | |
| parent | aabf2d095269ae4cb8e77dd61e5c0c16e602c1d8 (diff) | |
tests/pm_rps: vlv: load gpu for idle min/max tests
When changing the sysfs GT min/max frequencies, the kernel won't
explicitly change the current frequency, unless it becomes out of bound
based on the new min/max values. The test happens to work on non-VLV
platforms because on those the kernel resets the current frequency
unconditionally (to adjust the RPS interrupt mask as a side-effect) and
that will lead to an RPS interrupt setting the minimum frequency.
To fix this load the GPU after decreasing the min frequency and before
checking the current frequency. This should set the current frequency to
the minimum.
Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Imre Deak <imre.deak@intel.com>
| -rw-r--r-- | tests/pm_rps.c | 23 |
1 files changed, 19 insertions, 4 deletions
diff --git a/tests/pm_rps.c b/tests/pm_rps.c index 4429b2c5..389a5cb3 100644 --- a/tests/pm_rps.c +++ b/tests/pm_rps.c @@ -335,7 +335,14 @@ static void load_helper_deinit(void) drm_intel_bufmgr_destroy(lh.bufmgr); } -static void min_max_config(void (*check)(void)) +static void do_load_gpu(void) +{ + load_helper_run(LOW); + nsleep(10000000); + load_helper_stop(); +} + +static void min_max_config(void (*check)(void), bool load_gpu) { int fmid = (origfreqs[RPn] + origfreqs[RP0]) / 2; @@ -343,11 +350,15 @@ static void min_max_config(void (*check)(void)) fmid = fmid / 50 * 50; igt_debug("\nCheck original min and max...\n"); + if (load_gpu) + do_load_gpu(); check(); igt_debug("\nSet min=RPn and max=RP0...\n"); writeval(stuff[MIN].filp, origfreqs[RPn]); writeval(stuff[MAX].filp, origfreqs[RP0]); + if (load_gpu) + do_load_gpu(); check(); igt_debug("\nIncrease min to midpoint...\n"); @@ -368,10 +379,14 @@ static void min_max_config(void (*check)(void)) igt_debug("\nDecrease min to midpoint...\n"); writeval(stuff[MIN].filp, fmid); + if (load_gpu) + do_load_gpu(); check(); igt_debug("\nDecrease min to RPn...\n"); writeval(stuff[MIN].filp, origfreqs[RPn]); + if (load_gpu) + do_load_gpu(); check(); igt_debug("\nDecrease min below RPn (invalid)...\n"); @@ -605,14 +620,14 @@ igt_main } igt_subtest("basic-api") - min_max_config(basic_check); + min_max_config(basic_check, false); igt_subtest("min-max-config-idle") - min_max_config(idle_check); + min_max_config(idle_check, true); igt_subtest("min-max-config-loaded") { load_helper_run(HIGH); - min_max_config(loaded_check); + min_max_config(loaded_check, false); load_helper_stop(); } |