[SYCL][DOC] Introduce limitations to the Reusable Events specification.#22658
Draft
slawekptak wants to merge 6 commits into
Draft
[SYCL][DOC] Introduce limitations to the Reusable Events specification.#22658slawekptak wants to merge 6 commits into
slawekptak wants to merge 6 commits into
Conversation
Update the sycl_ext_oneapi_reusable_events specification with some limitations related to the following: - Limit the interoperability between the Reusable Events API and other event APIs. - Limit the enqueue_wait_event and enqueue_wait_events, so they can only be called with events on the same context as the queue. The limitations are related to how host tasks are used for cross-context dependencies in the SYCL RT.
Remove the host task section.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the SYCL sycl_ext_oneapi_reusable_events specification to explicitly distinguish between reusable vs non-reusable events and to constrain which events may be used with the extension’s APIs, motivated by runtime limitations around cross-context dependencies involving host tasks.
Changes:
- Defines reusable events vs non-reusable events and adds
event::ext_oneapi_reusable()to query reusability. - Specifies that APIs in the Reusable Events extension reject non-reusable events (via
errc::invalid). - Updates the IPC extension spec to state that events returned from
openare reusable.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| sycl/doc/extensions/proposed/sycl_ext_oneapi_reusable_events.asciidoc | Introduces reusable/non-reusable event semantics, adds a query API, and tightens constraints/throws for extension functions. |
| sycl/doc/extensions/experimental/sycl_ext_oneapi_inter_process_communication.asciidoc | Clarifies that IPC open returns a reusable event and references the reusable events extension. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update the sycl_ext_oneapi_reusable_events specification with the following:
The limitations are related to how host tasks are used for cross-context dependencies in the SYCL RT.