public inbox for glibc-bugs@sourceware.org
help / color / mirror / Atom feed
* [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

end of thread, other threads:[~2023-05-23 23:47 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [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
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
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
2023-05-23 23:47 ` cvs-commit at gcc dot gnu.org
2023-05-23 23:47 ` cvs-commit at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).