Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] i915/perf: Fix loop termination logic in two tests
@ 2024-01-18  4:55 Ashutosh Dixit
  2024-01-18  5:34 ` ✓ CI.xeBAT: success for " Patchwork
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Ashutosh Dixit @ 2024-01-18  4:55 UTC (permalink / raw)
  To: igt-dev

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@intel.com>
Signed-off-by: Ashutosh Dixit <ashutosh.dixit@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

^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-01-18 19:38 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-18  4:55 [PATCH i-g-t] i915/perf: Fix loop termination logic in two tests Ashutosh Dixit
2024-01-18  5:34 ` ✓ CI.xeBAT: success for " Patchwork
2024-01-18  5:46 ` ✓ Fi.CI.BAT: " Patchwork
2024-01-18  7:09 ` ✗ Fi.CI.IGT: failure " Patchwork
2024-01-18 19:38 ` [PATCH i-g-t] " Dixit, Ashutosh

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox