-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add event update to command-buffers #1823
base: main
Are you sure you want to change the base?
Conversation
e27e2de
to
6afed3f
Compare
a480a67
to
4c3389f
Compare
c559851
to
938cbe4
Compare
e8db775
to
ddf3f10
Compare
af9b056
to
ea35fb5
Compare
964f62a
to
e5b5e72
Compare
test/conformance/exp_command_buffer/event_sync_kernel_command.cpp
Outdated
Show resolved
Hide resolved
bcb101b
to
98a6f5e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly LGTM but a few small questions. Most importantly about checking whether profiling is enabled or not.
cea3086
to
28d5122
Compare
28d5122
to
1cf31c0
Compare
1cf31c0
to
f788588
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Native CPU lgtm, thank you
6550981
to
23d2720
Compare
Expand the command-buffer experimental feature API so that it can be used to implement [SYCL-Graph dynamic events](reble/llvm#372). This involves extending each command append entry-point to include the following extra parameters: * An output `ur_exp_command_buffer_command_handle_t`. * An Input `ur_event_handle_t` event wait-list of dependent events. * An output `ur_event_handle_t` event that is signaled when the command completes its next execution. New entry-points are also added to update the wait-list and signal event parameters of commands: * `urCommandBufferUpdateSignalEventExp` * `urCommandBufferUpdateWaitEventsExp` APIs implemented for CUDA adapter with CTS tests.
23d2720
to
0b5b89f
Compare
ping @oneapi-src/unified-runtime-level-zero-write for review |
Expand the command-buffer experimental feature API so that it can be used to implement SYCL-Graph dynamic
events.
This involves extending each command append entry-point to include the following extra parameters:
ur_exp_command_buffer_command_handle_t
.ur_event_handle_t
event wait-list of dependent events.ur_event_handle_t
event that is signaled when the command completes its next execution.New entry-points are also added to update the wait-list and signal event parameters of commands:
urCommandBufferUpdateSignalEventExp
urCommandBufferUpdateWaitEventsExp
The UR changes are implemented for the cuda adapter only in this PR to prove the concepts, and other adapter implementations will follow as subsequent PRs to ease review.
New CTS tests are also added, see
exp_command_buffer/event_sync_kernel_command.cpp
andexp_command_buffer/event_sync.cpp
DPC++ PR intel/llvm#14459