Skip to content

L0 provider: implement support for defer and blocking free #963

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

Conversation

igchor
Copy link
Member

@igchor igchor commented Dec 4, 2024

Description

This feature is necessary to properly support urUSMFree for v2 L0 adapter.

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly
  • CI workflows, not executed per PR (e.g. Nightly), execute properly
  • New tests added, especially if they will fail without my changes
  • Added/extended example(s) to cover this functionality
  • Extended the README/documentation
  • All newly added source files have a license
  • All newly added source files are referenced in CMake files
  • Logger (with debug/info/... messages) is used

@bratpiorka
Copy link
Contributor

@igchor do we need this for 0.10x / 2025.1?

@igchor
Copy link
Member Author

igchor commented Dec 5, 2024

@igchor do we need this for 0.10x / 2025.1?

No, for now I only want to use it for v2 L0 adapter which will not be released as part of 2025.1

@igchor igchor force-pushed the level_zero_provider_ext_free branch from 98b4e2a to 76672ed Compare December 5, 2024 15:27
@vinser52
Copy link
Contributor

vinser52 commented Dec 6, 2024

As a first step, I wanted to test this in UR (that's why I kept this as draft).

@igchor So we are not merging that till you confirm that UR works with the new UMF L0 config options and they do what UR needs?

@igchor
Copy link
Member Author

igchor commented Dec 6, 2024

As a first step, I wanted to test this in UR (that's why I kept this as draft).

@igchor So we are not merging that till you confirm that UR works with the new UMF L0 config options and they do what UR needs?

Yes, I'll undraft the PR once I manage to test this in UR

@igchor igchor force-pushed the level_zero_provider_ext_free branch 2 times, most recently from bb2d469 to 477edb6 Compare December 10, 2024 20:38
@lplewa
Copy link
Contributor

lplewa commented Jan 9, 2025

any tests?

@igchor igchor force-pushed the level_zero_provider_ext_free branch from 477edb6 to 6683056 Compare January 9, 2025 17:46
@igchor
Copy link
Member Author

igchor commented Jan 9, 2025

any tests?

For now, this will only be tested in UR. We can consider adding some tests in the future, but this will require adding a dependency on igc or committing a binary (the test requires executing a kernel).

@igchor igchor marked this pull request as ready for review January 9, 2025 18:00
@igchor igchor requested a review from a team as a code owner January 9, 2025 18:00
@igchor igchor force-pushed the level_zero_provider_ext_free branch from 6683056 to d19bc63 Compare January 15, 2025 18:20
@igchor igchor force-pushed the level_zero_provider_ext_free branch from d19bc63 to 39f77a1 Compare January 15, 2025 23:30
@igchor
Copy link
Member Author

igchor commented Jan 15, 2025

@bratpiorka could you please merge this? I have a few more fixes/improvements pending that but I'd like this to be merged first

@lukaszstolarczuk lukaszstolarczuk merged commit 7dc640c into oneapi-src:main Jan 16, 2025
78 checks passed
@igchor igchor deleted the level_zero_provider_ext_free branch January 16, 2025 16:45
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.

6 participants