Re: [PATCH v4] mm,hwpoison: Differentiate Action Required error in current and non current processes

[Aili Yao <yaoaili@kingsoft.com>]

On Wed, 20 Jan 2021 07:59:23 +0000
HORIGUCHI NAOYA(堀口　直也) <naoya.horiguchi@nec.com> 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