Re: [PATCH v2 1/2] powerpc/ptdump: Refactor update of st->last_pa

[Michael Ellerman]

On Mon, 29 Jun 2020 11:17:18 + (UTC), Christophe Leroy wrote:
> st->last_pa is always updated in note_page() so it can
> be done outside the if/elseif/else block.

Applied to powerpc/next.

[1/2] powerpc/ptdump: Refactor update of st->last_pa
  https://git.kernel.org/powerpc/c/846feeace51bce13f5c645d5bf162455b89841fd
[2/2] powerpc/ptdump: Refactor update of pg_state
  https://git.kernel.org/powerpc/c/e54e30bca40233139290aecfce932dea9b996516

cheers

[PATCH v2 1/2] powerpc/ptdump: Refactor update of st->last_pa

[Christophe Leroy]

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 
---
 arch/powerpc/mm/ptdump/ptdump.c | 5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/arch/powerpc/mm/ptdump/ptdump.c b/arch/powerpc/mm/ptdump/ptdump.c
index b2ed1ca4f254..20a039867934 100644
--- a/arch/powerpc/mm/ptdump/ptdump.c
+++ b/arch/powerpc/mm/ptdump/ptdump.c
@@ -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)
-- 
2.25.0