Misc cleanup of VM threading code #108171
Conversation
jkotas
commented
Sep 24, 2024
jkotas
commented
- Delete special-casing of thread finalizer
- Factor out Get/SetApartment of unstarted threads
- Delete some dead code
- Fix comments
- Delete special-casing of thread finalizer - Factor out Get/SetApartment of unstarted threads - Delete some dead code - Fix comments
ghost
added
the
|
Tagging subscribers to this area: @mangod9 |
| { | ||
| _mayNeedResetForThreadPool = true; | ||
| } | ||
| _mayNeedResetForThreadPool = true; |
There was a problem hiding this comment.
This was not handling the case of finalizer threads.
| { | ||
| public sealed partial class Thread | ||
| { | ||
| [ThreadStatic] |
|
|
||
| LOG((LF_GC, LL_INFO10, "Finalizer thread starting...\n")); | ||
|
|
||
| #if defined(FEATURE_COMINTEROP_APARTMENT_SUPPORT) && !defined(FEATURE_COMINTEROP) |
There was a problem hiding this comment.
The condition was always false.
| } | ||
| CONTRACTL_END; | ||
|
|
||
| // This method can be called on current thread only |
There was a problem hiding this comment.
Fixed the non-current thread callers to use SetApartmentOfUnstartedThread
|
Motivated by HMF->QCall conversion codereviews |
|
/azp run runtime-coreclr outerloop |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@jkotas, all test legs failed in runtime-coreclr libraries-jitstress/20240924.1. PTAL. |
|
I have looked at some of the failures. The failures are either asserts in the JIT
Or asserts/crashes in libraries that suggest bad codegen: |
|
Similar set of failures was hit in https://dev.azure.com/dnceng-public/public/_build/results?buildId=815402&view=ms.vss-test-web.build-test-results-tab that did not include this change. |
@EgorBo, PTAL. |
|
These failures should be fixed by #108169 |
- Delete special-casing of thread finalizer - Factor out Get/SetApartment of unstarted threads - Delete some dead code - Fix comments
3 participants
