Skip to content

fix(rocsparse): Fix SpMM batching with CSR rowsplit algorithm when batch count >65536#9933

Open
jsandham wants to merge 4 commits into
ROCm:developfrom
jsandham:spmm_batched_rowsplit2
Open

fix(rocsparse): Fix SpMM batching with CSR rowsplit algorithm when batch count >65536#9933
jsandham wants to merge 4 commits into
ROCm:developfrom
jsandham:spmm_batched_rowsplit2

Conversation

@jsandham

Copy link
Copy Markdown
Contributor

Motivation

Currently we use the y grid dimension to perform batch computation with SpMM row-split algorithm. This means that if the number of batches exceeds 65536 we will get launch errors. This PR fixes that by looping over the batch count in blocks of 65536 batches.

JIRA ID: AISPARSE-605

@jsandham
jsandham requested a review from a team as a code owner July 24, 2026 16:15
@therock-pr-bot

therock-pr-bot Bot commented Jul 24, 2026

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
🌿 Branch Name ✅ Pass
📝 PR Title/Description ✅ Pass
Forbidden Files ✅ Pass
🧪 Unit Test ✅ Pass
🔎 pre-commit ✅ Pass
🚫 Draft PR 🔜 To Be Enabled
🚩 Feature Flag 🔜 To Be Enabled
📊 Code Coverage 🔜 To Be Enabled
🤖 therock-pr-bot ✅ Pass

🎉 All checks passed! This PR is ready for review.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

@therock-pr-bot

Copy link
Copy Markdown

🎉 All checks passed! This PR is ready for review.

@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #9933   +/-   ##
========================================
  Coverage    69.56%   69.56%           
========================================
  Files         2770     2770           
  Lines       452498   452516   +18     
  Branches     66620    66620           
========================================
+ Hits        314760   314778   +18     
  Misses      117292   117292           
  Partials     20446    20446           
Flag Coverage Δ *Carryforward flag
TensileLite 34.31% <ø> (ø) Carriedforward from 3304791
TensileLite-CPP 38.09% <ø> (ø) Carriedforward from 3304791
TensileLite-Unit 64.76% <ø> (ø) Carriedforward from 3304791
hipBLAS 90.62% <ø> (ø) Carriedforward from 3304791
hipBLASLt 34.63% <ø> (ø) Carriedforward from 3304791
hipCUB 82.68% <ø> (ø) Carriedforward from 3304791
hipDNN 86.35% <ø> (ø) Carriedforward from 3304791
hipFFT 47.34% <ø> (ø) Carriedforward from 3304791
hipRAND 76.12% <ø> (ø) Carriedforward from 3304791
hipSOLVER 69.18% <ø> (ø) Carriedforward from 3304791
hipSPARSE 86.27% <ø> (ø) Carriedforward from 3304791
rocBLAS 47.95% <ø> (ø) Carriedforward from 3304791
rocFFT 47.38% <ø> (ø) Carriedforward from 3304791
rocRAND 57.01% <ø> (ø) Carriedforward from 3304791
rocSOLVER 76.92% <ø> (ø) Carriedforward from 3304791
rocSPARSE 72.59% <100.00%> (+0.01%) ⬆️
rocThrust 91.36% <ø> (ø) Carriedforward from 3304791

*This pull request uses carry forward flags. Click here to find out more.

Files with missing lines Coverage Δ
.../rocsparse/library/src/common/rocsparse_common.cpp 90.59% <100.00%> (+0.11%) ⬆️
.../src/level3/rocsparse_csrmm_template_row_split.cpp 76.15% <100.00%> (+0.66%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jsandham jsandham changed the title fix(rocsparse): Fix SpMM batching with rowsplitt algorithm when batch count >65536 fix(rocsparse): Fix SpMM batching with CSR rowsplit algorithm when batch count >65536 Jul 24, 2026
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.

1 participant