On Cannonlake+ the CS timestamp frequency might vary from one part to another. We have a new param to query this from the kernel (which reads the value from registers). v2: Skip the tests when timestamp frequency cannot be read on CNL+ (Lionel) Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx> Reviewed-by: Matthew Auld <matthew.auld@xxxxxxxxx> --- tests/perf.c | 31 +++++++++++++++++++++++++------ 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/tests/perf.c b/tests/perf.c index 7f988d7c..70166a7a 100644 --- a/tests/perf.c +++ b/tests/perf.c @@ -969,6 +969,29 @@ gen8_sanity_check_test_oa_reports(uint32_t *oa_report0, uint32_t *oa_report1, } } +static uint64_t +get_cs_timestamp_frequency(void) +{ + int cs_ts_freq = 0; + drm_i915_getparam_t gp; + + gp.param = I915_PARAM_CS_TIMESTAMP_FREQUENCY; + gp.value = &cs_ts_freq; + if (igt_ioctl(drm_fd, DRM_IOCTL_I915_GETPARAM, &gp) == 0) + return cs_ts_freq; + + igt_debug("Couldn't query CS timestamp frequency, trying to guess based on PCI-id\n"); + + if (IS_GEN7(devid) || IS_GEN8(devid)) + return 12500000; + if (IS_SKYLAKE(devid) || IS_KABYLAKE(devid) || IS_COFFEELAKE(devid)) + return 12000000; + if (IS_BROXTON(devid) || IS_GEMINILAKE(devid)) + return 19200000; + + igt_skip("Kernel with PARAM_CS_TIMESTAMP_FREQUENCY support required\n"); +} + static bool init_sys_info(void) { @@ -979,7 +1002,8 @@ init_sys_info(void) igt_assert_neq(card, -1); igt_assert_neq(devid, 0); - timestamp_frequency = 12500000; + timestamp_frequency = get_cs_timestamp_frequency(); + igt_assert_neq(timestamp_frequency, 0); if (IS_HASWELL(devid)) { /* We don't have a TestOa metric set for Haswell so use @@ -1030,10 +1054,8 @@ init_sys_info(void) igt_debug("unsupported Skylake GT size\n"); return false; } - timestamp_frequency = 12000000; } else if (IS_BROXTON(devid)) { test_set_uuid = "5ee72f5c-092f-421e-8b70-225f7c3e9612"; - timestamp_frequency = 19200000; } else if (IS_KABYLAKE(devid)) { switch (intel_gt(devid)) { case 1: @@ -1046,10 +1068,8 @@ init_sys_info(void) igt_debug("unsupported Kabylake GT size\n"); return false; } - timestamp_frequency = 12000000; } else if (IS_GEMINILAKE(devid)) { test_set_uuid = "dd3fd789-e783-4204-8cd0-b671bbccb0cf"; - timestamp_frequency = 19200000; } else if (IS_COFFEELAKE(devid)) { switch (intel_gt(devid)) { case 1: @@ -1059,7 +1079,6 @@ init_sys_info(void) igt_debug("unsupported Cannonlake GT size\n"); return false; } - timestamp_frequency = 12000000; } else { igt_debug("unsupported GT\n"); return false; -- 2.15.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx