Skip to content

gh-133885: Use locks instead of critical sections for _zstd #134289

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
colesbury merged 18 commits into from
May 23, 2025
Merged

gh-133885: Use locks instead of critical sections for _zstd #134289

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
daa5f43
Use locks instead of critical sections
emmatyping May 19, 2025
0d14dc2
Remove one more critical section
emmatyping May 19, 2025
1b68e1e
Make capsule management threadsafe
emmatyping May 20, 2025
f6602f4
Add a test for sharing a dictionary across threads
emmatyping May 20, 2025
d8854c5
Move lock check in dealloc to an assert
emmatyping May 20, 2025
02fc2b3
Run threading checks on GIL builds too
emmatyping May 20, 2025
edbf444
Remove extra blank line
emmatyping May 20, 2025
bf287cf
Remove extraneous whitespace
emmatyping May 20, 2025
005d1a0
Remove unused includes
emmatyping May 21, 2025
84c5b30
Revert "Remove unused includes"
emmatyping May 21, 2025
ec31019
Put locks around ZstdDict usage
emmatyping May 21, 2025
2dbed4e
Add tests for sharing all types of ZstdDict
emmatyping May 21, 2025
afdabd5
Remove @critical_section
emmatyping May 21, 2025
d8d48c2
Add missing include
emmatyping May 21, 2025
d142aa8
Updates based on Sam's review
emmatyping May 21, 2025
43d3dea
Pass NULL to PyDict_SetDefaultRef
emmatyping May 22, 2025
93dec23
Put locks around relevant functions and use PyDict_GetItemRef
emmatyping May 23, 2025
69a755b
Remove unused decref
emmatyping May 23, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Run threading checks on GIL builds too
  • Loading branch information
@emmatyping
emmatyping committed May 20, 2025
commit 02fc2b3e8ecba886c9e42dccef792ec913a2499f
2 changes: 0 additions & 2 deletions Lib/test/test_zstd.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2432,7 +2432,6 @@ def test_buffer_protocol(self):

class FreeThreadingMethodTests(unittest.TestCase):

@unittest.skipUnless(Py_GIL_DISABLED, 'this test can only possibly fail with GIL disabled')
@threading_helper.reap_threads
@threading_helper.requires_working_threading()
def test_compress_locking(self):
Expand Down Expand Up @@ -2469,7 +2468,6 @@ def run_method(method, input_data, output_data):
actual = b''.join(output) + rest2
self.assertEqual(expected, actual)

@unittest.skipUnless(Py_GIL_DISABLED, 'this test can only possibly fail with GIL disabled')
@threading_helper.reap_threads
@threading_helper.requires_working_threading()
def test_decompress_locking(self):
Expand Down
Loading