[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