powerpc/ptdump: Refactor update of st->last_pa
authorChristophe Leroy <christophe.leroy@csgroup.eu>
Mon, 29 Jun 2020 11:17:18 +0000 (11:17 +0000)
committerMichael Ellerman <mpe@ellerman.id.au>
Sun, 26 Jul 2020 14:01:30 +0000 (00:01 +1000)
st->last_pa is always updated in note_page() so it can
be done outside the if/elseif/else block.

Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/610d6b1a60ad0bedef865a90153c1110cfaa507e.1593429426.git.christophe.leroy@csgroup.eu
arch/powerpc/mm/ptdump/ptdump.c

index b2ed1ca..20a0398 100644 (file)
@@ -211,7 +211,6 @@ static void note_page(struct pg_state *st, unsigned long addr,
                st->current_flags = flag;
                st->start_address = addr;
                st->start_pa = pa;
-               st->last_pa = pa;
                st->page_size = page_size;
                pt_dump_seq_printf(st->seq, "---[ %s ]---\n", st->marker->name);
        /*
@@ -251,13 +250,11 @@ static void note_page(struct pg_state *st, unsigned long addr,
                }
                st->start_address = addr;
                st->start_pa = pa;
-               st->last_pa = pa;
                st->page_size = page_size;
                st->current_flags = flag;
                st->level = level;
-       } else {
-               st->last_pa = pa;
        }
+       st->last_pa = pa;
 }
 
 static void walk_pte(struct pg_state *st, pmd_t *pmd, unsigned long start)