Skip to content

CI: re-skip AOTI tests to validate they cause the macOS hang#19880

Open
SS-JIA wants to merge 3 commits into
mainfrom
pr19880
Open

CI: re-skip AOTI tests to validate they cause the macOS hang#19880
SS-JIA wants to merge 3 commits into
mainfrom
pr19880

Conversation

@SS-JIA
Copy link
Copy Markdown
Contributor

@SS-JIA SS-JIA commented May 29, 2026

Summary:
Three CI runs with AOTI tests enabled all showed the same pattern:
tests pass normally up to ~55%, then 38-42 minutes of complete silence
before job cancellation. Faulthandler cannot fire during the hang
(no dump appears), confirming the thread is blocked in native C/C++
code — consistent with the AOTI dlopen / compiler invocation path.

Re-introduce the AOTI skips from the first diagnostic commit. If the
job completes without timeout, this confirms the AOTI tests are the
sole cause of the hang.

Co-Authored-By: Claude noreply@anthropic.com


Stack created with Sapling. Best reviewed with ReviewStack.

SS-JIA and others added 3 commits May 29, 2026 13:26
Summary:
The macOS unittest job has been hitting its timeout for several runs in a
row with the progress bar frozen partway through pytest. Add
`pytest-timeout` so a stuck test fails with a nodeid and per-thread stack
trace, and set `faulthandler_timeout=180` so every xdist worker dumps its
threads every 3 minutes while tests are still running -- this surfaces the
hung test as it develops, not just at termination.

Diagnostic runs identified the hang fingerprint: tests stuck inside
`torch._inductor.package._package.__call__` (line 736), the AOTI-packaged
`.so` invocation. The same stack was observed for tests in
`examples/models/llama3_2_vision/{preprocess,vision_encoder}` and almost
certainly affects the other AOTI call sites we ship. Local M1 reruns of
the affected tests with the CI-pinned torch wheel all pass quickly, so
the hang is CI-environment-specific (suspect: AOTI dlopen under heavy
xdist + coverage contention on the macos-m1-stable runner).

Skip every AOTI-invoking test on macOS CI only, via
`unittest-macos-cmake.sh` -- linux, windows, and local dev continue to
run them. The skip covers the three vision tests (whole files) plus the
specific `*_aoti` methods in `extension/llm/modules/test/test_*.py`.
Job timeout drops back to 30 minutes for fast iteration; pytest
`--timeout=1500` gives any single test 25 minutes before it is treated
as hung.

Co-Authored-By: Claude <noreply@anthropic.com>
Summary:
Previous experiment used `-n 1` which still spawns an xdist worker
process that buffers all output. The CI logs showed 462 tests' worth of
progress dots flushed at a single timestamp, making it impossible to
identify which test was hanging.

Switch to `-p no:xdist` so tests run in the main process with
unbuffered output, and add `-v` so each test name prints as it
starts/completes. Combined with `faulthandler_timeout=180`, this will
show exactly which test is running when the hang occurs.

Faulthandler dumps from two prior CI runs pointed to the same test
position (~26% through the suite) but caught different XNNPACK passes
mid-retrace (FuseBatchNormPass in one, RemoveRedundantCopyPass in the
other). The common pattern is `super().call()` retracing large delegate
subgraphs — suspect is `test_all_models_with_recipes` running ResNet50
/ ViT / DeepLabV3 through the full XNNPACK pass pipeline.

Co-Authored-By: Claude <noreply@anthropic.com>
Summary:
Three CI runs with AOTI tests enabled all showed the same pattern:
tests pass normally up to ~55%, then 38-42 minutes of complete silence
before job cancellation. Faulthandler cannot fire during the hang
(no dump appears), confirming the thread is blocked in native C/C++
code — consistent with the AOTI `dlopen` / compiler invocation path.

Re-introduce the AOTI skips from the first diagnostic commit. If the
job completes without timeout, this confirms the AOTI tests are the
sole cause of the hang.

Co-Authored-By: Claude <noreply@anthropic.com>
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented May 29, 2026

🔗 Helpful Links

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

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

❌ 1 New Failure, 268 Pending

As of commit a59a5bf with merge base 88faab2 (image):

NEW FAILURE - The following job has failed:

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

@linux-foundation-easycla
Copy link
Copy Markdown

CLA Not Signed

@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 May 29, 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.

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.

1 participant