Skip to content
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

[Bindless][Exp] Add device queries for sampled image fetch #1264

Merged

Conversation

przemektmalon
Copy link
Contributor

@przemektmalon przemektmalon commented Jan 19, 2024

Added the following queries for device capabilities of fetching sampled images:

  • DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_1D_USM_EXP
  • DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_1D_EXP
  • DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_2D_USM_EXP
  • DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_2D_EXP
  • DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_3D_USM_EXP
  • DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_3D_EXP

@przemektmalon przemektmalon requested review from a team as code owners January 19, 2024 14:16
@przemektmalon przemektmalon requested a review from ldrumm January 19, 2024 14:16
@przemektmalon
Copy link
Contributor Author

Related DPC++ MR: intel/llvm#12447

@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch from aa442a3 to 9713f7d Compare January 22, 2024 09:52
@codecov-commenter
Copy link

codecov-commenter commented Jan 22, 2024

Codecov Report

Attention: Patch coverage is 1.17647% with 84 lines in your changes are missing coverage. Please review.

Project coverage is 12.46%. Comparing base (78ef1ca) to head (30e4fd3).
Report is 134 commits behind head on main.

Files Patch % Lines
include/ur_print.hpp 0.00% 72 Missing ⚠️
tools/urinfo/urinfo.hpp 0.00% 12 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1264      +/-   ##
==========================================
- Coverage   14.82%   12.46%   -2.36%     
==========================================
  Files         250      239      -11     
  Lines       36220    36087     -133     
  Branches     4094     4092       -2     
==========================================
- Hits         5369     4498     -871     
- Misses      30800    31585     +785     
+ Partials       51        4      -47     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch from 9713f7d to c9f5374 Compare January 23, 2024 12:58
@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch 2 times, most recently from 6ddda43 to 098b310 Compare February 6, 2024 10:44
@przemektmalon
Copy link
Contributor Author

Friendly ping @oneapi-src/unified-runtime-cuda-write

@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch 2 times, most recently from 81d9401 to 18f3039 Compare February 12, 2024 16:49
@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch 3 times, most recently from b386db5 to 0f07c69 Compare February 28, 2024 09:41
@przemektmalon
Copy link
Contributor Author

Friendly ping to @oneapi-src/unified-runtime-cuda-write and @ldrumm for re-reviews/approvals/merging.

The changes in this MR are minimal, however I would need them to be merged before I can update the related DPC++ MR to make it ready to review and get it closer to merging.

Note that the related DPC++ MR fails CI but this is stemming from tests that are unrelated to that particular MR/

Copy link
Contributor

@npmiller npmiller left a comment

Choose a reason for hiding this comment

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

CUDA changes LGTM

@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch from 0f07c69 to 7500178 Compare March 1, 2024 13:15
@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch 2 times, most recently from 803036e to 30e4fd3 Compare March 13, 2024 10:04
@przemektmalon przemektmalon added the ready to merge Added to PR's which are ready to merge label Mar 13, 2024
@przemektmalon
Copy link
Contributor Author

Marked as ready to merge. Corresponding DPC++ PR has gotten approvals and passed the last CI (currently running again due to git merge, but it is passing).

@kbenzie kbenzie added the loader Loader related feature/bug label Apr 3, 2024
@kbenzie kbenzie added specification Changes or additions to the specification cuda CUDA adapter specific issues labels Apr 3, 2024
@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch 8 times, most recently from 8fe880b to 60b8afe Compare April 10, 2024 11:02
@github-actions github-actions bot added the experimental Experimental feature additions/changes/specification label Apr 10, 2024
@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch 2 times, most recently from 69e70cb to 0ead0ca Compare April 10, 2024 15:43
Added the following queries for device capabilities of fetching
sampled images:

- DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_1D_USM_EXP
- DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_1D_EXP
- DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_2D_USM_EXP
- DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_2D_EXP
- DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_3D_USM_EXP
- DEVICE_INFO_BINDLESS_SAMPLED_IMAGE_FETCH_3D_EXP
@przemektmalon przemektmalon force-pushed the przemek/sampled-image-fetch branch from 0ead0ca to 21f023f Compare April 10, 2024 17:23
@kbenzie kbenzie added the images UR images label Apr 16, 2024
@aarongreig aarongreig merged commit 4347e0c into oneapi-src:main Apr 17, 2024
53 checks passed
aarongreig added a commit to aarongreig/unified-runtime that referenced this pull request Apr 17, 2024
…/sampled-image-fetch"

This reverts commit 4347e0c, reversing
changes made to 7611542.
@aarongreig aarongreig mentioned this pull request Apr 17, 2024
aarongreig added a commit that referenced this pull request Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuda CUDA adapter specific issues experimental Experimental feature additions/changes/specification images UR images loader Loader related feature/bug ready to merge Added to PR's which are ready to merge specification Changes or additions to the specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants