* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
@ 2023-04-24 20:57 ` hjl.tools at gmail dot com
2023-04-28 21:55 ` cvs-commit at gcc dot gnu.org
` (10 subsequent siblings)
11 siblings, 0 replies; 12+ messages in thread
From: hjl.tools at gmail dot com @ 2023-04-24 20:57 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
H.J. Lu <hjl.tools at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |hjl.tools at gmail dot com
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
2023-04-24 20:57 ` [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock hjl.tools at gmail dot com
@ 2023-04-28 21:55 ` cvs-commit at gcc dot gnu.org
2023-05-23 18:43 ` cvs-commit at gcc dot gnu.org
` (9 subsequent siblings)
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-04-28 21:55 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #3 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by H.J. Lu <hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=a443bd3fb233186038b8b483959ecb7978d1abea
commit a443bd3fb233186038b8b483959ecb7978d1abea
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Thu Apr 27 13:06:15 2023 -0700
__check_pf: Add a cancellation cleanup handler [BZ #20975]
There are reports for hang in __check_pf:
https://github.com/JoeDog/siege/issues/4
It is reproducible only under specific configurations:
1. Large number of cores (>= 64) and large number of threads (> 3X of
the number of cores) with long lived socket connection.
2. Low power (frequency) mode.
3. Power management is enabled.
While holding lock, __check_pf calls make_request which calls __sendto
and __recvmsg. Since __sendto and __recvmsg are cancellation points,
lock held by __check_pf won't be released and can cause deadlock when
thread cancellation happens in __sendto or __recvmsg. Add a cancellation
cleanup handler for __check_pf to unlock the lock when cancelled by
another thread. This fixes BZ #20975 and the siege hang issue.
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
2023-04-24 20:57 ` [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock hjl.tools at gmail dot com
2023-04-28 21:55 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 18:43 ` cvs-commit at gcc dot gnu.org
2023-05-23 20:10 ` cvs-commit at gcc dot gnu.org
` (8 subsequent siblings)
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 18:43 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #4 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.37/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=f5d377c896b95fefc712b0fd5e5804ae3f48d392
commit f5d377c896b95fefc712b0fd5e5804ae3f48d392
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Thu Apr 27 13:06:15 2023 -0700
__check_pf: Add a cancellation cleanup handler [BZ #20975]
There are reports for hang in __check_pf:
https://github.com/JoeDog/siege/issues/4
It is reproducible only under specific configurations:
1. Large number of cores (>= 64) and large number of threads (> 3X of
the number of cores) with long lived socket connection.
2. Low power (frequency) mode.
3. Power management is enabled.
While holding lock, __check_pf calls make_request which calls __sendto
and __recvmsg. Since __sendto and __recvmsg are cancellation points,
lock held by __check_pf won't be released and can cause deadlock when
thread cancellation happens in __sendto or __recvmsg. Add a cancellation
cleanup handler for __check_pf to unlock the lock when cancelled by
another thread. This fixes BZ #20975 and the siege hang issue.
(cherry picked from commit a443bd3fb233186038b8b483959ecb7978d1abea)
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
` (2 preceding siblings ...)
2023-05-23 18:43 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 20:10 ` cvs-commit at gcc dot gnu.org
2023-05-23 21:52 ` cvs-commit at gcc dot gnu.org
` (7 subsequent siblings)
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 20:10 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #5 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.36/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=7df9a276563e201fd5680c46f0d8c6f719ce1fc9
commit 7df9a276563e201fd5680c46f0d8c6f719ce1fc9
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Thu Apr 27 13:06:15 2023 -0700
__check_pf: Add a cancellation cleanup handler [BZ #20975]
There are reports for hang in __check_pf:
https://github.com/JoeDog/siege/issues/4
It is reproducible only under specific configurations:
1. Large number of cores (>= 64) and large number of threads (> 3X of
the number of cores) with long lived socket connection.
2. Low power (frequency) mode.
3. Power management is enabled.
While holding lock, __check_pf calls make_request which calls __sendto
and __recvmsg. Since __sendto and __recvmsg are cancellation points,
lock held by __check_pf won't be released and can cause deadlock when
thread cancellation happens in __sendto or __recvmsg. Add a cancellation
cleanup handler for __check_pf to unlock the lock when cancelled by
another thread. This fixes BZ #20975 and the siege hang issue.
(cherry picked from commit a443bd3fb233186038b8b483959ecb7978d1abea)
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
` (3 preceding siblings ...)
2023-05-23 20:10 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 21:52 ` cvs-commit at gcc dot gnu.org
2023-05-23 23:06 ` cvs-commit at gcc dot gnu.org
` (6 subsequent siblings)
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 21:52 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #6 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.35/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=2b9906f9a0f27c1ffa329f23ae1664bc9925df0f
commit 2b9906f9a0f27c1ffa329f23ae1664bc9925df0f
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Thu Apr 27 13:06:15 2023 -0700
__check_pf: Add a cancellation cleanup handler [BZ #20975]
There are reports for hang in __check_pf:
https://github.com/JoeDog/siege/issues/4
It is reproducible only under specific configurations:
1. Large number of cores (>= 64) and large number of threads (> 3X of
the number of cores) with long lived socket connection.
2. Low power (frequency) mode.
3. Power management is enabled.
While holding lock, __check_pf calls make_request which calls __sendto
and __recvmsg. Since __sendto and __recvmsg are cancellation points,
lock held by __check_pf won't be released and can cause deadlock when
thread cancellation happens in __sendto or __recvmsg. Add a cancellation
cleanup handler for __check_pf to unlock the lock when cancelled by
another thread. This fixes BZ #20975 and the siege hang issue.
(cherry picked from commit a443bd3fb233186038b8b483959ecb7978d1abea)
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
` (4 preceding siblings ...)
2023-05-23 21:52 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 23:06 ` cvs-commit at gcc dot gnu.org
2023-05-23 23:41 ` cvs-commit at gcc dot gnu.org
` (5 subsequent siblings)
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 23:06 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #7 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.34/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=1cd6626a897304a28dc4e2ca1e303bb5774db6d1
commit 1cd6626a897304a28dc4e2ca1e303bb5774db6d1
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Thu Apr 27 13:06:15 2023 -0700
__check_pf: Add a cancellation cleanup handler [BZ #20975]
There are reports for hang in __check_pf:
https://github.com/JoeDog/siege/issues/4
It is reproducible only under specific configurations:
1. Large number of cores (>= 64) and large number of threads (> 3X of
the number of cores) with long lived socket connection.
2. Low power (frequency) mode.
3. Power management is enabled.
While holding lock, __check_pf calls make_request which calls __sendto
and __recvmsg. Since __sendto and __recvmsg are cancellation points,
lock held by __check_pf won't be released and can cause deadlock when
thread cancellation happens in __sendto or __recvmsg. Add a cancellation
cleanup handler for __check_pf to unlock the lock when cancelled by
another thread. This fixes BZ #20975 and the siege hang issue.
(cherry picked from commit a443bd3fb233186038b8b483959ecb7978d1abea)
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
` (5 preceding siblings ...)
2023-05-23 23:06 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 23:41 ` cvs-commit at gcc dot gnu.org
2023-05-23 23:44 ` cvs-commit at gcc dot gnu.org
` (4 subsequent siblings)
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 23:41 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #8 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.33/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=24302748fcf85023fd64630de241973ec17f2dc1
commit 24302748fcf85023fd64630de241973ec17f2dc1
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Thu Apr 27 13:06:15 2023 -0700
__check_pf: Add a cancellation cleanup handler [BZ #20975]
There are reports for hang in __check_pf:
https://github.com/JoeDog/siege/issues/4
It is reproducible only under specific configurations:
1. Large number of cores (>= 64) and large number of threads (> 3X of
the number of cores) with long lived socket connection.
2. Low power (frequency) mode.
3. Power management is enabled.
While holding lock, __check_pf calls make_request which calls __sendto
and __recvmsg. Since __sendto and __recvmsg are cancellation points,
lock held by __check_pf won't be released and can cause deadlock when
thread cancellation happens in __sendto or __recvmsg. Add a cancellation
cleanup handler for __check_pf to unlock the lock when cancelled by
another thread. This fixes BZ #20975 and the siege hang issue.
(cherry picked from commit a443bd3fb233186038b8b483959ecb7978d1abea)
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
` (6 preceding siblings ...)
2023-05-23 23:41 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 23:44 ` cvs-commit at gcc dot gnu.org
2023-05-23 23:45 ` cvs-commit at gcc dot gnu.org
` (3 subsequent siblings)
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 23:44 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #9 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.37/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=0e3e9dbb0ea3e0a4885e3dc075cdfe92fc29da66
commit 0e3e9dbb0ea3e0a4885e3dc075cdfe92fc29da66
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Tue May 23 16:44:01 2023 -0700
Document BZ #20975 fix
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
` (7 preceding siblings ...)
2023-05-23 23:44 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 23:45 ` cvs-commit at gcc dot gnu.org
2023-05-23 23:46 ` cvs-commit at gcc dot gnu.org
` (2 subsequent siblings)
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 23:45 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #10 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.36/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=93bd77104cdd7380823d80c778776d2eafb69a91
commit 93bd77104cdd7380823d80c778776d2eafb69a91
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Tue May 23 16:45:03 2023 -0700
Document BZ #20975 fix
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
` (8 preceding siblings ...)
2023-05-23 23:45 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 23:46 ` cvs-commit at gcc dot gnu.org
2023-05-23 23:47 ` cvs-commit at gcc dot gnu.org
2023-05-23 23:47 ` cvs-commit at gcc dot gnu.org
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 23:46 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #11 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.35/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=739de21d3043f927e93490ee33f9e1b948556f5b
commit 739de21d3043f927e93490ee33f9e1b948556f5b
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Tue May 23 16:46:00 2023 -0700
Document BZ #20975 fix
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
` (9 preceding siblings ...)
2023-05-23 23:46 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 23:47 ` cvs-commit at gcc dot gnu.org
2023-05-23 23:47 ` cvs-commit at gcc dot gnu.org
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 23:47 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #12 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.34/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=cad3adf4ddeada37912c1c13b59a2ea5dd5d2832
commit cad3adf4ddeada37912c1c13b59a2ea5dd5d2832
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Tue May 23 16:46:54 2023 -0700
Document BZ #20975 fix
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread* [Bug libc/20975] Deferred cancellation triggers in __check_pf and looses lock leading to deadlock.
[not found] <bug-20975-131@http.sourceware.org/bugzilla/>
` (10 preceding siblings ...)
2023-05-23 23:47 ` cvs-commit at gcc dot gnu.org
@ 2023-05-23 23:47 ` cvs-commit at gcc dot gnu.org
11 siblings, 0 replies; 12+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2023-05-23 23:47 UTC (permalink / raw)
To: glibc-bugs
https://sourceware.org/bugzilla/show_bug.cgi?id=20975
--- Comment #13 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The release/2.33/master branch has been updated by H.J. Lu
<hjl@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=eed27b3e46d0c92eb8bff6b2b5d7059a70996a8b
commit eed27b3e46d0c92eb8bff6b2b5d7059a70996a8b
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Tue May 23 16:47:45 2023 -0700
Document BZ #20975 fix
--
You are receiving this mail because:
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 12+ messages in thread