diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2017-11-22 18:58:56 +0000 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2017-11-23 07:25:55 +0000 |
| commit | 6de721fee2066fef59f77c705e8163594066b31a (patch) | |
| tree | 9c1c30caa99383412bf7df5d1d8ae0f3220c78ec /tests | |
| parent | fb6ca825d0d8b97b6e02dea096ce3de180bbcace (diff) | |
igt/perf_pmu: Explicitly reset min_freq before max_freq
The kernel doesn't like it when you set max_freq < min_freq and
complains bitterly. So before setting max_freq = min_freq, first set
min_freq to min_freq!
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/perf_pmu.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c index 86336d20..34f10a3c 100644 --- a/tests/perf_pmu.c +++ b/tests/perf_pmu.c @@ -928,6 +928,8 @@ test_frequency(int gem_fd) min_freq = igt_sysfs_get_u32(sysfs, "gt_RPn_freq_mhz"); max_freq = igt_sysfs_get_u32(sysfs, "gt_RP0_freq_mhz"); boost_freq = igt_sysfs_get_u32(sysfs, "gt_boost_freq_mhz"); + igt_info("Frequency: min=%u, max=%u, boost=%u MHz\n", + min_freq, max_freq, boost_freq); igt_require(min_freq > 0 && max_freq > 0 && boost_freq > 0); igt_require(max_freq > min_freq); igt_require(boost_freq > min_freq); @@ -938,6 +940,8 @@ test_frequency(int gem_fd) /* * Set GPU to min frequency and read PMU counters. */ + igt_require(igt_sysfs_set_u32(sysfs, "gt_min_freq_mhz", min_freq)); + igt_require(igt_sysfs_get_u32(sysfs, "gt_min_freq_mhz") == min_freq); igt_require(igt_sysfs_set_u32(sysfs, "gt_max_freq_mhz", min_freq)); igt_require(igt_sysfs_get_u32(sysfs, "gt_max_freq_mhz") == min_freq); igt_require(igt_sysfs_set_u32(sysfs, "gt_boost_freq_mhz", min_freq)); |