[PATCH i-g-t] tests/xe_eudebug_online: breakpoint-many-sessions-* add delay before resume
Maciej Patelczyk
maciej.patelczyk at intel.com
Mon Jan 27 12:10:01 UTC 2025
For single tile workloads under debug are executed sequentially.
The test quickly resumes workload that hit the breakpoint without any
delay. That is then catched by the expectation that there shall be a
break between workloads.
Add sleep before resuming workload so that there is a forced gap
between workloads. If they were properly executed sequentially
then the diff between workloads shall be longer than sleep.
Additionally fix the 'ms' to 'us' typo report in case of assert.
Reviewed-by: Dominik Grzegorzek <dominik.grzegorzek at intel.com>
Signed-off-by: Maciej Patelczyk <maciej.patelczyk at intel.com>
---
tests/intel/xe_eudebug_online.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/tests/intel/xe_eudebug_online.c b/tests/intel/xe_eudebug_online.c
index 726fc55b1..d3504b79f 100644
--- a/tests/intel/xe_eudebug_online.c
+++ b/tests/intel/xe_eudebug_online.c
@@ -2305,6 +2305,7 @@ static void test_many_sessions_on_tiles(int fd, bool multi_tile)
attempt_mask |= BIT(i);
should_break = 1;
+ usleep(WORKLOAD_DELAY_US);
eus = (struct drm_xe_eudebug_event_eu_attention *)data[i]->exception_event;
eu_ctl_resume(s[i]->debugger->master_fd, s[i]->debugger->fd,
eus->client_handle, eus->exec_queue_handle,
@@ -2328,11 +2329,11 @@ static void test_many_sessions_on_tiles(int fd, bool multi_tile)
if (multi_tile)
igt_assert_f(diff < WORKLOAD_DELAY_US,
- "Expected to execute workloads concurrently. Actual delay: %" PRIu64 " ms\n",
+ "Expected to execute workloads concurrently. Actual delay: %" PRIu64 " us\n",
diff);
else
igt_assert_f(diff >= WORKLOAD_DELAY_US,
- "Expected a serialization of workloads. Actual delay: %" PRIu64 " ms\n",
+ "Expected a serialization of workloads. Actual delay: %" PRIu64 " us\n",
diff);
}
--
2.43.0
More information about the igt-dev
mailing list