Skip to content

fix(gpuagent): widen ECC block iterator to uint64_t for amdsmi 27.1.0 - #97

Merged
sarat-k merged 1 commit into
ROCm:mainfrom
bhatturu:user/bhatturu/gpuop-1081-ecc-block-overflow
Sep 24, 2026
Merged

sarat-k merged 1 commit into
ROCm:mainfrom
bhatturu:user/bhatturu/gpuop-1081-ecc-block-overflow

Conversation

@bhatturu

Copy link
Copy Markdown
Contributor

amdsmi PR rocm-systems#10973 expanded AMDSMI_GPU_BLOCK_LAST from bit 18 (MPIO) to bit 38 (UCIE_PCS). The uint32_t ECC loop iterator overflows at bit 31 and wraps to 0, creating an infinite loop (same bug RDC hit and fixed per rocm-systems#11722 changelog). Widen to uint64_t in all 4 sites. Update vendored amdsmi libs+header to 27.1.0+4c2f7505.

Motivation

Technical Details

Test Plan

Test Result

Submission Checklist

amdsmi PR rocm-systems#10973 expanded AMDSMI_GPU_BLOCK_LAST from bit 18
(MPIO) to bit 38 (UCIE_PCS). The uint32_t ECC loop iterator overflows
at bit 31 and wraps to 0, creating an infinite loop (same bug RDC hit
and fixed per rocm-systems#11722 changelog). Widen to uint64_t in all 4
sites. Update vendored amdsmi libs+header to 27.1.0+4c2f7505.
@spraveenio
spraveenio requested a review from sarat-k September 24, 2026 18:03

@rsrikanth86 rsrikanth86 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@sarat-k
sarat-k merged commit 26807f9 into ROCm:main Sep 24, 2026
2 checks passed
spraveenio pushed a commit to ROCm/device-metrics-exporter that referenced this pull request Sep 24, 2026
…#1631) (#659)

- ROCM_VERSION 10.1.0rc1 -> 10.1.0rc2
- GPUAGENT_COMMIT -> 26807f975a35 (ROCm/gpu-agent#97, ECC block
  iterator uint32 -> uint64 for amdsmi 27.1.0 block expansion)
- v1.5.3 release notes

(cherry picked from commit b51f214de47a4cf17515e7ef34b924a721d2bbc1)

Co-authored-by: Bhanu Kiran Atturu <bhatturu@amd.com>
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