Title: gh-152238: Revert gh-150490 and gh-152200. by nascheme · Pull Request #152232 · python/cpython · GitHub
Open Graph Title: gh-152238: Revert gh-150490 and gh-152200. by nascheme · Pull Request #152232 · python/cpython
X Title: gh-152238: Revert gh-150490 and gh-152200. by nascheme · Pull Request #152232 · python/cpython
Description: Revert commits: gh-151593: Fix dead lock in PyDict insert_split_key() (#152200) gh-150490: Raise PyType_Modified for insertion into split dictionary (#150489) For gh-150489, it violates locking discipline and results in deadlocks, gh-151593 is an example of it being hit in CI. The attempted fix of gh-152200 avoids the deadlock but introduces a data-race. The race window is small but can be triggered with pure Python code. Issue: gh-152238
Open Graph Description: Revert commits: gh-151593: Fix dead lock in PyDict insert_split_key() (#152200) gh-150490: Raise PyType_Modified for insertion into split dictionary (#150489) For gh-150489, it violates locking dis...
X Description: Revert commits: gh-151593: Fix dead lock in PyDict insert_split_key() (#152200) gh-150490: Raise PyType_Modified for insertion into split dictionary (#150489) For gh-150489, it violates locking dis...
Opengraph URL: https://github.com/python/cpython/pull/152232
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:c24ff6e8-1b28-f2aa-5c00-a0772178f49d |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | C980:4EB39:3FBDE9A:58992E5:6A50F84E |
| html-safe-nonce | ae7621f4548eb0744bdba2cc87980f992efba706fba533899ed142d957721cbe |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDOTgwOjRFQjM5OjNGQkRFOUE6NTg5OTJFNTo2QTUwRjg0RSIsInZpc2l0b3JfaWQiOiI2MDE1NjEwNTI0NjA3MjQ4NDYyIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 33c70ae870dee6b37a9343d3541cacfaf94334c4bf237e6090ded76ead6a572f |
| hovercard-subject-tag | pull_request:3937666375 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python/cpython/pull/152232/files |
| twitter:image | https://avatars.githubusercontent.com/u/690853?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/690853?s=400&v=4 |
| og:image:alt | Revert commits: gh-151593: Fix dead lock in PyDict insert_split_key() (#152200) gh-150490: Raise PyType_Modified for insertion into split dictionary (#150489) For gh-150489, it violates locking dis... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5266e58c17a510c403505cc811606465e90a881d2007ee7df1c4800d5c659838 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 8d836581d020676cc0ef4a83cd2fe17e4af02c3d |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width