Excerpts from Christophe Leroy's message of December 22, 2020 11:28 pm: > The address argument is not used by bad_page_fault(). > > Remove it. > > Suggested-by: Nicholas Piggin <npig...@gmail.com> > Signed-off-by: Christophe Leroy <christophe.le...@csgroup.eu> > --- > arch/powerpc/include/asm/bug.h | 4 ++-- > arch/powerpc/kernel/entry_32.S | 4 +--- > arch/powerpc/kernel/exceptions-64e.S | 3 +-- > arch/powerpc/kernel/exceptions-64s.S | 8 +++----- > arch/powerpc/kernel/traps.c | 2 +- > arch/powerpc/mm/book3s64/hash_utils.c | 2 +- > arch/powerpc/mm/book3s64/slb.c | 2 +- > arch/powerpc/mm/fault.c | 6 +++--- > arch/powerpc/platforms/8xx/machine_check.c | 2 +- > 9 files changed, 14 insertions(+), 19 deletions(-) > > diff --git a/arch/powerpc/include/asm/bug.h b/arch/powerpc/include/asm/bug.h > index 464f8ca8a5c9..af8c164254d0 100644 > --- a/arch/powerpc/include/asm/bug.h > +++ b/arch/powerpc/include/asm/bug.h > @@ -112,8 +112,8 @@ > > struct pt_regs; > extern int do_page_fault(struct pt_regs *, unsigned long, unsigned long); > -extern void bad_page_fault(struct pt_regs *, unsigned long, int); > -void __bad_page_fault(struct pt_regs *regs, unsigned long address, int sig); > +void bad_page_fault(struct pt_regs *regs, int sig); > +void __bad_page_fault(struct pt_regs *regs, int sig); > extern void _exception(int, struct pt_regs *, int, unsigned long); > extern void _exception_pkey(struct pt_regs *, unsigned long, int); > extern void die(const char *, struct pt_regs *, long); > diff --git a/arch/powerpc/kernel/entry_32.S b/arch/powerpc/kernel/entry_32.S > index 238eacfda7b0..abd95aebe73a 100644 > --- a/arch/powerpc/kernel/entry_32.S > +++ b/arch/powerpc/kernel/entry_32.S > @@ -671,15 +671,13 @@ ppc_swapcontext: > handle_page_fault: > addi r3,r1,STACK_FRAME_OVERHEAD > bl do_page_fault > - cmpwi r3,0 > + mr. r4,r3
This looks like an unrelated change so I'll leave it out. Nice little improvement though. Thanks, Nick