[igt-dev] [PATCH i-g-t] tests/i915/perf: drop check on report reason
Lionel Landwerlin
lionel.g.landwerlin at intel.com
Wed Dec 16 08:38:20 UTC 2020
Learning the experimental way that ICL can generate 2 back to back
timer reports with the same timestamp...
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
---
tests/i915/perf.c | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/tests/i915/perf.c b/tests/i915/perf.c
index f09431472..664fd0a94 100644
--- a/tests/i915/perf.c
+++ b/tests/i915/perf.c
@@ -780,7 +780,6 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0,
format.b_off);
uint32_t *rpt1_b = (uint32_t *)(((uint8_t *)oa_report1) +
format.b_off);
- uint32_t report_reason0, report_reason1;
uint32_t b;
uint32_t ref;
@@ -791,14 +790,6 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0,
freq = time_delta ? ((uint64_t)clock_delta * 1000) / time_delta : 0;
igt_debug("freq = %"PRIu64"\n", freq);
- /* Either we have 2 reports back to back with the same
- * timestamp and those should have different reason, or we
- * were able to compute a frequency because the timestamps are
- * different.
- */
- igt_assert(freq || (gen8_report_reason(oa_report0) !=
- gen8_report_reason(oa_report1)));
-
igt_debug("clock delta = %"PRIu32"\n", clock_delta);
max_delta = clock_delta * intel_perf->devinfo.n_eus;
--
2.29.2
More information about the igt-dev
mailing list