Skip to content

[cuda] Move the large-M quantized GEMM suites out of unittest-cuda too - #23676

Merged
Gasoonjia merged 2 commits into
mainfrom
gasoonjia/move-large-quantized-gemm-suites
Oct 12, 2026
Merged

Gasoonjia merged 2 commits into
mainfrom
gasoonjia/move-large-quantized-gemm-suites

Conversation

@Gasoonjia

Copy link
Copy Markdown
Contributor

unittest-cuda is cancelled at its 90-minute timeout on every recent main commit (e.g. https://github.com/pytorch/executorch/actions/runs/37991426749/job/114026586806). #23605 moved the quantized GEMM kernel suites into unittest-cuda-quantized-gemm, but the same push also added two new suites that still run in unittest-cuda:

  • test_int4_large_quantized_gemm.py (711dd81e23, ~7.7 min on the A10G)
  • test_int6_large_quantized_gemm.py (f19360763f, ~5.1 min)

On #23635 (https://github.com/pytorch/executorch/actions/runs/38006402026/job/114076255669), unittest-cuda's backends/cuda/tests pytest alone took 62 min, and the job was cancelled during the Gemma 4 tests that follow.

Change

Test plan

CI:

  • unittest-cuda finishes within 90 min.
  • unittest-cuda-quantized-gemm (large-int5-int6-int8-graph-timing) runs both large suites and also stays within 90 min.

@pytorch-bot

pytorch-bot Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/23676

Note: Links to docs will display an error until the docs builds have been completed.

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 10, 2026
@github-actions

Copy link
Copy Markdown

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@Gasoonjia
Gasoonjia deployed to upload-benchmark-results October 10, 2026 03:02 — with GitHub Actions Active
unittest-cuda is cancelled at its 90-minute timeout on every recent main
commit. #23605 moved the quantized GEMM kernel suites into
unittest-cuda-quantized-gemm, but the same push added
test_int4_large_quantized_gemm.py (~7.7 min on the A10G) and
test_int6_large_quantized_gemm.py (~5.1 min), which still run in
unittest-cuda. On #23635 its backends/cuda/tests pytest alone took 62 min and
the job was cancelled during the Gemma 4 tests that follow.

Run both in the second quantized-GEMM shard, renamed
large-int5-int6-int8-graph-timing (that shard took 44 min; the int4 shard
already takes ~60). unittest-cuda ignores them. No test is dropped.
@Gasoonjia
Gasoonjia force-pushed the gasoonjia/move-large-quantized-gemm-suites branch from a1b602f to b3337e1 Compare October 12, 2026 02:03
@Gasoonjia
Gasoonjia deployed to upload-benchmark-results October 12, 2026 03:38 — with GitHub Actions Active
@Gasoonjia
Gasoonjia marked this pull request as ready for review October 12, 2026 18:43
@Gasoonjia
Gasoonjia merged commit c7d8471 into main Oct 12, 2026
238 of 240 checks passed
@Gasoonjia
Gasoonjia deleted the gasoonjia/move-large-quantized-gemm-suites branch October 12, 2026 18:50
@Gasoonjia
Gasoonjia deployed to upload-benchmark-results October 12, 2026 20:24 — with GitHub Actions Active

This branch was successfully deployed

2 active deployments
upload-benchmark-results — 50f7b029 Deployed Oct 12, 2026 by Gasoonjia via upload-benchmark-results #20718
cadence — 50f7b029 Deployed Oct 12, 2026 by Gasoonjia via hifi-op-test / hifi4 #32950
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants