Re-land "[xplat] Update pthreadpool to google/pthreadpool github revision a56dcd7" (with patches)#20654
Re-land "[xplat] Update pthreadpool to google/pthreadpool github revision a56dcd7" (with patches)#20654GregoryComer wants to merge 1 commit into
Conversation
…sion a56dcd7" (with patches) Summary: Attempt pytorch#2 to land the pthreadpool dependency update - D104906625 was reverted in D109744957 due to performance regressions. I've patched back in per-thread wakeup for futex/condvar paths, as the new dispatch regresses when running with our (existing) subset of threads patch. See D107950404 for full performance testing on the new vs old stack. Differential Revision: D109876168
GregoryComer
requested review from
larryliu0820,
manuelcandales and
mergennachin
as code owners
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/20654
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 0d06e58 with merge base d54a0c0 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
meta-cla
Bot
added
the
CLA Signed
|
@GregoryComer has exported this pull request. If you are a Meta employee, you can view the originating Diff in D109876168. |
This PR needs a
|
1 participant
Summary:
Attempt #2 to land the pthreadpool dependency update - D104906625 was reverted in D109744957 due to performance regressions.
I've patched back in per-thread wakeup for futex/condvar paths, as the new dispatch regresses when running with our (existing) subset of threads patch. See D107950404 for full performance testing on the new vs old stack.
Differential Revision: D109876168