Skip to content

[SYCL][Level Zero] Don't use ZE_MEMORY_ADVICE_SET_READ_MOSTLY for read-only USM #7601

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 2 commits into from
Dec 2, 2022

Conversation

aelovikov-intel
Copy link
Contributor

There are issues in the underlying Level Zero RT causing stability problems. This partially reverts #7496.

…d-only USM

There are issues in the underlying Level Zero RT causing stability
problems. This partially reverts intel#7496.
@aelovikov-intel aelovikov-intel requested a review from a team as a code owner November 30, 2022 23:45
aelovikov-intel added a commit to aelovikov-intel/llvm-test-suite that referenced this pull request Nov 30, 2022
…read-only USM

There are issues in the underlying Level Zero RT causing stability problems.
This partially reverts intel#1409 and is
needed for intel/llvm#7601.
@aelovikov-intel
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1426

@aelovikov-intel aelovikov-intel marked this pull request as draft December 1, 2022 18:59
@aelovikov-intel aelovikov-intel marked this pull request as ready for review December 1, 2022 19:48
@aelovikov-intel
Copy link
Contributor Author

Failures:

SYCL :: AtomicRef/max_local.cpp - known, #7604
SYCL :: UserDefinedReductions/user_defined_reductions.cpp - known, being disabled in intel/llvm-test-suite#1429

@aelovikov-intel
Copy link
Contributor Author

@intel/llvm-gatekeepers , this PR is ready and needs to be merged together with intel/llvm-test-suite#1426.

againull pushed a commit to intel/llvm-test-suite that referenced this pull request Dec 2, 2022
…read-only USM (#1426)

There are issues in the underlying Level Zero RT causing stability problems.
This partially reverts #1409 and is
needed for intel/llvm#7601.
@againull againull merged commit 5a140b3 into intel:sycl Dec 2, 2022
aelovikov-intel added a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…read-only USM (intel/llvm-test-suite#1426)

There are issues in the underlying Level Zero RT causing stability problems.
This partially reverts intel/llvm-test-suite#1409 and is
needed for intel#7601.
@aelovikov-intel aelovikov-intel deleted the disable-memadvise branch April 7, 2023 15:56
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.

3 participants