Repository navigation
Stream-k initial support #1035
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
Merged
Stream-k initial support #1035
Changes from 41 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
848b6f6
Add tuning parameters with updated doc comments
AlexBrownAMD 6ff49e1
SK Port: (Host) Debug settings and Grid Selector (#1)
bethune-bryant c85586d
Merge branch 'ROCm:develop' into StreamK
bethune-bryant f25ba0b
Stream-K kernel arguments, metadata, and launch grid code
AlexBrownAMD 1e19915
Increase default max workspace for tensile tuning client
AlexBrownAMD 223aedc
Increase default workspace for bench client (#2)
bethune-bryant e1e38a7
Merge branch 'ROCm:develop' into StreamK
bethune-bryant d0d4bc3
Merge remote-tracking branch 'origin' into StreamK
AlexBrownAMD e9a56b0
Merge remote-tracking branch 'origin/StreamK' into StreamK
AlexBrownAMD 7809986
Update gitignore
AlexBrownAMD cbf4ef3
Option to disable GSU in kernels.
AlexBrownAMD b92f313
UpdatedSolutionStructs, reject cases and golbal accumulation code
AlexBrownAMD ca5221d
Stream-k code gen work in progress
AlexBrownAMD 8f5a915
Some bug fixes and more work on persistent loop
AlexBrownAMD 0d5cec7
Store branches and close persistent loop
AlexBrownAMD 2296748
Add code for workspace SRD, working on partials write, some updates/f…
AlexBrownAMD ac84fa9
Support for predicate arithmetic intensity added on top of streamK br…
adityalj 259f6b4
Write partials and some bug fixes
AlexBrownAMD a100b1f
Grid prediction (#4)
adityalj 5ce3017
Fixup/partials, flag init, bug fixes
AlexBrownAMD 6a50f3d
Enable sk grid and fix tile calculation
AlexBrownAMD 4df66e5
Merge branch 'StreamK' into StreamKGen
AlexBrownAMD fc0f23d
Merge pull request #6 from AlexBrownAMD/StreamKGen
AlexBrownAMD 16fa505
Some build fixes
AlexBrownAMD 9e8fa21
Fix race condition on flag reset
AlexBrownAMD 5642e5c
Flag point to input synchronizer
adityalj 2f882d6
point flag to synchronizer
adityalj 0c5159d
remove space allocated for flags
adityalj 445e1f1
Some bug fixes, general case sgemm tests passing
AlexBrownAMD 8367400
Merge pull request #7 from adityalj/flag_init
AlexBrownAMD f734836
SOme bug fixes
AlexBrownAMD dca3d24
Merge remote-tracking branch 'upstream/develop' into StreamKMerge
AlexBrownAMD 593e98f
Merge fixes for stream-k
AlexBrownAMD 3a96ecf
Some stream-k test files
AlexBrownAMD 2c7b462
Merge remote-tracking branch 'upstream/develop' into StreamKMerge2
AlexBrownAMD 7b84b91
Bug fixes from merge
AlexBrownAMD cfdaf49
Update targets for new tests
AlexBrownAMD 44f2d35
Remove duplicate code and addreject for unsupported config
AlexBrownAMD e7d7d57
If stream-k enabled, force GSU=0
AlexBrownAMD bfbfde1
Bug fixes for SKXCCM and SK=2
AlexBrownAMD e6de95f
Fix race condition when resetting synchronizer flags
AlexBrownAMD fc61d87
Disable LRA backup, not needed for stream-k, and not compatible with …
AlexBrownAMD e304cf7
Small optimization from PR review
AlexBrownAMD 256da9b
Add new test case based on race condition
AlexBrownAMD 3bbeff9
Merge remote-tracking branch 'upstream/develop' into StreamKMerge2
AlexBrownAMD 21abe72
Fix merge issue
AlexBrownAMD 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
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.
Don't we need to change halfDataRegPerVI (line 105).
It refers DestDataType.
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.
I think line 105 is OK. It is just checking if the write target is a half type (and excluding 8-bit types). The isWorkspace parameter makes bpeC use internal compute size instead of external write size so numVgprsPerDataPerVI is correct for the partials write case.