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

[OpenCL] Fix USM alignment error check to occur always and return nulllptr #2011

Merged
merged 1 commit into from
Aug 27, 2024

Conversation

nrspruit
Copy link
Contributor

  • USM calls to alloc must return a nullptr and error given invalid alignment. This cannot be relied upon in the validation layer.

@nrspruit nrspruit force-pushed the fix_opencl_usm_align_check branch 3 times, most recently from 658c0a8 to be90943 Compare August 26, 2024 21:44
@nrspruit nrspruit added the v0.10.x Include in the v0.10.x release label Aug 27, 2024
…lptr

- USM calls to alloc must return a nullptr and error given invalid
  alignment. This cannot be relied upon in the validation layer.

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@kbenzie kbenzie marked this pull request as ready for review August 27, 2024 12:16
@kbenzie kbenzie requested a review from a team as a code owner August 27, 2024 12:16
@kbenzie kbenzie added the ready to merge Added to PR's which are ready to merge label Aug 27, 2024
@omarahmed1111 omarahmed1111 merged commit b766009 into oneapi-src:main Aug 27, 2024
74 checks passed
omarahmed1111 added a commit to omarahmed1111/unified-runtime that referenced this pull request Sep 10, 2024
…_check

[OpenCL] Fix USM alignment error check to occur always and return nulllptr
omarahmed1111 added a commit to omarahmed1111/unified-runtime that referenced this pull request Sep 12, 2024
…_check

[OpenCL] Fix USM alignment error check to occur always and return nulllptr
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Added to PR's which are ready to merge v0.10.x Include in the v0.10.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants