Skip to content

Conversation

@PJAvinash
Copy link
Contributor

@PJAvinash PJAvinash commented Feb 4, 2026

Details

Cherry pick PR for Navi 4 tuning

PR in migrated Repo - ROCm/rocm-systems#3052

Work item:
Internal

What were the changes?
Changes to tuning table , Ring vs Tree BW, LL cutoffs

Why were the changes made?
To improve performance

How was the outcome achieved?
By changing algo selection policy weights

Additional Documentation:
What else should the reviewer know?

Approval Checklist

  • Verify the CHANGELOG has been updated, if
    • there are any NCCL API version changes - NO
    • any changes impact library users, and/or -NO
    • any changes impact any other ROCm library - NO

Copy link
Contributor

@corey-derochie-amd corey-derochie-amd left a comment

Choose a reason for hiding this comment

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

LGTM in principle, please only merge if the performance has been validated.

@JeniferC99
Copy link

@JeniferC99 JeniferC99 merged commit 97eb9bd into release/rocm-rel-7.2 Feb 9, 2026
16 of 20 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.

3 participants