unified-runtime icon indicating copy to clipboard operation
unified-runtime copied to clipboard

[L0] Phase 2 of Counter-Based Event Implementation

Open winstonzhang-intel opened this issue 1 year ago • 1 comments

-enable counter-based events for regular commandlist -counter-based events may be reused even though they are not done -when ref count goes to not used by external clients value it means that event may be reused by subsequent calls -move events that are no longer externally visible to re-usable pool and reuse those more aggressively

intel/llvm PR: intel/llvm#14754

winstonzhang-intel avatar Aug 02 '24 18:08 winstonzhang-intel

I rebased your PR to make sure you are using the latest main, since E2E L0 was failing. Unfortunately, it's still failing after the rebase:

Failed Tests (11):
  SYCL :: DiscardEvents/discard_events_mixed_calls.cpp
  SYCL :: ESIMD/BitonicSortKv2.cpp
  SYCL :: ESIMD/kmeans/kmeans.cpp
  SYCL :: Graph/RecordReplay/barrier_multi_queue.cpp
  SYCL :: Graph/RecordReplay/dotp_in_order.cpp
  SYCL :: Graph/RecordReplay/dotp_in_order_pause.cpp
  SYCL :: Graph/RecordReplay/dotp_in_order_with_empty_nodes.cpp
  SYCL :: Graph/RecordReplay/dotp_multiple_queues.cpp
  SYCL :: Graph/RecordReplay/host_task_in_order.cpp
  SYCL :: Graph/RecordReplay/sub_graph_in_order.cpp
  SYCL :: Graph/RecordReplay/usm_copy_in_order.cpp

These failures do not show up on main or other PRs. If you can't reproduce this locally, please reach out and I can give you access to the CI system where this is failing.

pbalcer avatar Aug 05 '24 11:08 pbalcer