Skip to content

UCT/ROCM: adjust hsa memory type check for new rocm releases - #11236

Merged
yosefe merged 3 commits into
openucx:masterfrom
mshanthagit:pr/hipMallocManaged-fix
Mar 7, 2026
Merged

yosefe merged 3 commits into
openucx:masterfrom
mshanthagit:pr/hipMallocManaged-fix

Conversation

@mshanthagit

Copy link
Copy Markdown
Contributor

Newer rocm releases return HSA_EXT_POINTER_TYPE_RESERVED_ADDR instead of HSA_EXT_POINTER_TYPE_UNKNOWN for hip managed memory.

Add rocm version detection and guard code based on the detected version

What?

Adding functionality to detect rocm version and enable code paths based on the version.

Why?

Some of the hip tests related to "managed memory" are failing with rocm 7.x

How?

It is optional, but for complex PRs, please provide information about the design,
architecture, approach, etc.

Newer rocm releases return HSA_EXT_POINTER_TYPE_RESERVED_ADDR instead of
HSA_EXT_POINTER_TYPE_UNKNOWN for hip managed memory.

Add rocm version detection and guard code based on the detected version
@mshanthagit
mshanthagit force-pushed the pr/hipMallocManaged-fix branch from 0e5689e to 8e00804 Compare March 5, 2026 22:33
Comment thread src/uct/rocm/base/rocm_base.c Resolved
Comment thread config/m4/rocm.m4 Resolved
Comment thread config/m4/rocm.m4 Resolved
@tvegas1

tvegas1 commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

Need to add diff below to unblock CI:

diff --git a/AUTHORS b/AUTHORS
index 9918baff3..5612420e9 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -67,6 +67,7 @@ Leonid Genkin <lgenkin@nvidia.com>
 Lior Paz <liorpa@nvidia.com>
 Luis E. Pena <l31g@hotmail.com>
 Manjunath Gorentla Venkata <manjugv@gmail.com>
+Manu Shantharam <manu.shantharam@amd.com>
 Marek Schimara <Marek.Schimara@bull.net>
 Mark Allen <markalle@us.ibm.com>
 Matthew Baker <bakermb@ornl.gov>

@yosefe
yosefe enabled auto-merge (squash) March 7, 2026 12:19
@yosefe
yosefe merged commit 27acabd into openucx:master Mar 7, 2026
152 checks passed
mshanthagit added a commit to mshanthagit/ucx that referenced this pull request Mar 10, 2026
…#11236)

* UCT/ROCM: adjust hsa memory type check for new rocm releases

Newer rocm releases return HSA_EXT_POINTER_TYPE_RESERVED_ADDR instead of
HSA_EXT_POINTER_TYPE_UNKNOWN for hip managed memory.

Add rocm version detection and guard code based on the detected version

* UCT/ROCM: Fix formatting issue and update the Copyright date

* UCT/ROCM: Addressing review comments
yosefe pushed a commit that referenced this pull request Mar 11, 2026
#11244)

UCT/ROCM: adjust hsa memory type check for new rocm releases (#11236)

* UCT/ROCM: adjust hsa memory type check for new rocm releases

Newer rocm releases return HSA_EXT_POINTER_TYPE_RESERVED_ADDR instead of
HSA_EXT_POINTER_TYPE_UNKNOWN for hip managed memory.

Add rocm version detection and guard code based on the detected version

* UCT/ROCM: Fix formatting issue and update the Copyright date

* UCT/ROCM: Addressing review comments
jeynmann pushed a commit to jeynmann/ucx that referenced this pull request Mar 17, 2026
…#11236)

* UCT/ROCM: adjust hsa memory type check for new rocm releases

Newer rocm releases return HSA_EXT_POINTER_TYPE_RESERVED_ADDR instead of
HSA_EXT_POINTER_TYPE_UNKNOWN for hip managed memory.

Add rocm version detection and guard code based on the detected version

* UCT/ROCM: Fix formatting issue and update the Copyright date

* UCT/ROCM: Addressing review comments
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