Gentle nudge on this one. On 10/29/20 10:13 AM, Jens Axboe wrote: > Wire up TIF_NOTIFY_SIGNAL handling for h8300. > > Cc: uclin****@lists***** > Signed-off-by: Jens Axboe <axboe****@kerne*****> > --- > > 5.11 has support queued up for TIF_NOTIFY_SIGNAL, see this posting > for details: > > https://lore.kernel.org/io-uring/20201****@kerne*****/ > > As part of that work, I'm adding TIF_NOTIFY_SIGNAL support to all archs, > as that will enable a set of cleanups once all of them support it. I'm > happy carrying this patch if need be, or it can be funelled through the > arch tree. Let me know. > > arch/h8300/include/asm/thread_info.h | 4 +++- > arch/h8300/kernel/signal.c | 2 +- > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/h8300/include/asm/thread_info.h b/arch/h8300/include/asm/thread_info.h > index 0cdaa302d3d2..a518214d4ddd 100644 > --- a/arch/h8300/include/asm/thread_info.h > +++ b/arch/h8300/include/asm/thread_info.h > @@ -73,6 +73,7 @@ static inline struct thread_info *current_thread_info(void) > #define TIF_SYSCALL_AUDIT 7 /* syscall auditing active */ > #define TIF_SYSCALL_TRACEPOINT 8 /* for ftrace syscall instrumentation */ > #define TIF_POLLING_NRFLAG 9 /* true if poll_idle() is polling TIF_NEED_RESCHED */ > +#define TIF_NOTIFY_SIGNAL 10 /* signal notifications exist */ > > /* as above, but as bit values */ > #define _TIF_SYSCALL_TRACE (1 << TIF_SYSCALL_TRACE) > @@ -83,6 +84,7 @@ static inline struct thread_info *current_thread_info(void) > #define _TIF_SYSCALL_AUDIT (1 << TIF_SYSCALL_AUDIT) > #define _TIF_SYSCALL_TRACEPOINT (1 << TIF_SYSCALL_TRACEPOINT) > #define _TIF_POLLING_NRFLAG (1 << TIF_POLLING_NRFLAG) > +#define _TIF_NOTIFY_SIGNAL (1 << TIF_NOTIFY_SIGNAL) > > /* work to do in syscall trace */ > #define _TIF_WORK_SYSCALL_MASK (_TIF_SYSCALL_TRACE | _TIF_SINGLESTEP | \ > @@ -92,7 +94,7 @@ static inline struct thread_info *current_thread_info(void) > #define _TIF_ALLWORK_MASK (_TIF_SYSCALL_TRACE | _TIF_SIGPENDING | \ > _TIF_NEED_RESCHED | _TIF_SYSCALL_AUDIT | \ > _TIF_SINGLESTEP | _TIF_NOTIFY_RESUME | \ > - _TIF_SYSCALL_TRACEPOINT) > + _TIF_SYSCALL_TRACEPOINT | _TIF_NOTIFY_SIGNAL) > > /* work to do on interrupt/exception return */ > #define _TIF_WORK_MASK (_TIF_ALLWORK_MASK & ~(_TIF_SYSCALL_TRACE | \ > diff --git a/arch/h8300/kernel/signal.c b/arch/h8300/kernel/signal.c > index 75d9b7e626b2..75a1c36b105a 100644 > --- a/arch/h8300/kernel/signal.c > +++ b/arch/h8300/kernel/signal.c > @@ -279,7 +279,7 @@ static void do_signal(struct pt_regs *regs) > > asmlinkage void do_notify_resume(struct pt_regs *regs, u32 thread_info_flags) > { > - if (thread_info_flags & _TIF_SIGPENDING) > + if (thread_info_flags & (_TIF_SIGPENDING | _TIF_NOTIFY_SIGNAL)) > do_signal(regs); > > if (thread_info_flags & _TIF_NOTIFY_RESUME) > -- Jens Axboe