Skip to content

gh-114271: Make PyInterpreterState.threads.count thread-safe in free-threaded builds - #115093

Merged
ericsnowcurrently merged 1 commit into
python:mainfrom
mpage:gh-114271-threads.count
Feb 12, 2024
Merged

gh-114271: Make PyInterpreterState.threads.count thread-safe in free-threaded builds#115093
ericsnowcurrently merged 1 commit into
python:mainfrom
mpage:gh-114271-threads.count

Conversation

@mpage

@mpagempage commented Feb 6, 2024

Copy link
Copy Markdown
Contributor

Use atomics to mutate PyInterpreterState.threads.count

@mpage
mpageforce-pushed the gh-114271-threads.count branch from b22b2f0 to 7274538CompareFebruary 6, 2024 17:37
@mpage
mpage marked this pull request as ready for review February 6, 2024 19:12
@mpage

mpage commented Feb 6, 2024

Copy link
Copy Markdown
ContributorAuthor

@colesbury - Can you add the "skip news" label to this?

@mpage

Copy link
Copy Markdown
ContributorAuthor

@colesbury - Would you take a look at this, please?

@colesburycolesbury left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ericsnowcurrentlyericsnowcurrently left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ericsnowcurrently
ericsnowcurrently merged commit de7d67b into python:mainFeb 12, 2024
fsc-eriker pushed a commit to fsc-eriker/cpython that referenced this pull request Feb 14, 2024
…in free-threaded builds (pythongh-115093)
Use atomics to mutate PyInterpreterState.threads.count.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@mpage@colesbury@ericsnowcurrently