Skip to content

[UR][L0V2] Add experimental oneapi reusable events implementation#22349

Draft
kweronsx wants to merge 2 commits into
intel:syclfrom
kweronsx:experimental-oneapi-impl-reusable-events
Draft

[UR][L0V2] Add experimental oneapi reusable events implementation#22349
kweronsx wants to merge 2 commits into
intel:syclfrom
kweronsx:experimental-oneapi-impl-reusable-events

Conversation

@kweronsx

@kweronsx kweronsx commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

This depends on: #22186

@kweronsx kweronsx force-pushed the experimental-oneapi-impl-reusable-events branch 2 times, most recently from e5951de to 7c38f06 Compare June 18, 2026 08:45
@kweronsx kweronsx marked this pull request as ready for review June 18, 2026 08:45
@kweronsx kweronsx requested review from a team as code owners June 18, 2026 08:45
@kweronsx kweronsx requested review from kswiecicki and pbalcer June 18, 2026 08:45
@kweronsx kweronsx marked this pull request as draft June 18, 2026 09:15
// Pool of standalone, reusable events created by urEventCreateExp. Uses
// non-counter based events so they can be host-reset and reused across
// multiple enqueues. A separate pool is used depending on whether profiling
// was requested at event creation time.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment is too verbose, getReusableEventsPool seems pretty straightforward.

ASSERT_SUCCESS(urEventRelease(event));
}

// A reusable event can wait on events from a different queue and is signaled

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a similar but inverted test case that makes an operation from different queue wait on a reusable event.

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.

2 participants