diff options
| author | Paul Eggert <eggert@cs.ucla.edu> | 2025-08-01 13:46:33 -0700 |
|---|---|---|
| committer | Paul Eggert <eggert@cs.ucla.edu> | 2025-08-03 19:48:06 -0700 |
| commit | 93805484c55ac5da2304eecb0e69356f40a89552 (patch) | |
| tree | 4ca47975eea791a9260a90aa772d291c3aca47d2 /scripts/git-hooks/commit-msg | |
| parent | tail: prefer < 0 to == -1 (diff) | |
| download | coreutils-93805484c55ac5da2304eecb0e69356f40a89552.tar.gz coreutils-93805484c55ac5da2304eecb0e69356f40a89552.zip | |
tail: fix unlikely races with >=2 --pids
Also, fix commentary to talk about “nonexistent” rather than
“dead” processes, since the code looks for the former not the
latter and the difference matters for zombies.
* src/tail.c (some_writers_exist): Rename from writers_are_dead,
negate the sense, don’t have a special and counterintuitive case
for !nbpids, remove PIDs found not to exist, and avoid some
though not all unlikely races when kernels reuse PIDs.
(tail_forever): Optimize via blocking I/O even if --pid was used,
so long as all the writers no longer exist.
(tail_forever, tail_forever_inotify): Simplify the writers_dead
logic; there is no need to have a local var to track this, since
we can use pids and nbpids now.
(parse_options): Also free and clear pids if !forever.
Diffstat (limited to 'scripts/git-hooks/commit-msg')
0 files changed, 0 insertions, 0 deletions
