Skip to content

Conversation

@ronlieb
Copy link
Collaborator

@ronlieb ronlieb commented Nov 2, 2025

…ockSize for MI2xx

Recovers performance for CQE testing of 513.soma on mi2xx. Disavles adjustment for mi2xx and enables for all others.

see:
https://ontrack-internal.amd.com/browse/SWDEV-553832

…ockSize for MI2xx

Recovers performance for CQE testing of 513.soma on mi2xx.
Disavles adjustment for mi2xx and enables for all others.

see:
https://ontrack-internal.amd.com/browse/SWDEV-553832
@z1-cciauto
Copy link
Collaborator

@ronlieb ronlieb requested a review from jplehr November 3, 2025 00:02
Comment on lines +5057 to +5061
{"MI250X",{.OMPX_UseMultipleSdmaEngines = true,
.OMPX_AdjustNumTeamsForXteamRedSmallBlockSize=0}},
{"MI250X/MI250",{
.OMPX_UseMultipleSdmaEngines = true,
.OMPX_AdjustNumTeamsForXteamRedSmallBlockSize=0}},
Copy link

Choose a reason for hiding this comment

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

why 2 entries for mi250x?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

r11, a supermicro, returns the following
Marketing Name: AMD Instinct MI250X/MI250

whereas lockhart returns
Marketing Name: AMD Instinct MI250X

Copy link

Choose a reason for hiding this comment

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

Interesting, thanks. I did not expect the r11 GPUs to have mi250X in their names.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

to add to the confusion:
lockhart mi250X has 110 CUs
r11 mi250X/MI250 has 104 CUs
r9 mi210 has 104 CUs

Copy link

Choose a reason for hiding this comment

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

yeah, I looked up the specs.
lockhart has proper mi250X. That's top-of-the-line 110 CUs in OAM form factor.
mi250 is the same number of CUs (104) as mi210 but in OAM form factor.
mi210 is PCIe.

Copy link

@dhruvachak dhruvachak left a comment

Choose a reason for hiding this comment

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

LGTM.

@ronlieb ronlieb merged commit 59ef54a into amd-staging Nov 3, 2025
6 checks passed
@ronlieb ronlieb deleted the amd/dev/rlieberm/swdev-553832 branch November 3, 2025 15:48
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.

6 participants