From da04d13807044aaf17615b66577fb0e832011ab1 Mon Sep 17 00:00:00 2001 From: "Neil R. Spruit" Date: Mon, 6 Jan 2025 14:19:10 -0800 Subject: [PATCH] [L0]: Fix Out Event in Enqueue Wait Events to prevent reuse - Fix Out Event created in Enqueue Wait Events to ensure the event is not reused. - Removed completion logic that was allowing caching of the event. - event_handle_t zeEventDestroy handles the memory leak issue that this branch was attempting to fix. Signed-off-by: Neil R. Spruit --- source/adapters/level_zero/event.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/source/adapters/level_zero/event.cpp b/source/adapters/level_zero/event.cpp index 3eb5d6fbcf..172d58dd99 100644 --- a/source/adapters/level_zero/event.cpp +++ b/source/adapters/level_zero/event.cpp @@ -145,10 +145,6 @@ ur_result_t urEnqueueEventsWait( std::unique_lock Lock(Queue->Mutex); resetCommandLists(Queue); } - if (OutEvent && (*OutEvent)->Completed) { - UR_CALL(CleanupCompletedEvent((*OutEvent), false, false)); - UR_CALL(urEventReleaseInternal((*OutEvent))); - } return UR_RESULT_SUCCESS; } @@ -795,7 +791,7 @@ urEventWait(uint32_t NumEvents, ///< [in] number of events in the event list // ur_event_handle_t_ *Event = ur_cast(e); if (!Event->hasExternalRefs()) - die("urEventsWait must not be called for an internal event"); + die("urEventWait must not be called for an internal event"); ze_event_handle_t ZeHostVisibleEvent; if (auto Res = Event->getOrCreateHostVisibleEvent(ZeHostVisibleEvent))