Skip to content

Commit 2a94843

Browse files
committed
[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 <[email protected]>
1 parent a7e096d commit 2a94843

File tree

1 file changed

+3
-14
lines changed

1 file changed

+3
-14
lines changed

sycl/test-e2e/Adapters/level_zero_batch_event_status.cpp

Lines changed: 3 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -14,21 +14,10 @@
1414
// to make sure that the batching is submitted when the urEventGetInfo is
1515
// done, rather than some other dynamic batching criteria.
1616
//
17-
// CHECK: ---> urEnqueueKernelLaunch
18-
// CHECK: ZE ---> zeCommandListAppendLaunchKernel
19-
// Shouldn't have closed until we see a urEventGetInfo
20-
// CHECK-NOT: ZE ---> zeCommandListClose
21-
// CHECK-NOT: ZE ---> zeCommandQueueExecuteCommandLists
22-
// CHECK: ---> urEventGetInfo
23-
// Shouldn't see another urGetEventInfo until after closing command list
24-
// CHECK-NOT: ---> urEventGetInfo
25-
// Look for close and Execute after urEventGetInfo
26-
// CHECK: ZE ---> zeCommandListClose
27-
// CHECK: ZE ---> zeCommandQueueExecuteCommandLists
17+
// The call to urEventGetInfo should trigger the executeOpenCommandList to
18+
// execute the batch.
2819
// CHECK: ---> urEventGetInfo
29-
// CHECK-NOT: ---> urEventsWait
30-
// CHECK: ---> urEnqueueKernelLaunch
31-
// CHECK: ZE ---> zeCommandListAppendLaunchKernel
20+
// CHECK: UR ---> UrQueue->executeOpenCommandList
3221
// CHECK: ---> urQueueFinish
3322
// Look for close and Execute after urQueueFinish
3423
// CHECK: ZE ---> zeCommandListClose

0 commit comments

Comments
 (0)