Repository navigation
[ROCm] gfx1250 optimizations and model enablement #58800
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
+861
−74
Merged
Changes from all commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
5664186
Triton version bump
jpvillam-amd e1b8021
Perf pass 1
JadenMathias 1ea8c62
experimental v4 preshuffle
24b9fa0
Push up rocm version
jpvillam-amd 864ea32
Update docker args
JadenMathias cf53bc2
Narrow torch and rocm sdk device to 1250
JadenMathias e5c6438
Move gfx1250 SITU handling to oracle mxfp4
JadenMathias f7f6935
Use native AITER shuffle for all gfx1250 MXFP4 MoE activations
JadenMathias 2612751
Enable AITER a4w4 MoE on gfx1250 for DeepSeek R1
JadenMathias e8fdf03
Enable DSV4F Aiter MoE
JadenMathias 6666981
gfx1250 tp4 e2e patch (#1223)
omuhamma 366cb28
Add aiter Triton a4w4 class
JadenMathias d9820be
Fix transpose
JadenMathias bdb34b3
Fix 2D shapes
JadenMathias 0af9f6b
enabling a4w4 moe triton path
9194ecc
Match atom implementation
JadenMathias fc22aa9
Fix DeepSeek-V4-Flash on gfx1250 by skipping bpreshuffle GEMM
d53c753
enable dsr1 path
55c71f2
remove amdsmi build, should be there by default
a18b6f5
update rock version and add amdsmi pythonpath
e761f04
Fix GPT oss mxfp4 stride
JadenMathias 239703f
Fix Minimax-m3 routing, activation and weight prep
JadenMathias 72f307b
Clean up
jpvillam-amd d859357
Remove bench file
jpvillam-amd 685fa05
pull bench, user aiter api for tunned gemms
jpvillam-amd 20262a7
gemm_config bug
jpvillam-amd 455839c
use better rocmsdk10 version (#1296)
JadenMathias 334fd92
deleting _get_padding_mask(), aiter commit addressing the issue alrea…
1f8386a
Pin ainic version
JadenMathias 3481bbc
Clean up
jpvillam-amd 60fc8f4
Drop K3 gfx1250 enablement, upstreaming separately
jpvillam-amd 9e6be53
DSV4 Core dump fix
JadenMathias 3a173d2
Split AiterW4A16ExpertsMonolithic and AiterW4A4ExpertsMonolithic out …
JadenMathias 24df9ad
Backend selection fix (#1320)
JadenMathias cedef94
sanitize router logits and pass an fp32 bias to aiter W4A4 MoE routin…
danichan-mkm e7d3a4f
Pass routing replay to Monolithic A4W4 class
JadenMathias e9ff539
small spelling error
jpvillam-amd 7025e86
Linter changes
jpvillam-amd 9e21a93
Change aiter pin commit to fix build
JadenMathias ba5dd3f
Review and transposed needs to be misspelled
6aecd06
Avoid the misspell, pass positionally
c886e90
Merge branch 'main' into gfx1250_perf1
jpvillam-amd 8561eda
Drop gpt-oss RMSNorm+pad custom op in favor of fuse_act_padding
5d86664
Merge branch 'main' into gfx1250_perf1
jpvillam-amd 480bc0c
Seperate Priority backend list for gfx1250 (#1362)
JadenMathias 5499c6e
Merge branch 'main' into gfx1250_perf1
jpvillam-amd 8158cbc
linter after merge
aacd2ea
Merge branch 'main' into gfx1250_perf1
jpvillam-amd f500858
Merge branch 'main' into gfx1250_perf1
jpvillam-amd 63040d7
Fixing merge
f319a98
Add new Enum to handle gfx1250 backends (#1365)
JadenMathias f234df4
Merge branch 'main' into gfx1250_perf1
jpvillam-amd 5ba5536
Merge branch 'main' into gfx1250_perf1
jpvillam-amd b97acf1
Merge branch 'main' into gfx1250_perf1
jpvillam-amd 3b492a3
Merge branch 'main' into gfx1250_perf1
jpvillam-amd 0d84379
Merge branch 'main' into gfx1250_perf1
jpvillam-amd ddf7eb6
Review comments
8d048d8
More review comments
f66ef0b
Remove from priority backend (#1374)
JadenMathias 48d006a
Review meeting comments
0b0ae59
Merge branch 'main' into gfx1250_perf1
jpvillam-amd 9fbb90e
Merge branch 'main' into gfx1250_perf1
jpvillam-amd e8a04fc
Merge branch 'main' into gfx1250_perf1
jpvillam-amd File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: should we consider creating a
AiterBpreshuffledFp8BlockScaledMMKernelbackend?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just trying to keep the PR length somewhat manageable but that is a good suggestion I can take it as a follow up