On Fri, Oct 16 2020 at 08:53, Jens Axboe wrote: > On 10/16/20 8:51 AM, Oleg Nesterov wrote: >> On 10/16, Thomas Gleixner wrote: >>> >>> With moving the handling into get_signal() you don't need more changes >>> to arch/* than adding the TIF bit, right? >> >> we still need to do something like >> >> - if (thread_flags & _TIF_SIGPENDING) >> + if (thread_flags & (_TIF_SIGPENDING | _TIF_NOTIFY_SIGNAL)) >> do_signal(...); >> >> and add _TIF_NOTIFY_SIGNAL to the WORK-PENDING mask in arch/* code. > > Yes, but it becomes really minimal at that point, and just that. There's > no touching any of the arch do_signal() code. > > Just finished the update of the branch to this model, and it does simplify > things quite a bit! Most arch patches are now exactly just what you write > above, no more. Except for all the nasty ones which have these checks in ASM :)
