* [PATCH] nios32: add support for TIF_NOTIFY_SIGNAL
@ 2020-10-29 16:18 Jens Axboe
2020-10-30 0:27 ` Tan, Ley Foon
0 siblings, 1 reply; 3+ messages in thread
From: Jens Axboe @ 2020-10-29 16:18 UTC (permalink / raw)
To: Ley Foon Tan, linux-kernel
Wire up TIF_NOTIFY_SIGNAL handling for nios32.
Cc: Ley Foon Tan <ley.foon.tan@intel.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
---
5.11 has support queued up for TIF_NOTIFY_SIGNAL, see this posting
for details:
https://lore.kernel.org/io-uring/20201026203230.386348-1-axboe@kernel.dk/
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/nios2/include/asm/thread_info.h | 2 ++
arch/nios2/kernel/signal.c | 3 ++-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/arch/nios2/include/asm/thread_info.h b/arch/nios2/include/asm/thread_info.h
index 7349a4fa635b..272d2c72a727 100644
--- a/arch/nios2/include/asm/thread_info.h
+++ b/arch/nios2/include/asm/thread_info.h
@@ -86,6 +86,7 @@ static inline struct thread_info *current_thread_info(void)
#define TIF_MEMDIE 4 /* is terminating due to OOM killer */
#define TIF_SECCOMP 5 /* secure computing */
#define TIF_SYSCALL_AUDIT 6 /* syscall auditing active */
+#define TIF_NOTIFY_SIGNAL 7 /* signal notifications exist */
#define TIF_RESTORE_SIGMASK 9 /* restore signal mask in do_signal() */
#define TIF_POLLING_NRFLAG 16 /* true if poll_idle() is polling
@@ -97,6 +98,7 @@ static inline struct thread_info *current_thread_info(void)
#define _TIF_NEED_RESCHED (1 << TIF_NEED_RESCHED)
#define _TIF_SECCOMP (1 << TIF_SECCOMP)
#define _TIF_SYSCALL_AUDIT (1 << TIF_SYSCALL_AUDIT)
+#define _TIF_NOTIFY_SIGNAL (1 << TIF_NOTIFY_SIGNAL)
#define _TIF_RESTORE_SIGMASK (1 << TIF_RESTORE_SIGMASK)
#define _TIF_POLLING_NRFLAG (1 << TIF_POLLING_NRFLAG)
diff --git a/arch/nios2/kernel/signal.c b/arch/nios2/kernel/signal.c
index cf2dca2ac7c3..2009ae2d3c3b 100644
--- a/arch/nios2/kernel/signal.c
+++ b/arch/nios2/kernel/signal.c
@@ -306,7 +306,8 @@ asmlinkage int do_notify_resume(struct pt_regs *regs)
if (!user_mode(regs))
return 0;
- if (test_thread_flag(TIF_SIGPENDING)) {
+ if (test_thread_flag(TIF_SIGPENDING) ||
+ test_thread_flag(TIF_NOTIFY_SIGNAL)) {
int restart = do_signal(regs);
if (unlikely(restart)) {
--
2.29.0
--
Jens Axboe
^ permalink raw reply related [flat|nested] 3+ messages in thread
* RE: [PATCH] nios32: add support for TIF_NOTIFY_SIGNAL
2020-10-29 16:18 [PATCH] nios32: add support for TIF_NOTIFY_SIGNAL Jens Axboe
@ 2020-10-30 0:27 ` Tan, Ley Foon
2020-10-30 2:34 ` Jens Axboe
0 siblings, 1 reply; 3+ messages in thread
From: Tan, Ley Foon @ 2020-10-30 0:27 UTC (permalink / raw)
To: Jens Axboe, linux-kernel
> -----Original Message-----
> From: Jens Axboe <axboe@kernel.dk>
> Sent: Friday, October 30, 2020 12:18 AM
> To: Tan, Ley Foon <ley.foon.tan@intel.com>; linux-kernel@vger.kernel.org
> Subject: [PATCH] nios32: add support for TIF_NOTIFY_SIGNAL
>
> Wire up TIF_NOTIFY_SIGNAL handling for nios32.
>
> Cc: Ley Foon Tan <ley.foon.tan@intel.com>
> Signed-off-by: Jens Axboe <axboe@kernel.dk>
> ---
>
> 5.11 has support queued up for TIF_NOTIFY_SIGNAL, see this posting for
> details:
>
> https://lore.kernel.org/io-uring/20201026203230.386348-1-
> axboe@kernel.dk/
>
> 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/nios2/include/asm/thread_info.h | 2 ++
> arch/nios2/kernel/signal.c | 3 ++-
> 2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/arch/nios2/include/asm/thread_info.h
> b/arch/nios2/include/asm/thread_info.h
> index 7349a4fa635b..272d2c72a727 100644
> --- a/arch/nios2/include/asm/thread_info.h
> +++ b/arch/nios2/include/asm/thread_info.h
> @@ -86,6 +86,7 @@ static inline struct thread_info
> *current_thread_info(void)
> #define TIF_MEMDIE 4 /* is terminating due to OOM killer */
> #define TIF_SECCOMP 5 /* secure computing */
> #define TIF_SYSCALL_AUDIT 6 /* syscall auditing active */
> +#define TIF_NOTIFY_SIGNAL 7 /* signal notifications exist */
> #define TIF_RESTORE_SIGMASK 9 /* restore signal mask in
> do_signal() */
>
> #define TIF_POLLING_NRFLAG 16 /* true if poll_idle() is polling
> @@ -97,6 +98,7 @@ static inline struct thread_info
> *current_thread_info(void)
> #define _TIF_NEED_RESCHED (1 << TIF_NEED_RESCHED)
> #define _TIF_SECCOMP (1 << TIF_SECCOMP)
> #define _TIF_SYSCALL_AUDIT (1 << TIF_SYSCALL_AUDIT)
> +#define _TIF_NOTIFY_SIGNAL (1 << TIF_NOTIFY_SIGNAL)
> #define _TIF_RESTORE_SIGMASK (1 << TIF_RESTORE_SIGMASK)
> #define _TIF_POLLING_NRFLAG (1 << TIF_POLLING_NRFLAG)
>
> diff --git a/arch/nios2/kernel/signal.c b/arch/nios2/kernel/signal.c index
> cf2dca2ac7c3..2009ae2d3c3b 100644
> --- a/arch/nios2/kernel/signal.c
> +++ b/arch/nios2/kernel/signal.c
> @@ -306,7 +306,8 @@ asmlinkage int do_notify_resume(struct pt_regs
> *regs)
> if (!user_mode(regs))
> return 0;
>
> - if (test_thread_flag(TIF_SIGPENDING)) {
> + if (test_thread_flag(TIF_SIGPENDING) ||
> + test_thread_flag(TIF_NOTIFY_SIGNAL)) {
> int restart = do_signal(regs);
>
> if (unlikely(restart)) {
> --
Please help integrate this patch thru your git tree.
Acked-by: Ley Foon Tan <ley.foon.tan@intel.com>
Regards
Ley Foon
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-10-30 2:34 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-29 16:18 [PATCH] nios32: add support for TIF_NOTIFY_SIGNAL Jens Axboe
2020-10-30 0:27 ` Tan, Ley Foon
2020-10-30 2:34 ` Jens Axboe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).