diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2017-12-15 17:03:32 +0000 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2017-12-22 15:08:59 +0000 |
commit | b1b6cb18a119d72748267fd08b6ed3b18ceb6f2e (patch) | |
tree | 4cc52cad32b98a36e5ac1414dce3bc56c84bd6f7 | |
parent | aba7b4081bdf492cc5d08903d4941ca64a51f7e4 (diff) |
igt/perf_pmu: Speed up frequency measurement
Use the normal batch_duration_ns and display the sampled frequency:
Frequency: min=100, max=750, boost=750 MHz
Min frequency: requested 100.0, actual 100.0
Max frequency: requested 755.6, actual 755.6
v2: Remove the early spin_batch_end and assert the measured frequencies
are within tolerance of our target.
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>
-rw-r--r-- | tests/perf_pmu.c | 39 |
1 files changed, 23 insertions, 16 deletions
diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c index d88287c17..eb8791cd4 100644 --- a/tests/perf_pmu.c +++ b/tests/perf_pmu.c @@ -931,9 +931,10 @@ test_interrupts(int gem_fd) static void test_frequency(int gem_fd) { - const uint64_t duration_ns = 2e9; uint32_t min_freq, max_freq, boost_freq; - uint64_t min[2], max[2], start[2]; + uint64_t val[2], start[2]; + double min[2], max[2]; + unsigned long slept; igt_spin_t *spin; int fd, sysfs; @@ -962,17 +963,18 @@ test_frequency(int gem_fd) igt_require(igt_sysfs_set_u32(sysfs, "gt_boost_freq_mhz", min_freq)); igt_require(igt_sysfs_get_u32(sysfs, "gt_boost_freq_mhz") == min_freq); + gem_quiescent_gpu(gem_fd); /* Idle to be sure the change takes effect */ + spin = igt_spin_batch_new(gem_fd, 0, I915_EXEC_RENDER, 0); pmu_read_multi(fd, 2, start); - spin = igt_spin_batch_new(gem_fd, 0, I915_EXEC_RENDER, 0); - igt_spin_batch_set_timeout(spin, duration_ns); - gem_sync(gem_fd, spin->handle); + slept = measured_usleep(batch_duration_ns / 1000); - pmu_read_multi(fd, 2, min); - min[0] -= start[0]; - min[1] -= start[1]; + pmu_read_multi(fd, 2, val); + min[0] = 1e9*(val[0] - start[0]) / slept; + min[1] = 1e9*(val[1] - start[1]) / slept; igt_spin_batch_free(gem_fd, spin); + gem_quiescent_gpu(gem_fd); /* Don't leak busy bo into the next phase */ usleep(1e6); @@ -987,17 +989,18 @@ test_frequency(int gem_fd) igt_require(igt_sysfs_set_u32(sysfs, "gt_min_freq_mhz", max_freq)); igt_require(igt_sysfs_get_u32(sysfs, "gt_min_freq_mhz") == max_freq); + gem_quiescent_gpu(gem_fd); + spin = igt_spin_batch_new(gem_fd, 0, I915_EXEC_RENDER, 0); pmu_read_multi(fd, 2, start); - spin = igt_spin_batch_new(gem_fd, 0, I915_EXEC_RENDER, 0); - igt_spin_batch_set_timeout(spin, duration_ns); - gem_sync(gem_fd, spin->handle); + slept = measured_usleep(batch_duration_ns / 1000); - pmu_read_multi(fd, 2, max); - max[0] -= start[0]; - max[1] -= start[1]; + pmu_read_multi(fd, 2, val); + max[0] = 1e9*(val[0] - start[0]) / slept; + max[1] = 1e9*(val[1] - start[1]) / slept; igt_spin_batch_free(gem_fd, spin); + gem_quiescent_gpu(gem_fd); /* * Restore min/max. @@ -1008,8 +1011,12 @@ test_frequency(int gem_fd) min_freq, igt_sysfs_get_u32(sysfs, "gt_min_freq_mhz")); close(fd); - igt_assert(min[0] < max[0]); - igt_assert(min[1] < max[1]); + igt_info("Min frequency: requested %.1f, actual %.1f\n", + min[0], min[1]); + igt_info("Max frequency: requested %.1f, actual %.1f\n", + max[0], max[1]); + assert_within_epsilon(min[0], min_freq, tolerance); + assert_within_epsilon(max[0], max_freq, tolerance); } static bool wait_for_rc6(int fd) |