Skip to content

Commit

Permalink
[SYCL][test][e2e][L0] Fix L0 batch event test reliability
Browse files Browse the repository at this point in the history
- 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]>
  • Loading branch information
nrspruit committed Jan 30, 2025
1 parent a7e096d commit 2a94843
Showing 1 changed file with 3 additions and 14 deletions.
17 changes: 3 additions & 14 deletions sycl/test-e2e/Adapters/level_zero_batch_event_status.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 2a94843

Please sign in to comment.