linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] entry: Remove has_signal comment of arch_do_signal_or_restart() prototype
@ 2022-09-02  3:41 Tiezhu Yang
  2022-09-02 23:10 ` Kees Cook
  0 siblings, 1 reply; 4+ messages in thread
From: Tiezhu Yang @ 2022-09-02  3:41 UTC (permalink / raw)
  To: Thomas Gleixner, Peter Zijlstra, Andy Lutomirski
  Cc: linux-kernel, Kees Cook, Eric W. Biederman

The argument has_signal of arch_do_signal_or_restart() has been removed
in commit 8ba62d37949e ("task_work: Call tracehook_notify_signal from
get_signal on all architectures"), let us remove the related comment.

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
 include/linux/entry-common.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/include/linux/entry-common.h b/include/linux/entry-common.h
index 84a466b..d95ab85 100644
--- a/include/linux/entry-common.h
+++ b/include/linux/entry-common.h
@@ -253,7 +253,6 @@ static __always_inline void arch_exit_to_user_mode(void) { }
 /**
  * arch_do_signal_or_restart -  Architecture specific signal delivery function
  * @regs:	Pointer to currents pt_regs
- * @has_signal:	actual signal to handle
  *
  * Invoked from exit_to_user_mode_loop().
  */
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] entry: Remove has_signal comment of arch_do_signal_or_restart() prototype
  2022-09-02  3:41 [PATCH] entry: Remove has_signal comment of arch_do_signal_or_restart() prototype Tiezhu Yang
@ 2022-09-02 23:10 ` Kees Cook
  2022-09-29  9:58   ` Tiezhu Yang
  0 siblings, 1 reply; 4+ messages in thread
From: Kees Cook @ 2022-09-02 23:10 UTC (permalink / raw)
  To: Tiezhu Yang
  Cc: Thomas Gleixner, Peter Zijlstra, Andy Lutomirski, linux-kernel,
	Eric W. Biederman

On Fri, Sep 02, 2022 at 11:41:46AM +0800, Tiezhu Yang wrote:
> The argument has_signal of arch_do_signal_or_restart() has been removed
> in commit 8ba62d37949e ("task_work: Call tracehook_notify_signal from
> get_signal on all architectures"), let us remove the related comment.
> 
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>

Fixes: 8ba62d37949e ("task_work: Call tracehook_notify_signal from get_signal on all architectures")
Reviewed-by: Kees Cook <keescook@chromium.org>

-- 
Kees Cook

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] entry: Remove has_signal comment of arch_do_signal_or_restart() prototype
  2022-09-02 23:10 ` Kees Cook
@ 2022-09-29  9:58   ` Tiezhu Yang
  2022-10-08  3:30     ` Tiezhu Yang
  0 siblings, 1 reply; 4+ messages in thread
From: Tiezhu Yang @ 2022-09-29  9:58 UTC (permalink / raw)
  To: Kees Cook
  Cc: Thomas Gleixner, Peter Zijlstra, Andy Lutomirski, linux-kernel,
	Eric W. Biederman



On 09/03/2022 07:10 AM, Kees Cook wrote:
> On Fri, Sep 02, 2022 at 11:41:46AM +0800, Tiezhu Yang wrote:
>> The argument has_signal of arch_do_signal_or_restart() has been removed
>> in commit 8ba62d37949e ("task_work: Call tracehook_notify_signal from
>> get_signal on all architectures"), let us remove the related comment.
>>
>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>
> Fixes: 8ba62d37949e ("task_work: Call tracehook_notify_signal from get_signal on all architectures")
> Reviewed-by: Kees Cook <keescook@chromium.org>
>

Hi all,

Do you know which tree this patch will go through?

Thanks,
Tiezhu


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] entry: Remove has_signal comment of arch_do_signal_or_restart() prototype
  2022-09-29  9:58   ` Tiezhu Yang
@ 2022-10-08  3:30     ` Tiezhu Yang
  0 siblings, 0 replies; 4+ messages in thread
From: Tiezhu Yang @ 2022-10-08  3:30 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Kees Cook, Thomas Gleixner, Peter Zijlstra, Andy Lutomirski,
	linux-kernel, Eric W. Biederman



On 09/29/2022 05:58 PM, Tiezhu Yang wrote:
>
>
> On 09/03/2022 07:10 AM, Kees Cook wrote:
>> On Fri, Sep 02, 2022 at 11:41:46AM +0800, Tiezhu Yang wrote:
>>> The argument has_signal of arch_do_signal_or_restart() has been removed
>>> in commit 8ba62d37949e ("task_work: Call tracehook_notify_signal from
>>> get_signal on all architectures"), let us remove the related comment.
>>>
>>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>>
>> Fixes: 8ba62d37949e ("task_work: Call tracehook_notify_signal from
>> get_signal on all architectures")
>> Reviewed-by: Kees Cook <keescook@chromium.org>
>>
>
> Hi all,
>
> Do you know which tree this patch will go through?
>
> Thanks,
> Tiezhu

Hi Andrew,

Could you please pick it up via your tree in this merge window?

https://lore.kernel.org/lkml/1662090106-5545-1-git-send-email-yangtiezhu@loongson.cn/

Thanks,
Tiezhu


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-10-08  3:31 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-02  3:41 [PATCH] entry: Remove has_signal comment of arch_do_signal_or_restart() prototype Tiezhu Yang
2022-09-02 23:10 ` Kees Cook
2022-09-29  9:58   ` Tiezhu Yang
2022-10-08  3:30     ` Tiezhu Yang

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).