[PATCH i-g-t] i915/perf: Fix loop termination logic in two tests
Ashutosh Dixit
ashutosh.dixit at intel.com
Thu Jan 18 04:55:25 UTC 2024
The code in af9910b3967d ("i915/perf: Check regularly if we are done
reading reports") does not match the commit description. Fix the code to
match the commit description.
Fixes: af9910b3967d ("i915/perf: Check regularly if we are done reading reports")
Cc: Umesh Nerlige Ramappa <umesh.nerlige.ramappa at intel.com>
Signed-off-by: Ashutosh Dixit <ashutosh.dixit at intel.com>
---
tests/intel/perf.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/intel/perf.c b/tests/intel/perf.c
index 3565d61cc393..94738b8a58b4 100644
--- a/tests/intel/perf.c
+++ b/tests/intel/perf.c
@@ -3052,7 +3052,7 @@ test_buffer_fill(const struct intel_execution_engine2 *e)
first_timestamp = oa_timestamp(report, fmt);
last_timestamp = oa_timestamp(report, fmt);
- if (((last_timestamp - first_timestamp) * oa_period) < (fill_duration / 2))
+ if (((last_timestamp - first_timestamp) * oa_period) >= (fill_duration / 2))
break;
if (oa_report_is_periodic(oa_exponent, report)) {
@@ -3279,7 +3279,7 @@ test_enable_disable(const struct intel_execution_engine2 *e)
oa_report_is_periodic(oa_exponent, report),
oa_report_get_ctx_id(report));
- if (((last_timestamp - first_timestamp) * oa_period) < (fill_duration / 2))
+ if (((last_timestamp - first_timestamp) * oa_period) >= (fill_duration / 2))
break;
if (oa_report_is_periodic(oa_exponent, report)) {
--
2.41.0
More information about the igt-dev
mailing list