Skip to content

Conversation

@peterjunpark
Copy link
Contributor

@peterjunpark peterjunpark commented Jan 28, 2026

Motivation

Address @gabrpham 's comments in #80

Technical Details

Test Plan

Test Result

Submission Checklist

@peterjunpark peterjunpark requested review from a team and saadrahim as code owners January 28, 2026 22:35
@peterjunpark peterjunpark requested a review from gabrpham January 28, 2026 22:37
Copy link

@gabrpham gabrpham left a comment

Choose a reason for hiding this comment

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

Just one last change here. We took out amd-smi reset -r and want people to just use sudo modprobe -r amdgpu and sudo modprobe amdgpu now.

Copy link

@gabrpham gabrpham left a comment

Choose a reason for hiding this comment

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

Previous review comment was made in error. Everything here is correct now. Thanks!

@peterjunpark peterjunpark merged commit 862a223 into develop Jan 28, 2026
3 checks passed
@peterjunpark peterjunpark deleted the amd-smi-set-mem-partition branch January 28, 2026 23:09
peterjunpark added a commit that referenced this pull request Jan 28, 2026
peterjunpark added a commit that referenced this pull request Jan 28, 2026
peterjunpark added a commit that referenced this pull request Jan 28, 2026
peterjunpark added a commit that referenced this pull request Jan 28, 2026
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