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 #1535

Merged
merged 1 commit into from
May 2, 2024

Conversation

przemektmalon
Copy link
Contributor

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 April 22, 2024 07:29
@github-actions github-actions bot added loader Loader related feature/bug images UR images specification Changes or additions to the specification experimental Experimental feature additions/changes/specification cuda CUDA adapter specific issues labels Apr 22, 2024
@przemektmalon
Copy link
Contributor Author

Related DPC++ PR: intel/llvm#12447

@przemektmalon przemektmalon marked this pull request as draft April 22, 2024 10:57
@kbenzie
Copy link
Contributor

kbenzie commented Apr 22, 2024

Should we close #1529?

@przemektmalon
Copy link
Contributor Author

Should we close #1529?

Apologies, when I created this PR I wasn't aware of #1529.

I think it's easier for me if I have control over this PR, in case I need to make changes to fix the DPC++ CI failures. So I would prefer to keep this PR and close #1529.

@kbenzie
Copy link
Contributor

kbenzie commented Apr 22, 2024

@przemektmalon yeah not problem, I agree it'll be easier for you to work on your own fork if any changes are necessary.

@przemektmalon przemektmalon marked this pull request as ready for review April 30, 2024 14:59
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
Copy link
Contributor Author

Pinging @oneapi-src/unified-runtime-bindless-images-write @oneapi-src/unified-runtime-cuda-write @oneapi-src/unified-runtime-maintain @GeorgeWeb.

This PR was merged before and reverted due to DPC++ CI failures and higher priority PRs coming in.

The code from the original PR hasn't changed, and DPC++ CI now passes. Reviews/approvals appreciated!

@kbenzie kbenzie added the ready to merge Added to PR's which are ready to merge label May 2, 2024
@kbenzie
Copy link
Contributor

kbenzie commented May 2, 2024

Since this was already approved before being reverted I'll go ahead an merge this anyway.

@kbenzie kbenzie merged commit ebf873f into oneapi-src:main May 2, 2024
51 checks passed
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.

3 participants