Commit e1ce8a97 authored by Colin Ian King's avatar Colin Ian King Committed by Linus Torvalds

nilfs2: remove redundant pointer sbufs

Pointer sbufs is being assigned a value but it's not being used later
on.  The pointer is redundant and can be removed.  Cleans up scan-build
static analysis warning:

  fs/nilfs2/page.c:203:8: warning: Although the value stored to 'sbufs'
    is used in the enclosing expression, the value is never actually read
    from 'sbufs' [deadcode.DeadStores]
        sbh = sbufs = page_buffers(src);

Link: https://lkml.kernel.org/r/20211211180955.550380-1-colin.i.king@gmail.com
Link: https://lkml.kernel.org/r/1640712476-15136-1-git-send-email-konishi.ryusuke@gmail.comSigned-off-by: default avatarColin Ian King <colin.i.king@gmail.com>
Signed-off-by: default avatarRyusuke Konishi <konishi.ryusuke@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 9630f0d6
...@@ -195,12 +195,12 @@ void nilfs_page_bug(struct page *page) ...@@ -195,12 +195,12 @@ void nilfs_page_bug(struct page *page)
*/ */
static void nilfs_copy_page(struct page *dst, struct page *src, int copy_dirty) static void nilfs_copy_page(struct page *dst, struct page *src, int copy_dirty)
{ {
struct buffer_head *dbh, *dbufs, *sbh, *sbufs; struct buffer_head *dbh, *dbufs, *sbh;
unsigned long mask = NILFS_BUFFER_INHERENT_BITS; unsigned long mask = NILFS_BUFFER_INHERENT_BITS;
BUG_ON(PageWriteback(dst)); BUG_ON(PageWriteback(dst));
sbh = sbufs = page_buffers(src); sbh = page_buffers(src);
if (!page_has_buffers(dst)) if (!page_has_buffers(dst))
create_empty_buffers(dst, sbh->b_size, 0); create_empty_buffers(dst, sbh->b_size, 0);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment