Excerpts from Christophe Leroy's message of January 16, 2021 3:09 am:
> 
> 
> Le 15/01/2021 à 17:49, Nicholas Piggin a écrit :
>> Similar to the previous patch this makes interrupt handler function
>> types more regular so they can be wrapped with the next patch.
>> 
>> Signed-off-by: Nicholas Piggin <npig...@gmail.com>
>> ---
>>   arch/powerpc/include/asm/bug.h             |  5 +++--
>>   arch/powerpc/kernel/entry_32.S             |  3 +--
>>   arch/powerpc/kernel/exceptions-64e.S       |  3 +--
>>   arch/powerpc/kernel/exceptions-64s.S       |  4 +---
>>   arch/powerpc/kernel/traps.c                |  2 +-
>>   arch/powerpc/mm/book3s64/hash_utils.c      |  4 ++--
>>   arch/powerpc/mm/book3s64/slb.c             |  2 +-
>>   arch/powerpc/mm/fault.c                    | 13 ++++++++++---
>>   arch/powerpc/platforms/8xx/machine_check.c |  2 +-
>>   9 files changed, 21 insertions(+), 17 deletions(-)
>> 
>> diff --git a/arch/powerpc/include/asm/bug.h b/arch/powerpc/include/asm/bug.h
>> index f7827e993196..8f09ddae9305 100644
>> --- a/arch/powerpc/include/asm/bug.h
>> +++ b/arch/powerpc/include/asm/bug.h
>> @@ -112,8 +112,9 @@
>>   
>>   struct pt_regs;
>>   long do_page_fault(struct pt_regs *);
>> -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 *, int);
>> +void __bad_page_fault(struct pt_regs *regs, int sig);
>> +void do_bad_page_fault_segv(struct pt_regs *regs);
> 
> What is that do_bad_page_fault_segv() ? Shouldn't it be in a separate patch ?

Hmm, yeah probably. It will be an interrupt handler (that doesn't 
require SIGSEGV argument).

Thanks,
Nick

Reply via email to