This one has no effect because this value is not used before it is assigned again. To: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx> To: linux-fsdevel@xxxxxxxxxxxxxxx To: linux-kernel@xxxxxxxxxxxxxxx Signed-off-by: Max Kellermann <max.kellermann@xxxxxxxxx> --- fs/pipe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/pipe.c b/fs/pipe.c index aca717a89631..b2075ecd4751 100644 --- a/fs/pipe.c +++ b/fs/pipe.c @@ -487,7 +487,7 @@ pipe_write(struct kiocb *iocb, struct iov_iter *from) head = pipe->head; if (!pipe_full(head, pipe->tail, pipe->max_usage)) { unsigned int mask = pipe->ring_size - 1; - struct pipe_buffer *buf = &pipe->bufs[head & mask]; + struct pipe_buffer *buf; struct page *page = pipe->tmp_page; int copied; -- 2.34.0