Move CoreCLR over to the managed wait subsystem #117788

jkoritzinsky · 2025-07-17T21:36:17Z

Use the shared managed wait subsystem for CoreCLR's managed code instead of the Win32 PAL

Also, remove the named mutex support from the CoreCLR PAL as nothing uses it now.

TODO: Once #117688 is in, we can remove Mutex support from the Win32 PAL entirely

Unblocks #115685

Depends on #117635

dotnet-policy-service · 2025-07-17T21:37:05Z

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

src/libraries/System.Private.CoreLib/src/System/Threading/LowLevelLifoSemaphore.Unix.cs

src/coreclr/vm/threads.cpp

src/coreclr/System.Private.CoreLib/src/System/Threading/Thread.CoreCLR.cs

src/libraries/Common/src/Interop/Unix/System.Native/Interop.GetPageSize.cs

src/coreclr/vm/threads.cpp

… logic is only for externally-attached threads)

src/coreclr/System.Private.CoreLib/src/System/Threading/Thread.CoreCLR.cs

… finalizer.

… Thread.Interrupt on non-Windows (it's not needed any more).

… them

src/coreclr/pal/src/config.h.in

src/coreclr/vm/comsynchronizable.cpp

src/coreclr/vm/threads.cpp

jkotas · 2025-11-06T00:04:02Z

src/coreclr/vm/threads.cpp

+        // finalizer should always park in default domain
+


Suggested change

// finalizer should always park in default domain

obsolete comment. (Also, delete it where it was copy&pasted from.)

src/coreclr/vm/threads.cpp

jkoritzinsky added area-System.Threading runtime-coreclr specific to the CoreCLR runtime labels Jul 17, 2025

dotnet-policy-service bot assigned jkoritzinsky Jul 17, 2025

jkoritzinsky mentioned this pull request Jul 17, 2025

[Android] Add support for named mutex on mobile platforms in CoreCLR #117760

Open

jkoritzinsky added this to the 11.0.0 milestone Jul 17, 2025

jkoritzinsky added the blocked Issue/PR is blocked on something - see comments label Jul 17, 2025

jkoritzinsky force-pushed the coreclr-managed-wait branch from 25446d0 to 63dc9af Compare July 21, 2025 19:44

jkoritzinsky force-pushed the coreclr-managed-wait branch 3 times, most recently from 06de3b2 to 63a1aa6 Compare July 31, 2025 16:44

This was referenced Jul 31, 2025

slow macOS - "##[error]The job running on agent Azure Pipelines 9 ran longer than the maximum time of 60 minutes." dotnet/dnceng#1883

Open

[browser] Wasm.Build.Tests timeout - Timed out after 10s waiting for 'WASM EXIT' #116697

Open

jkoritzinsky mentioned this pull request Aug 5, 2025

Move the implementation of Monitor to managed in CoreCLR #118371

Merged

teo-tsirpanis reviewed Aug 5, 2025

View reviewed changes

src/libraries/System.Private.CoreLib/src/System/Threading/LowLevelLifoSemaphore.Unix.cs Outdated Show resolved Hide resolved

jkotas reviewed Aug 7, 2025

View reviewed changes

src/coreclr/vm/threads.cpp Outdated Show resolved Hide resolved

jkoritzinsky force-pushed the coreclr-managed-wait branch from 63a1aa6 to 2b610bf Compare August 14, 2025 18:54

jkotas reviewed Aug 14, 2025

View reviewed changes

src/coreclr/System.Private.CoreLib/src/System/Threading/Thread.CoreCLR.cs Outdated Show resolved Hide resolved

jkotas reviewed Aug 14, 2025

View reviewed changes

src/libraries/Common/src/Interop/Unix/System.Native/Interop.GetPageSize.cs Outdated Show resolved Hide resolved

jkoritzinsky force-pushed the coreclr-managed-wait branch from 4a27a27 to ac08ce6 Compare August 14, 2025 23:13

jkotas reviewed Aug 14, 2025

View reviewed changes

src/coreclr/vm/threads.cpp Outdated Show resolved Hide resolved

jkotas reviewed Aug 15, 2025

View reviewed changes

src/coreclr/vm/threads.cpp Outdated Show resolved Hide resolved

jkoritzinsky added 5 commits October 30, 2025 11:45

Fix messed up merge commit

4fe3f06

Remove export that no longer exists

8adc8c0

Interrupt the wait subsystem when Interrupt is called.

95a1ed3

Handle the standard .NET-owned thread case (apparently the "detached"…

6595959

… logic is only for externally-attached threads)

Fix invalidnames test now that the implementations are unified.

a563d38

jkotas reviewed Oct 31, 2025

View reviewed changes

src/coreclr/System.Private.CoreLib/src/System/Threading/Thread.CoreCLR.cs Show resolved Hide resolved

jkotas reviewed Oct 31, 2025

View reviewed changes

src/coreclr/System.Private.CoreLib/src/System/Threading/Thread.CoreCLR.cs Show resolved Hide resolved

build-analysis bot mentioned this pull request Oct 31, 2025

Android System.Security.Cryptography test failure: Signing key verifies the CRL #119023

Open

jkoritzinsky added 6 commits October 31, 2025 17:37

Clean up foreign threads in their own helper thread instead of on the…

d6d650c

… finalizer.

Move Thread.Join to managed and remove the call into CoreCLR's native…

9c6c00c

… Thread.Interrupt on non-Windows (it's not needed any more).

Remove support for altertable waits in the PAL now that we don't need…

da6ad45

… them

Delete now-unused SignalObjectAndWait from the PAL

5fda9a6

Set up the foreign cleanup thread on thread attach, not detach.

a0643f9

Remove duplicate Join

8971c9d

am11 reviewed Nov 3, 2025

View reviewed changes

src/coreclr/pal/src/config.h.in Show resolved Hide resolved

am11 reviewed Nov 3, 2025

View reviewed changes

src/coreclr/vm/comsynchronizable.cpp Outdated Show resolved Hide resolved

jkotas reviewed Nov 3, 2025

View reviewed changes

src/coreclr/vm/threads.cpp Outdated Show resolved Hide resolved

jkotas reviewed Nov 3, 2025

View reviewed changes

src/coreclr/vm/threads.cpp Outdated Show resolved Hide resolved

This was referenced Nov 4, 2025

[android] Android.Device_Emulator.JIT.Test failing on emulators with CoreCLR #112633

Open

Failed to install runtime_python_requirements #114924

Open

AppHost test failure: I/O failure when writing extracted files #120653

Open

jkoritzinsky added 3 commits November 5, 2025 21:12

PR feedback from @am11

93a9154

Structure the cleanup thread to work jsut like the finalizer thread.

8510e64

Call HasStarted, that sets up a bunch of stuff.

7be436b

jkotas reviewed Nov 6, 2025

View reviewed changes

src/coreclr/vm/threads.cpp Outdated Show resolved Hide resolved

Don't shut down the cleanup thread. It can die when the process dies

4cf8760

This was referenced Nov 6, 2025

[android-arm64] The Operation will be canceled. The next steps may not contain expected logs. dotnet/dnceng#6408

Open

error : HttpRequestException: The SSL connection could not be established, see inner exception. dotnet/dnceng#5015

Open

jkoritzinsky mentioned this pull request Nov 6, 2025

Certificate issues when Sending To Helix from Apple platforms dotnet/dnceng#6410

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Move CoreCLR over to the managed wait subsystem #117788

Move CoreCLR over to the managed wait subsystem #117788

jkoritzinsky commented Jul 17, 2025 •

edited

Loading

Uh oh!

dotnet-policy-service bot commented Jul 17, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jkotas Nov 6, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Move CoreCLR over to the managed wait subsystem #117788

Are you sure you want to change the base?

Move CoreCLR over to the managed wait subsystem #117788

Conversation

jkoritzinsky commented Jul 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

dotnet-policy-service bot commented Jul 17, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jkotas Nov 6, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

jkoritzinsky commented Jul 17, 2025 •

edited

Loading