summaryrefslogtreecommitdiff
path: root/lib/parser.c
diff options
context:
space:
mode:
authorLinus Torvalds <torvalds@linux-foundation.org>2019-12-11 22:46:19 +0300
committerLinus Torvalds <torvalds@linux-foundation.org>2019-12-11 22:46:19 +0300
commitd1c6a2aa02af09238ad09493eb3c8685ccc3fe12 (patch)
tree138bfecd61e39cbe7492eaba1db0092d1e8cc8e7 /lib/parser.c
parent6794862a16ef41f753abd75c03a152836e4c8028 (diff)
downloadlinux-d1c6a2aa02af09238ad09493eb3c8685ccc3fe12.tar.xz
pipe: simplify signal handling in pipe_read() and add comments
There's no need to separately check for signals while inside the locked region, since we're going to do "wait_event_interruptible()" right afterwards anyway, and the error handling is much simpler there. The check for whether we had already read anything was also redundant, since we no longer do the odd merging of reads when there are pending writers. But perhaps more importantly, this adds commentary about why we still need to wake up possible writers even though we didn't read any data, and why we can skip all the finishing touches now if we get a signal (or had a signal pending) while waiting for more data. [ This is a split-out cleanup from my "make pipe IO use exclusive wait queues" thing, which I can't apply because it triggers a nasty bug in the GNU make jobserver - Linus ] Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'lib/parser.c')
0 files changed, 0 insertions, 0 deletions