Skip to content

Conversation

againull
Copy link
Contributor

@againull againull commented Sep 23, 2025

In multi-device context, ensure that pointer kernel arguments are set only for the device associated with the queue being used for kernel launch. Previously, arguments were set for all devices in the kernel's device map, which was unnecessary and potentially incorrect when launching on a specific device.

Same will be done for v2 adapter here: #20179

@againull againull force-pushed the set_ptr_arg_via_pending branch from b1a4342 to 19922a0 Compare September 23, 2025 22:10
@againull againull force-pushed the set_ptr_arg_via_pending branch from 19922a0 to 7286382 Compare September 23, 2025 22:16
@againull againull force-pushed the set_ptr_arg_via_pending branch from 7286382 to 32ed923 Compare September 23, 2025 22:19
@againull againull closed this Sep 24, 2025
@againull againull reopened this Sep 24, 2025
@againull againull force-pushed the set_ptr_arg_via_pending branch from 32ed923 to 779e0de Compare September 24, 2025 16:41
@againull againull force-pushed the set_ptr_arg_via_pending branch from 779e0de to 5765617 Compare September 24, 2025 20:47
@againull againull force-pushed the set_ptr_arg_via_pending branch from 5765617 to ced1389 Compare September 24, 2025 20:48
Ensure that pointer kernel arguments are set only for the device associated
with the queue being used for kernel launch. Previously, arguments
were set for all devices in the kernel's device map, which was
unnecessary and potentially incorrect when launching on a specific
device.
@againull againull marked this pull request as ready for review September 25, 2025 04:27
@againull againull requested review from a team as code owners September 25, 2025 04:27
@againull againull merged commit 0031df1 into intel:sycl Sep 26, 2025
54 of 55 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.

4 participants