Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-117657: fix race:sock_recv_impl suppressions for free-thread building #123697

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
merged 6 commits into from
Sep 6, 2024
Merged

gh-117657: fix race:sock_recv_impl suppressions for free-thread building #123697

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
58ff9b3
gh-123695: fix race:sock_recv_impl suppressions for free-thread building
Zheaoli Sep 4, 2024
81361a0
fix test failed
Zheaoli Sep 4, 2024
756c584
fix review idea
Zheaoli Sep 6, 2024
c5b114b
fix review idea
Zheaoli Sep 6, 2024
4bfd9d9
fix review idea here
Zheaoli Sep 6, 2024
33ac4a3
fix timeout setting
Zheaoli Sep 6, 2024
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
Next Next commit
gh-123695: fix race:sock_recv_impl suppressions for free-thread building 
Signed-off-by: Manjusaka <[email protected]>
  • Loading branch information
@Zheaoli
Zheaoli committed Sep 4, 2024
commit 58ff9b3ad8aa74c37b82a7a0d8a1076bb9dc22e0
9 changes: 5 additions & 4 deletions Lib/test/test_socket.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4806,15 +4806,16 @@ def testInterruptedSendmsgTimeout(self):


class TCPCloserTest(ThreadedTCPSocketTest):
def __init__(self, methodName='runTest'):
super().__init__(methodName)

def testClose(self):
conn, addr = self.serv.accept()
conn.close()

sd = self.cli
read, write, err = select.select([sd], [], [], 1.0)
self.assertEqual(read, [sd])
self.assertEqual(sd.recv(1), b'')
read, write, err = select.select([conn], [], [], 1.0)
self.assertEqual(read, [conn])
self.assertEqual(conn.recv(1), b'')

# Calling close() many times should be safe.
conn.close()
Expand Down
2 changes: 0 additions & 2 deletions Tools/tsan/suppressions_free_threading.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ race:set_allocator_unlocked
# These entries are for warnings that trigger in a library function, as called
# by a CPython function.

# https://gist.github.com/swtaarrs/8e0e365e1d9cecece3269a2fb2f2b8b8
race:sock_recv_impl
# https://gist.github.com/swtaarrs/08dfe7883b4c975c31ecb39388987a67
race:free_threadstate

Expand Down
Loading