On Wed, 20 Jan 2021 07:59:23 +0000 HORIGUCHI NAOYA(堀口 直也) <naoya.horiguchi@xxxxxxx> wrote: > It's desirable that $SUBJECT represents what that patch does, so > how about something like > "mm, hwpoison: send SIGBUS to PF_MCE_EARLY processes on action required events" ? This one looks better, i will change to that. Thanks > ... > > @@ -440,10 +443,13 @@ static struct task_struct *find_early_kill_thread(struct task_struct *tsk) > > * Determine whether a given process is "early kill" process which expects > > * to be signaled when some page under the process is hwpoisoned. > > * Return task_struct of the dedicated thread (main thread unless explicitly > > - * specified) if the process is "early kill," and otherwise returns NULL. > > + * specified) if the process is "early kill" and otherwise returns NULL. > > * > > - * Note that the above is true for Action Optional case, but not for Action > > - * Required case where SIGBUS should sent only to the current thread. > > + * Note that the above is true for Action Optional case. For Action Required > > + * case, it's only meaningful to the current thread which need to be signaled > > + * with SIGBUS, this error is Action Optional for other non current > > + * processes sharing the same error page,if the process is "early kill",the > Thanks for reminding, I will fix this in v5 patch Thank you! -- Best Regards! Aili Yao
