Skip to content

Conversation

againull
Copy link
Contributor

@againull againull commented Aug 28, 2025

Context creation was failing when sub-sub-devices were exposed, because
the Level Zero adapter attempted to add multiple USM pools with
identical descriptors. This occurred since, for the L0 backend,
sub-sub-devices and their parent sub-devices share the same Level Zero
device handle and should use the same USM pool.

This PR resolves the issue by ensuring only devices with unique Level Zero
handles are collected for USM pool creation, preventing duplicate pools.

Additionally, this PR fixes an issue with an uninitialized
ur_device_partition_property_t variable passed to UR. Previously, this
could result in an unexpected value for value.affinity_domain when the
property type was UR_DEVICE_PARTITION_BY_CSLICE resulting on error from
adapter.

@againull againull force-pushed the subsub_context branch 2 times, most recently from 551a6a4 to cdbe9f6 Compare August 28, 2025 23:49
@againull againull changed the title [UR][L0] Fix pool creation when sub-sub-devices are exposed [UR] Fix usm pools creation failure at context initialization Aug 28, 2025
Context creation was failing when sub-sub-devices were exposed, because
the Level Zero adapter attempted to add multiple USM pools with
identical descriptors. This occurred since, for the L0 backend,
sub-sub-devices and their parent sub-devices share the same Level Zero
device handle and should use the same USM pool.

This PR resolves the issue by ensuring only devices with unique Level Zero
handles are collected for USM pool creation, preventing duplicate pools.

Additionally, this PR fixes an issue with an uninitialized
`ur_device_partition_property_t` variable passed to UR. Previously, this
could result in an unexpected value for `value.affinity_domain` when the
property type was `UR_DEVICE_PARTITION_BY_CSLICE` resulting on error from
adapter.
@againull againull marked this pull request as ready for review August 28, 2025 23:59
@againull againull requested review from a team as code owners August 28, 2025 23:59
@againull
Copy link
Contributor Author

againull commented Sep 2, 2025

Failures are unrelated.

********************
Unexpectedly Passed Tests (2):
  SYCL :: Matrix/joint_matrix_bf16_fill_k_cache_arg_dim.cpp
  SYCL :: Matrix/joint_matrix_bf16_fill_k_cache_runtime_dim.cpp

fixed by #19933

CMake Error: install(EXPORT "unified-runtime-targets" ...) includes target "xptifw" which requires target "emhash" that is not in any export set.
#19944

@againull againull merged commit ac9dcf3 into intel:sycl Sep 2, 2025
49 of 53 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.

3 participants