Bug 247324 - [run-webkit-tests] Extra thread spawned for each worker
Summary: [run-webkit-tests] Extra thread spawned for each worker
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jonathan Bedard
URL:
Keywords: InRadar
Depends on: 247409
Blocks:
  Show dependency tree / graph
 
Reported: 2022-11-01 10:35 PDT by Jonathan Bedard
Modified: 2022-11-03 12:20 PDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Bedard 2022-11-01 10:35:33 PDT 
multiprocessing.Queue spawns a thread, which is used to support features we don't use. Newer versions of Python define a "SimpleQueue" which has the set of features we need, but Python 3.9 doesn't support 'timeout' and 'block' on the 'get()' function, which we need. These are relatively simple to implement ourselves, though, which is what fixing this bug will entail.
Comment 1 Radar WebKit Bug Importer 2022-11-01 10:36:02 PDT 
<rdar://problem/101813483>
Comment 2 Jonathan Bedard 2022-11-01 10:42:57 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/6001
Comment 3 EWS 2022-11-02 17:49:59 PDT 
Committed 256262@main (db09bfbcf7d0): <https://commits.webkit.org/256262@main>

Reviewed commits have been landed. Closing PR #6001 and removing active labels.
Comment 4 WebKit Commit Bot 2022-11-02 22:44:32 PDT 
Re-opened since this is blocked by bug 247409
Comment 5 Jonathan Bedard 2022-11-03 12:20:25 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/6096