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

EXSWHTEC-70 - Reimplement tests for hipMemPrefetchAsync #3004

Open
wants to merge 35 commits into
base: develop
Choose a base branch
from

Conversation

music-dino
Copy link
Contributor

  • Negative parameter tests
  • Validate that memory is usable on a device after prefetching to it
  • Validate asynchronous behavior
  • Validate round down for base address and roundup for base address + count behavior

music-dino and others added 29 commits September 30, 2022 02:38
- Reimplement Unit_hipMemPrefetchAsync
- Reimplement negative tests
- Implement additional negative tests
- Implement tests for synchronization and rounding behavior
chrispaquot
chrispaquot previously approved these changes Oct 27, 2022
@music-dino music-dino marked this pull request as ready for review November 3, 2022 07:01
@mangupta
Copy link
Contributor

mangupta commented Nov 4, 2022

@music-dino : PR fails on CI. Details shared separately.

@mangupta mangupta added the pr:needs_updates PR initially approved. But needs rework label Nov 4, 2022
@music-dino
Copy link
Contributor Author

@music-dino : PR fails on CI. Details shared separately.

PR updated, please proceed with merge.

@mangupta mangupta added pr:ready_for_ci PR ready to be tested and removed pr:needs_updates PR initially approved. But needs rework labels Nov 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:ready_for_ci PR ready to be tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants