Skip to content

Conversation

@colesbury
Copy link
Contributor

@colesburycolesbury commented Feb 26, 2025

The race condition with free_threadstate and daemon threads exists in both the free threading and default builds. We were missing a suppression in the default build.

The race condition with `free_threadstate` and daemon threads exists in both the free threading and default builds. We were missing a suppression in the default build.
@colesburycolesbury requested a review from mpageFebruary 26, 2025 20:41
@colesburycolesbury changed the title gh-124878: Add temporary suppression for free_threadstategh-124878: Add temporary TSAN suppression for free_threadstateFeb 26, 2025
@colesburycolesbury marked this pull request as ready for review February 26, 2025 21:08
@colesburycolesbury merged commit cc17307 into python:mainFeb 28, 2025
48 checks passed
@miss-islington-app
Copy link

Thanks @colesbury for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

@colesburycolesbury deleted the gh-124878-tsan-skip branch February 28, 2025 14:27
@miss-islington-app
Copy link

Sorry, @colesbury, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker cc17307faaa248535c65f6a7668e06dc8ef04575 3.13 

colesbury added a commit to colesbury/cpython that referenced this pull request Feb 28, 2025
…dstate (pythongh-130602) The race condition with `free_threadstate` and daemon threads exists in both the free threading and default builds. We were missing a suppression in the default build. (cherry picked from commit cc17307) Co-authored-by: Sam Gross <[email protected]>
@bedevere-app
Copy link

GH-130687 is a backport of this pull request to the 3.13 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.13 bugs and security fixes label Feb 28, 2025
colesbury added a commit that referenced this pull request Feb 28, 2025
…gh-130602) (gh-130687) The race condition with `free_threadstate` and daemon threads exists in both the free threading and default builds. We were missing a suppression in the default build. (cherry picked from commit cc17307)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@colesbury@mpage