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-224 - Test cases ID clean up and documentation for Device Memory Access #88

Conversation

milos-mozetic
Copy link
Contributor

@milos-mozetic milos-mozetic commented Dec 19, 2022

Added Doxygen comments for all Test Cases regarding the following Device Memory Access APIs:

  • hipDeviceCanAccessPeer
  • hipDeviceEnablePeerAccess
  • hipDeviceDisablePeerAccess
  • hipMemGetAddressRange
  • hipMemcpyPeer
  • hipMemcpyPeerAsync

The goal is to generate HTML pages for Test Plan based on the state of tests source code.
Depends on:

As an addition, an empty group for fully deprecated group Context Management is added.

gargrahul and others added 19 commits October 26, 2022 03:59
Change-Id: I66f0c09e9c7405ec7430b1883e0e89542fdb87a0
Change-Id: I212b82b1b3a78a368b85ea64e338371a34b405f9
Change-Id: Ib455f72b5be77e1a81137d15c07ea41161b16a3e
… definitions

Customized Doxygen configuration named DoxyfileTests is added, as well as separate header file
used for definition of test groups.
Change-Id: Ief96e274f4143e80ceb3e40f04d38ae217777583
Change-Id: I9c03cde09b42c8e3726153c2a177359efc8d6d29
… arrays

- Add resource guards for 2D and 3D allocations
- Add resource guards for arrays
- Add utils for hadling pitched memory
- Add hip_array_common.hh to gather type, size and format mapping
…tAddressRange tests

- Reimplement and expand hipMemcpyPeer/hipMemcpyPeerAsync tests using resource guards
- Implement positive and negative test for hipMemGetAddressRange
…o include #if and #ifdef sections in documentation
…-tests into doxygen_device_memory_access_documentation
@rorake
Copy link
Contributor

rorake commented Jun 21, 2023

@milos-mozetic
Will you please resolve the merge conflicts for this PR?

@mangupta mangupta merged commit 15c9180 into ROCm:develop Jun 28, 2023
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.

6 participants