From 2a948437a38b895fc1a672a7e08ec175bd8619e5 Mon Sep 17 00:00:00 2001 From: "Neil R. Spruit" Date: Thu, 30 Jan 2025 08:41:42 -0800 Subject: [PATCH] [SYCL][test][e2e][L0] Fix L0 batch event test reliability - Updated the Level Zero batch event test to correctly check only for the necessary messages to be printed proving the functionality is behaving correctly. Signed-off-by: Neil R. Spruit --- .../Adapters/level_zero_batch_event_status.cpp | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/sycl/test-e2e/Adapters/level_zero_batch_event_status.cpp b/sycl/test-e2e/Adapters/level_zero_batch_event_status.cpp index 91eefba54793c..59da4dff46cff 100644 --- a/sycl/test-e2e/Adapters/level_zero_batch_event_status.cpp +++ b/sycl/test-e2e/Adapters/level_zero_batch_event_status.cpp @@ -14,21 +14,10 @@ // to make sure that the batching is submitted when the urEventGetInfo is // done, rather than some other dynamic batching criteria. // -// CHECK: ---> urEnqueueKernelLaunch -// CHECK: ZE ---> zeCommandListAppendLaunchKernel -// Shouldn't have closed until we see a urEventGetInfo -// CHECK-NOT: ZE ---> zeCommandListClose -// CHECK-NOT: ZE ---> zeCommandQueueExecuteCommandLists -// CHECK: ---> urEventGetInfo -// Shouldn't see another urGetEventInfo until after closing command list -// CHECK-NOT: ---> urEventGetInfo -// Look for close and Execute after urEventGetInfo -// CHECK: ZE ---> zeCommandListClose -// CHECK: ZE ---> zeCommandQueueExecuteCommandLists +// The call to urEventGetInfo should trigger the executeOpenCommandList to +// execute the batch. // CHECK: ---> urEventGetInfo -// CHECK-NOT: ---> urEventsWait -// CHECK: ---> urEnqueueKernelLaunch -// CHECK: ZE ---> zeCommandListAppendLaunchKernel +// CHECK: UR ---> UrQueue->executeOpenCommandList // CHECK: ---> urQueueFinish // Look for close and Execute after urQueueFinish // CHECK: ZE ---> zeCommandListClose