Skip to content

[UR][L0] Create pool descriptors from subdevices... #17465

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

Merged
merged 1 commit into from
Mar 17, 2025

Conversation

kswiecicki
Copy link
Contributor

stored in device handle. The pool_descriptor::create function retrieves subdevices partitioned with UR_DEVICE_PARTITION_BY_CSLICE by default. This causes problem in a SYCL scenario where user obtains subdevices partitioned with eg. sycl::info::partition_affinity_domain::numa.

@kswiecicki kswiecicki requested a review from a team as a code owner March 14, 2025 10:30
@kswiecicki kswiecicki force-pushed the pool-descriptor-subdev-fix branch from d63513d to e70beed Compare March 14, 2025 10:32
@kswiecicki
Copy link
Contributor Author

@igchor could you look at this? I expect v2 L0 adapter to have a similar issue.

@kswiecicki kswiecicki force-pushed the pool-descriptor-subdev-fix branch from e70beed to f5cd795 Compare March 14, 2025 10:42
stored in device handle. The pool_descriptor::create function retrieves
subdevices partitioned with UR_DEVICE_PARTITION_BY_CSLICE by default.
This causes problem in a SYCL scenario where user obtains subdevices
partitioned with eg. sycl::info::partition_affinity_domain::numa.
@kswiecicki kswiecicki requested a review from pbalcer March 14, 2025 16:08
@kswiecicki
Copy link
Contributor Author

@intel/llvm-gatekeepers this PR is ready to be merged.

@martygrant martygrant merged commit 19dbfb7 into intel:sycl Mar 17, 2025
32 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