flywithu/ntel-gpu-tools
1
0
Fork 0
mirror of https://github.com/tiagovignatti/intel-gpu-tools.git synced 2025-06-09 08:56:11 +00:00
Code Issues Releases Wiki Activity

pm_rps: Fix test to target original min and max

Browse Source 
The goal of the test is to confirm that gt_cur_freq_mhz can be forced
to the boundaries of the frequency range by collapsing gt_min_freq_mhz
and gt_max_freq_mhz to the target value. But we miss testing the upper
end of the range by targetting the current value of max after it has
been set equal to min. So fix by targetting orginal max instead of
current max.

This correction exposes a problem in setfreq where min is always set
to target before max, which should fail if the target value is greater
than max. So fix that too.

Signed-off-by: Jeff McGee <jeff.mcgee@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
This commit is contained in:
Jeff McGee 2014-01-10 15:12:32 -06:00 committed by Daniel Vetter
parent 44b5d7b330
commit d0b5fd2bff
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
tests/pm_rps.c
View File

@ -101,8 +101,13 @@ static int do_writeval(FILE *filp, int val, int lerrno)
static void setfreq(int val)
{
if (val > fmax) {
writeval(stuff[MAX].filp, val);
writeval(stuff[MIN].filp, val);
} else {
writeval(stuff[MIN].filp, val);
writeval(stuff[MAX].filp, val);
}
}
static void checkit(void)
@ -166,11 +171,11 @@ igt_simple_main
dumpit();
checkit();
setfreq(fmin);
setfreq(origmin);
if (verbose)
dumpit();
restore_assert(fcur == fmin);
setfreq(fmax);
setfreq(origmax);
if (verbose)
dumpit();
restore_assert(fcur == fmax);