Skip to content

[UR] Delay recycling of events created during recording until graph destruction - #23516

Open
kweronsx wants to merge 4 commits into
intel:syclfrom
kweronsx:delay_event_recycling_during_graph_recording
Open

kweronsx wants to merge 4 commits into
intel:syclfrom
kweronsx:delay_event_recycling_during_graph_recording

Conversation

@kweronsx

@kweronsx kweronsx commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

This change:

  • makes graphs retain events that are created during recording, so that destructors of event handles that release do not actually put the events back into the pool, from which they were allocated, as long as the graph, with which they are associated, is not destroyed.
  • modifies the event pool allocate() function logic, so that events are not reused if they are still in use.
  • adds a new flag that forces the even pool to create a new event, even if an event fit for reuse is available.
  • adds tests that check if event allocation using the new logic works correctly.

Check that using the force creation flag does not reuse existing events;
Check that reused events are not in progress;
Check that events do not return to the pool as long as the graph exists;
@kweronsx
kweronsx marked this pull request as ready for review October 23, 2026 09:03
@kweronsx
kweronsx requested review from a team as code owners October 23, 2026 09:03
@kweronsx
kweronsx requested a review from kswiecicki October 23, 2026 10:18

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant