[L0 v2] do not return ze_event_handle_t to the cache

on every event release.

Author: igchor

source/adapters/level_zero/v2/event.cpp

@@ -12,23 +12,17 @@
 #include "ze_api.h"
 
 namespace v2 {
-void ur_event::attachZeHandle(event_allocation event) {
-  type = event.type;
-  zeEvent = std::move(event.borrow);
-}
-
-event_borrowed ur_event::detachZeHandle() {
+void ur_event::reset() {
   // consider make an abstraction for regular/counter based
   // events if there's more of this type of conditions
   if (type == event_type::EVENT_REGULAR) {
     zeEventHostReset(zeEvent.get());
   }
-  auto e = std::move(zeEvent);
-  zeEvent = nullptr;
-
-  return e;
 }
 
 ze_event_handle_t ur_event::getZeEvent() { return zeEvent.get(); }
 
+ur_event::ur_event(event_allocation eventAllocation)
+    : type(eventAllocation.type), zeEvent(std::move(eventAllocation.borrow)) {}
+
 } // namespace v2

source/adapters/level_zero/v2/event.hpp

@@ -21,9 +21,9 @@ namespace v2 {
 
 class ur_event {
 public:
-  void attachZeHandle(event_allocation);
-  event_borrowed detachZeHandle();
+  ur_event(event_allocation eventAllocation);
 
+  void reset();
   ze_event_handle_t getZeEvent();
 
 private:

source/adapters/level_zero/v2/event_pool.cpp

@@ -18,25 +18,20 @@ ur_event *event_pool::allocate() {
   if (freelist.empty()) {
     auto start = events.size();
     auto end = start + EVENTS_BURST;
-    events.resize(end);
     for (; start < end; ++start) {
+      events.emplace_back(provider->allocate());
       freelist.push_back(&events.at(start));
     }
   }
 
   auto event = freelist.back();
-
-  auto ZeEvent = provider->allocate();
-  event->attachZeHandle(std::move(ZeEvent));
-
   freelist.pop_back();
 
   return event;
 }
 
 void event_pool::free(ur_event *event) {
-  auto _ = event->detachZeHandle();
-
+  event->reset();
   freelist.push_back(event);
 }