mm/shmem: remove unneeded assignments in shmem_get_folio_gfp()
authorLukas Bulwahn <lukas.bulwahn@gmail.com>
Fri, 7 Oct 2022 08:50:27 +0000 (10:50 +0200)
committerAndrew Morton <akpm@linux-foundation.org>
Wed, 9 Nov 2022 01:37:13 +0000 (17:37 -0800)
After the rework of shmem_get_folio_gfp() to use a folio, the local
variable hindex is only needed to be set once before passing it to
shmem_add_to_page_cache().

Remove the unneeded initialization and assignments of the variable hindex
before the actual effective assignment and first use.

No functional change. No change in object code.

Link: https://lkml.kernel.org/r/20221007085027.6309-1-lukas.bulwahn@gmail.com
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Cc: Hugh Dickins <hughd@google.com>
Cc: Matthew Wilcox <willy@infradead.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/shmem.c

index 9c897cf..57d878b 100644 (file)
@@ -1826,7 +1826,7 @@ static int shmem_get_folio_gfp(struct inode *inode, pgoff_t index,
        struct shmem_sb_info *sbinfo;
        struct mm_struct *charge_mm;
        struct folio *folio;
-       pgoff_t hindex = index;
+       pgoff_t hindex;
        gfp_t huge_gfp;
        int error;
        int once = 0;
@@ -1864,7 +1864,6 @@ repeat:
        }
 
        if (folio) {
-               hindex = folio->index;
                if (sgp == SGP_WRITE)
                        folio_mark_accessed(folio);
                if (folio_test_uptodate(folio))