Skip to content

[UR][L0 v2] Add USM pool tracking in context and fix async pool cleanup #18802

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jun 6, 2025

Conversation

staniewzki
Copy link
Contributor

@staniewzki staniewzki commented Jun 4, 2025

This PR mirrors fixes implemented in #18406 to adapter L0 v1.
To cleanup async pools properly, USM pool handles must be tracked. This feature enables UR_USM_ALLOC_INFO_POOL prop to be implemented in urUSMGetMemAllocInfo.

@staniewzki staniewzki marked this pull request as ready for review June 5, 2025 07:17
@staniewzki staniewzki requested a review from a team as a code owner June 5, 2025 07:17
Copy link
Contributor

@pbalcer pbalcer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be a test somewhere for this functionality.

@staniewzki staniewzki force-pushed the l0v2-async-cleanup branch from 8834c4a to dd78c29 Compare June 5, 2025 09:53
@staniewzki staniewzki requested a review from a team as a code owner June 5, 2025 09:53
@staniewzki
Copy link
Contributor Author

@intel/llvm-gatekeepers please merge

@kbenzie kbenzie merged commit 9fa76f9 into intel:sycl Jun 6, 2025
32 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants