Re: [RESEND PATCH V2] arm64: fault: avoid send SIGBUS two times

2017-12-12 Thread gengdongjiu
> Reviewed-by: James Morse > > > Nit: Your 'RESEND V2' and 'V2' are not the same patch. > 'RESEND' is to indicate you're reposting exactly the same patch, usually with > a > fixed CC list. Anyone who receives both can ignore one as you've said they are > the same. James, Thanks for the remin

Re: [RESEND PATCH V2] arm64: fault: avoid send SIGBUS two times

2017-12-12 Thread James Morse
Hi Dongjiu Geng, On 11/12/17 16:05, Dongjiu Geng wrote: > do_sea() calls arm64_notify_die() which will always signal > user-space. It also returns whether APEI claimed the external > abort as a RAS notification. If it returns failure do_mem_abort() > will signal user-space too. > > do_mem_abort()

Re: [RESEND PATCH V2] arm64: fault: avoid send SIGBUS two times

2017-12-11 Thread gengdongjiu
On 2017/12/12 11:31, Xie XiuQi wrote: >> +return 0; > It looks good to me. do_sea() has done all necessary action for SEA, so it > should always return 0, > no matter ghes_notify_sea() return true or false. yes, it is. > > Reviewed-by: Xie XiuQi Thanks XiuQi's review and comments. > >>

Re: [RESEND PATCH V2] arm64: fault: avoid send SIGBUS two times

2017-12-11 Thread Xie XiuQi
Hi Dongjiu, On 2017/12/12 0:05, Dongjiu Geng wrote: > do_sea() calls arm64_notify_die() which will always signal > user-space. It also returns whether APEI claimed the external > abort as a RAS notification. If it returns failure do_mem_abort() > will signal user-space too. > > do_mem_abort() wan

[RESEND PATCH V2] arm64: fault: avoid send SIGBUS two times

2017-12-11 Thread Dongjiu Geng
do_sea() calls arm64_notify_die() which will always signal user-space. It also returns whether APEI claimed the external abort as a RAS notification. If it returns failure do_mem_abort() will signal user-space too. do_mem_abort() wants to know if we handled the error, we always call arm64_notify_d