diff options
| author | Malte Skarupke <malteskarupke@fastmail.fm> | 2024-12-04 07:55:50 -0500 |
|---|---|---|
| committer | Carlos O'Donell <carlos@redhat.com> | 2025-01-17 14:56:58 -0500 |
| commit | b42cc6af11062c260c7dfa91f1c89891366fed3e (patch) | |
| tree | fd68a897155668cf407af4d61fc6327a138c84be /sysdeps/unix/sysv/linux | |
| parent | 0cc973160c23bb67f895bc887dd6942d29f8fee3 (diff) | |
| download | glibc-b42cc6af11062c260c7dfa91f1c89891366fed3e.zip glibc-b42cc6af11062c260c7dfa91f1c89891366fed3e.tar.gz glibc-b42cc6af11062c260c7dfa91f1c89891366fed3e.tar.bz2 | |
nptl: Remove unnecessary catch-all-wake in condvar group switch
This wake is unnecessary. We only switch groups after every sleeper in a group
has been woken. Sure, they may take a while to actually wake up and may still
hold a reference, but waking them a second time doesn't speed that up. Instead
this just makes the code more complicated and may hide problems.
In particular this safety wake wouldn't even have helped with the bug that was
fixed by Barrus' patch: The bug there was that pthread_cond_signal would not
switch g1 when it should, so we wouldn't even have entered this code path.
Signed-off-by: Malte Skarupke <malteskarupke@fastmail.fm>
Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Diffstat (limited to 'sysdeps/unix/sysv/linux')
0 files changed, 0 insertions, 0 deletions