gh-129824: fix data races in subinterpreters under TSAN - #135794
Conversation
Sorry, something went wrong.
|
🤖 New build scheduled with the buildbot fleet by @kumaraditya303 for commit be846e8 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F135794%2Fmerge If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again. |
Sorry, something went wrong.
|
This also fixes #116909 so more of test_capi tests can be enabled under free-threading TSAN (will do in follow up). |
Sorry, something went wrong.
colesbury
left a comment
There was a problem hiding this comment.
This looks okay to me, but I don't understand the interaction between type_ready and subinterpreters very well, so I'd appreciate @ericsnowcurrently's review.
Sorry, something went wrong.
ericsnowcurrently
left a comment
There was a problem hiding this comment.
Mostly LGTM. I've left one mild suggestion and I'll leave it up to you.
FWIW, the relationship with subinterpreters is that the builtin types are "initialized" multiple times. There are certain things that only need to be done the first time, when the main interpreter is initialized, and some that need to be done for each interpreter. This is reflected by "initial".
Sorry, something went wrong.
|
Thanks @kumaraditya303 for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14. |
Sorry, something went wrong.
…GH-135794) This fixes the data races in typeobject.c in subinterpreters under free-threading. The type flags and slots are only modified in the main interpreter as all static types are first initialised in main interpreter. (cherry picked from commit b582d75) Co-authored-by: Kumar Aditya <kumaraditya@python.org>
|
GH-136266 is a backport of this pull request to the 3.14 branch. |
Sorry, something went wrong.
|
Thanks @kumaraditya303 for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13. |
Sorry, something went wrong.
|
Sorry, @kumaraditya303, I could not cleanly backport this to 3.13 due to a conflict. cherry_picker b582d751b4968d4534fdb7894e50713676789b2f 3.13 |
Sorry, something went wrong.
…5794) (#136266) gh-129824: fix data races in subinterpreters under TSAN (GH-135794) This fixes the data races in typeobject.c in subinterpreters under free-threading. The type flags and slots are only modified in the main interpreter as all static types are first initialised in main interpreter. (cherry picked from commit b582d75) Co-authored-by: Kumar Aditya <kumaraditya@python.org>
…#135794) This fixes the data races in typeobject.c in subinterpreters under free-threading. The type flags and slots are only modified in the main interpreter as all static types are first initialised in main interpreter.
…#135794) This fixes the data races in typeobject.c in subinterpreters under free-threading. The type flags and slots are only modified in the main interpreter as all static types are first initialised in main interpreter.
…#135794) This fixes the data races in typeobject.c in subinterpreters under free-threading. The type flags and slots are only modified in the main interpreter as all static types are first initialised in main interpreter.
…#135794) This fixes the data races in typeobject.c in subinterpreters under free-threading. The type flags and slots are only modified in the main interpreter as all static types are first initialised in main interpreter.
|
Reminder about backporting. @kumaraditya303 |
Sorry, something went wrong.
…#135794) This fixes the data races in typeobject.c in subinterpreters under free-threading. The type flags and slots are only modified in the main interpreter as all static types are first initialised in main interpreter.
This fixes the data races in typeobject.c in subinterpreters under free-threading. The type flags are only modified in the main interpreter as all static types are first initialized in main interpreter. With the fix the test_interpreter_pool tests have no data races in typeobject.c under TSAN (there are still some unrelated data races in module exec like posix and struct).