Skip to content

Fix amdrocm-amdsmi7.11 dependency error in amdrocm-test rpm#4033

Merged
jonatluu merged 3 commits intomainfrom
users/jonatluu/fix_dependency
Mar 19, 2026
Merged

Fix amdrocm-amdsmi7.11 dependency error in amdrocm-test rpm#4033
jonatluu merged 3 commits intomainfrom
users/jonatluu/fix_dependency

Conversation

@jonatluu
Copy link
Contributor

@jonatluu jonatluu commented Mar 18, 2026

Motivation

Resolve amdrocm-amdsmi7.11 dependency error seen when installing amdrocm-test on rpm

Technical Details

Adding direct dependency to amdrocm-amdsmi package in test packages for rpm

Test Plan

Install rpm test packages "amdrocm-blas-test", "amdrocm-dnn-test", or "amdrocm-rccl-test", and ensure amdrocm-amdsmi7.11 is no logner used

Test Result

passed: https://github.com/ROCm/TheRock/actions/runs/23258835228

Submission Checklist

nunnikri pushed a commit that referenced this pull request Mar 18, 2026
## Motivation

This pull request updates the Linux packaging configuration to ensure
that the `amdrocm-amdsmi` package is included as a dependency for
several components. This change helps guarantee that the required system
management interface is available during installation.

Cherrypick of #4033


## Technical Details

Dependency updates:

* Added `amdrocm-amdsmi` to both `DEBDepends` and `RPMRequires` lists
for the main package, ensuring it is installed alongside
`amdrocm-runtime` and `amdrocm-blas-devel`
(`build_tools/packaging/linux/package.json`).
* Included `amdrocm-amdsmi` as a dependency for profiler-related
packages, alongside `amdrocm-runtime`, `amdrocm-sysdeps`, and
`amdrocm-profiler-base` (`build_tools/packaging/linux/package.json`).
* Added `amdrocm-amdsmi` to the dependency lists for RCCL-related
packages, together with `amdrocm-base` and `amdrocm-profiler-base`
(`build_tools/packaging/linux/package.json`).

## Test Plan

<!-- Explain any relevant testing done to verify this PR. -->

## Test Result

<!-- Briefly summarize test outcomes. -->

## Submission Checklist

- [ ] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.

---------

Co-authored-by: Jonathan Luu <jonatluu@amd.com>
radhaksri added a commit that referenced this pull request Mar 19, 2026
 #4033 #4095 #4096 #4097 #4099 #4100 #4102 #4104 #4106 #4108 #4110 #4111
@jonatluu jonatluu merged commit 8bca294 into main Mar 19, 2026
354 of 404 checks passed
@jonatluu jonatluu deleted the users/jonatluu/fix_dependency branch March 19, 2026 16:51
@github-project-automation github-project-automation bot moved this from TODO to Done in TheRock Triage Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants