Skip to content

Windows wheels: fix smoke test hang by forcing single-threaded execution#18379

Merged
manuelcandales merged 1 commit intorelease/1.2from
manuel/release-only-windows-wheels-fix-C
Mar 20, 2026
Merged

Windows wheels: fix smoke test hang by forcing single-threaded execution#18379
manuelcandales merged 1 commit intorelease/1.2from
manuel/release-only-windows-wheels-fix-C

Conversation

@manuelcandales
Copy link
Copy Markdown
Contributor

pthreadpool's condvar-based synchronization on Windows can deadlock with multiple threads due to a lost-wakeup bug in signal_num_recruited_threads. Reset the threadpool to 1 thread before loading the model to avoid the issue.

…ution

pthreadpool's condvar-based synchronization on Windows can deadlock
with multiple threads due to a lost-wakeup bug in signal_num_recruited_threads.
Reset the threadpool to 1 thread before loading the model to avoid the issue.
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Mar 20, 2026

🔗 Helpful Links

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

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

⏳ No Failures, 85 Pending

As of commit 490d554 with merge base 8c0a60b (image):
💚 Looks good so far! There are no failures yet. 💚

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 Mar 20, 2026
@manuelcandales manuelcandales changed the title Windows wheels: fix XNNPACK test hang by forcing single-threaded execution Windows wheels: fix smoke test hang by forcing single-threaded execution Mar 20, 2026
@manuelcandales manuelcandales merged commit 4679fbf into release/1.2 Mar 20, 2026
408 of 419 checks passed
@manuelcandales manuelcandales deleted the manuel/release-only-windows-wheels-fix-C branch March 20, 2026 21:11
manuelcandales added a commit that referenced this pull request Mar 20, 2026
manuelcandales added a commit that referenced this pull request Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries 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