Skip to content

Add Stream K experimental support for RDNA#1554

Merged
stsokolo merged 20 commits into
developfrom
users/stsokolo/navi_sk_support
Oct 27, 2025
Merged

Add Stream K experimental support for RDNA#1554
stsokolo merged 20 commits into
developfrom
users/stsokolo/navi_sk_support

Conversation

@stsokolo
Copy link
Copy Markdown
Contributor

@stsokolo stsokolo commented Sep 11, 2025

This PR introduces initial changes needed for stream-k RDNA support. Some experimental tuning was conducted for a specific model, and it is working. Now we are conducting additional tuning and doing performance analysis.

Major changes:

  • The s_store instruction needed to be swapped with buffer_store since s_store is not supported on RDNA
  • Additional modifiers were added to rocisa that were missing for gfx11 and gfx12 and an "off" option was added for vgpr
  • Add gfx1100 and gfx1201 architectures to Origami Analytics

@stsokolo stsokolo requested a review from a team as a code owner September 11, 2025 13:58
@stsokolo stsokolo changed the title Initial Stream-K changes for RDNA support Add Stream K support for RDNA Sep 11, 2025
@stsokolo stsokolo changed the title Add Stream K support for RDNA Add Stream K experimental support for RDNA Sep 11, 2025
Comment thread projects/hipblaslt/tensilelite/src/analytical/Hardware.cpp Outdated
Comment thread projects/hipblaslt/tensilelite/src/analytical/Hardware.cpp Outdated
Comment thread projects/hipblaslt/tensilelite/Tensile/Components/StreamK.py Outdated
Comment thread projects/hipblaslt/tensilelite/Tensile/Components/StreamK.py Outdated
@stsokolo stsokolo force-pushed the users/stsokolo/navi_sk_support branch from 31f57ea to 86d94ee Compare September 12, 2025 09:00
@cmingch cmingch requested a review from cliffxzx September 12, 2025 09:11
Comment thread projects/hipblaslt/tensilelite/rocisa/rocisa/include/hardware_caps.hpp Outdated
@stsokolo stsokolo force-pushed the users/stsokolo/navi_sk_support branch 2 times, most recently from 2e259e9 to 814b5df Compare September 22, 2025 09:46
@stsokolo stsokolo force-pushed the users/stsokolo/navi_sk_support branch from 814b5df to 3744e4a Compare September 23, 2025 09:38
cliffxzx
cliffxzx previously approved these changes Sep 23, 2025
Copy link
Copy Markdown
Contributor

@cliffxzx cliffxzx left a comment

Choose a reason for hiding this comment

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

LGTM

@stsokolo stsokolo force-pushed the users/stsokolo/navi_sk_support branch from 3744e4a to e6e67a8 Compare September 30, 2025 10:12
@stsokolo stsokolo requested a review from a team as a code owner September 30, 2025 10:12
Comment thread shared/origami/include/origami/hardware.hpp
@cmingch cmingch requested a review from a team October 1, 2025 04:26
Copy link
Copy Markdown
Contributor

@cmingch cmingch left a comment

Choose a reason for hiding this comment

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

LGTM, please wait for CI to pass

Comment thread projects/hipblaslt/tensilelite/Tensile/SolutionStructs/Solution.py Outdated
@stsokolo stsokolo force-pushed the users/stsokolo/navi_sk_support branch 2 times, most recently from ae9c6ee to 1559492 Compare October 3, 2025 10:10
@chiranjeevipattigidi
Copy link
Copy Markdown

!verify

@stsokolo stsokolo force-pushed the users/stsokolo/navi_sk_support branch from 1559492 to 98cb6d1 Compare October 16, 2025 09:03
@stsokolo stsokolo force-pushed the users/stsokolo/navi_sk_support branch from 2f2fb4b to f8383aa Compare October 16, 2025 16:18
@fjankovi
Copy link
Copy Markdown
Contributor

@AlexBrownAMD @cmingch @ROCm/hipblaslt-reviewers Can we get another review / approval for this PR? Thanks.

Comment thread projects/hipblaslt/tensilelite/Tensile/Components/StreamK.py
@fjankovi
Copy link
Copy Markdown
Contributor

fjankovi commented Oct 23, 2025

Can we please get a review from @ROCm/origami-reviewers? Thanks.
CC @neoblizz

Copy link
Copy Markdown
Member

@neoblizz neoblizz left a comment

Choose a reason for hiding this comment

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

Origami pieces LGTM!

@stsokolo stsokolo force-pushed the users/stsokolo/navi_sk_support branch from f8383aa to 93a8907 Compare October 27, 2025 09:34
@stsokolo stsokolo merged commit b4c7c8a into develop Oct 27, 2025
25 of 28 checks passed
@stsokolo stsokolo deleted the users/stsokolo/navi_sk_support branch October 27, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants