Skip to content

Add Radeon AI PRO R9700S, R9600D, and W7900S GPU entries#194

Merged
pirabhur merged 4 commits into
ROCm:masterfrom
bhatnitish:rocm/KUBE-22-r9700s-product-name
Jun 17, 2026
Merged

Add Radeon AI PRO R9700S, R9600D, and W7900S GPU entries#194
pirabhur merged 4 commits into
ROCm:masterfrom
bhatnitish:rocm/KUBE-22-r9700s-product-name

Conversation

@bhatnitish

Copy link
Copy Markdown
Contributor

Summary

  • Adds amdgpu.ids entries in the node labeller for Radeon AI PRO R9700S (7551, C1), R9600D (7551, C8), and Radeon PRO W7900S (744A, 00)
  • Adds amdgpu.ids entry in the device plugin container image for R9700S (7551, C1)
  • These entries are not yet in upstream libdrm, so hardcoding them ensures correct GPU product name resolution

bhatnitish and others added 2 commits June 15, 2026 21:00
The R9700S revision (device 7551, rev C1) is not present in upstream
libdrm's amdgpu.ids, causing amdgpu_get_marketing_name() to return
a generic "AMD Radeon Graphics" name. This results in the node labeller
reporting amd.com/gpu.product-name=AMD_Radeon_Graphics instead of the
correct product name.

Add the entry to labeller.Dockerfile following the existing pattern
for other missing device IDs (MI325X VF, MI100, W6900X, W6800X,
MI308X HF VF).

Fixes: KUBE-22

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
Also backfills R9700S entry into ubi-labeller.Dockerfile which was
missing it.

Fixes: KUBE-22
@bhatnitish bhatnitish force-pushed the rocm/KUBE-22-r9700s-product-name branch from bb83fe1 to 979e46b Compare June 15, 2026 21:00

@yansun1996 yansun1996 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Update the product name for device 744A (rev 00) from "AMD Radeon PRO W7900S"
to "AMD Radeon PRO W7900 Dual Slot" in both labeller Dockerfiles.

@spraveenio spraveenio left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pirabhur pirabhur merged commit 325a6ea into ROCm:master Jun 17, 2026
4 checks passed
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.

4 participants