Skip to content

gh-117657: Fix missing atomic in dict_resize - #119312

Merged
DinoV merged 1 commit into
python:mainfrom
DinoV:nogil_dict_resize
May 22, 2024
Merged

gh-117657: Fix missing atomic in dict_resize#119312
DinoV merged 1 commit into
python:mainfrom
DinoV:nogil_dict_resize

Conversation

@DinoV

@DinoVDinoV commented May 21, 2024

Copy link
Copy Markdown
Contributor

Use atomic to assign to PyDictValues->valid like we do elsewhere

@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.

We should backport this to 3.13

@DinoV

Copy link
Copy Markdown
ContributorAuthor

We should backport this to 3.13

Backport to 3.13: #119416

@DinoV
DinoV merged commit 2b3fb76 into python:mainMay 22, 2024
@miss-islington-app

Copy link
Copy Markdown

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 22, 2024
Fix missing atomic in dict_resize
(cherry picked from commit 2b3fb76)
Co-authored-by: Dino Viehland <dinoviehland@meta.com>
@bedevere-app

Copy link
Copy Markdown

GH-119417 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 May 22, 2024
DinoV added a commit that referenced this pull request May 22, 2024
)
gh-117657: Fix missing atomic in dict_resize (GH-119312)
Fix missing atomic in dict_resize
(cherry picked from commit 2b3fb76)
Co-authored-by: Dino Viehland <dinoviehland@meta.com>
@JelleZijlstraJelleZijlstra mentioned this pull request May 28, 2024
@DinoV
DinoV deleted the nogil_dict_resize branch May 31, 2024 18:21
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
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.

2 participants

@DinoV@colesbury