Skip to content

fix(hipblaslt): Reuse host arrays to avoid scaling host pointers by block count - #11514

Open
pmckinno wants to merge 1 commit into
developfrom
users/pmckinno/oom_reuse_staging
Open

fix(hipblaslt): Reuse host arrays to avoid scaling host pointers by block count#11514
pmckinno wants to merge 1 commit into
developfrom
users/pmckinno/oom_reuse_staging

Conversation

@pmckinno

Copy link
Copy Markdown
Collaborator

Motivation

Using rotating buffers with smaller problem sizes results in high block counts. Reuse host arrays instead of scaling by block count to reduce memory consumption.

JIRA ID: AIHPBLAS-4180

…lock count

Using rotating buffers with smaller problem sizes results in high block counts. Reuse host arrays instead of scaling by block count to reduce memory consumption.

JIRA ID: AIHPBLAS-4180
@pmckinno
pmckinno requested a review from a team as a code owner August 31, 2026 23:37
@pmckinno pmckinno changed the title fix(hipblaslt): Reuse host arrays to avoid scaling host pointers by lock count fix(hipblaslt): Reuse host arrays to avoid scaling host pointers by block count Aug 31, 2026
@therock-pr-bot

therock-pr-bot Bot commented Aug 31, 2026

Copy link
Copy Markdown

✅ All Checks Passed — Ready for Review

Check Status Details
📝 PR Description ✅ Pass
Forbidden Files ✅ Pass
🧪 Unit Test ✅ Pass PR does not contain code files — Unit Test auto-passed
🔎 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.

🙋 Wish to Override Policy?

@therock-pr-bot

Copy link
Copy Markdown

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

@codecov

codecov Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

❌ Your project check has failed because the head coverage (35.22%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #11514   +/-   ##
========================================
  Coverage    69.35%   69.35%           
========================================
  Files         2810     2810           
  Lines       464609   464609           
  Branches     68557    68557           
========================================
  Hits        322227   322227           
  Misses      118866   118866           
  Partials     23516    23516           
Flag Coverage Δ *Carryforward flag
TensileLite-CPP 38.33% <ø> (ø) Carriedforward from 630a341
TensileLite-Unit 76.15% <ø> (ø) Carriedforward from 630a341
hipBLAS 90.62% <ø> (ø) Carriedforward from 630a341
hipBLASLt 35.22% <ø> (ø)
hipCUB 82.68% <ø> (ø) Carriedforward from 630a341
hipDNN 86.85% <ø> (ø) Carriedforward from 630a341
hipFFT 44.78% <ø> (ø) Carriedforward from 630a341
hipRAND 76.12% <ø> (ø) Carriedforward from 630a341
hipSOLVER 69.03% <ø> (ø) Carriedforward from 630a341
hipSPARSE 86.99% <ø> (ø) Carriedforward from 630a341
rocBLAS 48.22% <ø> (ø) Carriedforward from 630a341
rocFFT 46.60% <ø> (ø) Carriedforward from 630a341
rocRAND 56.91% <ø> (ø) Carriedforward from 630a341
rocSOLVER 77.32% <ø> (ø) Carriedforward from 630a341
rocSPARSE 74.60% <ø> (ø) Carriedforward from 630a341
rocThrust 91.60% <ø> (ø) Carriedforward from 630a341

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

🚀 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.

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