Skip to content

[SYCL][UR][L0] First version of UR L0 adapter #8744

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 50 commits into from
May 26, 2023
Merged

Conversation

jandres742
Copy link
Contributor

@jandres742 jandres742 commented Mar 23, 2023

Signed-off-by: Jaime Arteaga [email protected]
Signed-off-by: Brandon Yates [email protected]
Signed-off-by: Callum Fare [email protected]

@jandres742 jandres742 temporarily deployed to aws March 23, 2023 06:51 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 23, 2023 23:39 — with GitHub Actions Inactive
@jandres742 jandres742 force-pushed the url0 branch 2 times, most recently from a789054 to daf13db Compare March 24, 2023 00:17
@jandres742 jandres742 temporarily deployed to aws March 24, 2023 00:54 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 24, 2023 01:29 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 24, 2023 08:42 — with GitHub Actions Inactive
@jandres742 jandres742 force-pushed the url0 branch 3 times, most recently from 95607a2 to 4398719 Compare March 27, 2023 00:52
@jandres742 jandres742 temporarily deployed to aws March 27, 2023 00:56 — with GitHub Actions Inactive
@jandres742 jandres742 force-pushed the url0 branch 2 times, most recently from 6d4bea6 to c524deb Compare March 27, 2023 01:01
@jandres742 jandres742 temporarily deployed to aws March 27, 2023 01:05 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 27, 2023 06:49 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 27, 2023 08:02 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 28, 2023 03:21 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 28, 2023 05:25 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 28, 2023 07:34 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 28, 2023 10:27 — with GitHub Actions Inactive
@jandres742 jandres742 changed the title Url0 [SYCL][UR][L0] First version of UR L0 adapter Mar 29, 2023
@jandres742 jandres742 temporarily deployed to aws March 29, 2023 08:19 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws March 29, 2023 11:33 — with GitHub Actions Inactive
Jaime Arteaga and others added 17 commits May 26, 2023 07:32
Signed-off-by: Jaime Arteaga <[email protected]>
* Fixes for porting to UR repo (#4)

* Fixes for porting to UR repo

Signed-off-by: Brandon Yates <[email protected]>
pi_bool is uint32_t and ur_bool_t is uint8_t, so to make sure
correct functionality is maintain, use uint32_t as replacement
for pi_bool, instead of ur_bool_t.

Also, add back check for urMemImageCreate that was before in
piMemImageCreate.

Signed-off-by: Jaime Arteaga <[email protected]>
- Add changes to fix tests after

[SYCL] Add Unified Runtime plugin and route to it with SYCL_PREFER_UR
intel#9232

Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Brandon Yates <[email protected]>
this to absorb latest changes in queue native handle APIs

Signed-off-by: Jaime Arteaga <[email protected]>
@jandres742 jandres742 temporarily deployed to aws May 26, 2023 14:57 — with GitHub Actions Inactive
@jandres742 jandres742 temporarily deployed to aws May 26, 2023 15:43 — with GitHub Actions Inactive
@jandres742 jandres742 marked this pull request as ready for review May 26, 2023 16:20
@jandres742 jandres742 requested a review from a team as a code owner May 26, 2023 16:21
@againull againull merged commit 3c35b1b into intel:sycl May 26, 2023
@jandres742
Copy link
Contributor Author

Some CI errors being addressed in
#9628

kchusha pushed a commit that referenced this pull request May 26, 2023
Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Brandon Yates <[email protected]>
Signed-off-by: Callum Fare <[email protected]>

---------

Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Brandon Yates <[email protected]>
Co-authored-by: Brandon Yates <[email protected]>
Co-authored-by: Callum Fare <[email protected]>
dm-vodopyanov pushed a commit that referenced this pull request Jun 14, 2023
This moves the CUDA plugin implementation to Unified Runtime; and
changes the pi_cuda plugin to use pi2ur to implement PI. The changes to
the implementation have been kept to a minimum and should be
functionally the same. Documentation and comments have been moved
verbatim, other than changing PI references to UR.

This PR is based on top of the Level Zero adapter (#8744) so will only
be ready when that is merged.

---------

Co-authored-by: Petr Vesely <[email protected]>
Co-authored-by: Omar Ahmed <[email protected]>
Co-authored-by: Martin Morrison-Grant <[email protected]>
Co-authored-by: Aaron Greig <[email protected]>
fineg74 pushed a commit to fineg74/llvm that referenced this pull request Jun 15, 2023
This moves the CUDA plugin implementation to Unified Runtime; and
changes the pi_cuda plugin to use pi2ur to implement PI. The changes to
the implementation have been kept to a minimum and should be
functionally the same. Documentation and comments have been moved
verbatim, other than changing PI references to UR.

This PR is based on top of the Level Zero adapter (intel#8744) so will only
be ready when that is merged.

---------

Co-authored-by: Petr Vesely <[email protected]>
Co-authored-by: Omar Ahmed <[email protected]>
Co-authored-by: Martin Morrison-Grant <[email protected]>
Co-authored-by: Aaron Greig <[email protected]>
veselypeta pushed a commit to veselypeta/llvm that referenced this pull request Sep 21, 2023
Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Brandon Yates <[email protected]>
Signed-off-by: Callum Fare <[email protected]>

---------

Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Brandon Yates <[email protected]>
Co-authored-by: Brandon Yates <[email protected]>
Co-authored-by: Callum Fare <[email protected]>
veselypeta pushed a commit to veselypeta/llvm that referenced this pull request Sep 21, 2023
Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Brandon Yates <[email protected]>
Signed-off-by: Callum Fare <[email protected]>

---------

Signed-off-by: Jaime Arteaga <[email protected]>
Signed-off-by: Brandon Yates <[email protected]>
Co-authored-by: Brandon Yates <[email protected]>
Co-authored-by: Callum Fare <[email protected]>
fabiomestre pushed a commit to fabiomestre/llvm that referenced this pull request Sep 26, 2023
This moves the CUDA plugin implementation to Unified Runtime; and
changes the pi_cuda plugin to use pi2ur to implement PI. The changes to
the implementation have been kept to a minimum and should be
functionally the same. Documentation and comments have been moved
verbatim, other than changing PI references to UR.

This PR is based on top of the Level Zero adapter (intel#8744) so will only
be ready when that is merged.

---------

Co-authored-by: Petr Vesely <[email protected]>
Co-authored-by: Omar Ahmed <[email protected]>
Co-authored-by: Martin Morrison-Grant <[email protected]>
Co-authored-by: Aaron Greig <[email protected]>
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.

5 participants