Skip to content
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

Fabio/cmd buffer kernel update staging #2073

Merged

Conversation

fabiomestre
Copy link
Contributor

No description provided.

@github-actions github-actions bot added loader Loader related feature/bug conformance Conformance test suite issues. specification Changes or additions to the specification experimental Experimental feature additions/changes/specification level-zero L0 adapter specific issues cuda CUDA adapter specific issues hip HIP adapter specific issues native-cpu Native CPU adapter specific issues command-buffer Command Buffer feature addition/changes/specification labels Sep 9, 2024
@fabiomestre fabiomestre force-pushed the fabio/cmd_buffer_kernel_update_staging branch 7 times, most recently from 22753f2 to 9ff534c Compare September 10, 2024 17:21
@fabiomestre fabiomestre force-pushed the fabio/cmd_buffer_kernel_update_staging branch 2 times, most recently from 96bfa9c to dce24a9 Compare September 26, 2024 13:16
- Updates the specification to add support for command-buffer kernel handle updates.
- Adds new UR tests for this feature.
- Adds an implementation for the Cuda and Hip adapters.
- Changes the UR_DEVICE_INFO_COMMAND_BUFFER_UPDATE_SUPPORT_EXP enum to a new UR_DEVICE_INFO_COMMAND_BUFFER_UPDATE_CAPABILITIES_EXP which uses a bitfield instead of a boolean.
- Changes the spec for urCommandBufferUpdateKernelLaunchExp and ur_exp_command_buffer_update_kernel_launch_desc_t to make it more intuitive with less complicated errors codes:
    - Passing a non-nullptr local work-group will now always update the command to use that value.
    - Passing a nullptr local work-group will now always keep the current command behaviour (either generated by the implementation or user-defined).
    - Passing zero to newWorkDim is now an error.
@fabiomestre fabiomestre force-pushed the fabio/cmd_buffer_kernel_update_staging branch from dce24a9 to d944ff3 Compare September 26, 2024 15:30
@aarongreig aarongreig merged commit 532a4ec into oneapi-src:main Sep 30, 2024
142 of 148 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
command-buffer Command Buffer feature addition/changes/specification conformance Conformance test suite issues. cuda CUDA adapter specific issues experimental Experimental feature additions/changes/specification hip HIP adapter specific issues level-zero L0 adapter specific issues loader Loader related feature/bug native-cpu Native CPU adapter specific issues specification Changes or additions to the specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants