Skip to content

[SYCL][Doc] Propose a graph external event extension to reusable events - #23377

Merged
sarnex merged 5 commits into
intel:syclfrom
adamfidel:external_event_spec_pr
Oct 13, 2026
Merged

sarnex merged 5 commits into
intel:syclfrom
adamfidel:external_event_spec_pr

Conversation

@mmichel11

Copy link
Copy Markdown
Contributor
  • Adds a property list parameter to reusable event enqueue operations.
  • Proposes a new property graph_external which enables event dependencies outside a command_graph when passed to enqueue_signal_event or enqueue_wait_event[s].
  • Clarifies the behavior of the reusable event extension when recording to a command_graph.

Co-authored-by: Adam Fidel <adam.fidel@intel.com>
@mmichel11
mmichel11 force-pushed the external_event_spec_pr branch from de9f283 to 8ac3ea2 Compare October 5, 2026 22:11
@mmichel11
mmichel11 marked this pull request as ready for review October 5, 2026 22:44
@mmichel11
mmichel11 requested review from a team as code owners October 5, 2026 22:44
@mmichel11
mmichel11 requested a review from adamfidel October 5, 2026 22:44
Comment thread sycl/doc/extensions/experimental/sycl_ext_oneapi_reusable_events.asciidoc Outdated
@mmichel11
mmichel11 requested a review from gmlueck October 7, 2026 22:49
Comment thread sycl/doc/extensions/experimental/sycl_ext_oneapi_reusable_events.asciidoc Outdated
Comment thread sycl/doc/extensions/experimental/sycl_ext_oneapi_reusable_events.asciidoc Outdated
Comment thread sycl/doc/extensions/experimental/sycl_ext_oneapi_reusable_events.asciidoc Outdated
Comment thread sycl/doc/extensions/experimental/sycl_ext_oneapi_reusable_events.asciidoc Outdated
Comment thread sycl/doc/extensions/experimental/sycl_ext_oneapi_reusable_events.asciidoc Outdated
Comment thread sycl/doc/extensions/experimental/sycl_ext_oneapi_reusable_events.asciidoc Outdated
KornevNikita pushed a commit that referenced this pull request Oct 9, 2026
Bumps the fetched Level Zero loader from v1.32.0 (spec 1.17) to v1.34.0
(spec 1.19) and the matching `pkg_check_modules` minimum.

Motivation: upcoming graph-external event support in the L0v2 adapter
(URT-1305, SYCL side in #23377) needs API that first appeared in spec
1.18:
- `zeCommandListAppendSignalEventWithParameters` /
`zeCommandListAppendWaitOnEventsWithParameters`
- `ZE_EVENT_COUNTER_BASED_FLAG_GRAPH_EXTERNAL`

v1.34.0 is already the loader version pinned in
`devops/dependencies.json`.

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
* Move SYCL graph interaction and property to graph extension
* Move `graph_external` examples to SYCL graph usage guide
* Update wording of enqueue function property constraints
* Update exception ec outside recording to `errc::invalid`
Comment thread sycl/doc/extensions/experimental/sycl_ext_oneapi_graph.asciidoc Outdated
@sys-ce-bb

Copy link
Copy Markdown
Contributor

@intel/llvm-gatekeepers please consider merging

@sarnex
sarnex merged commit dac8b73 into intel:sycl Oct 13, 2026
3 checks passed
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.

6 participants