public inbox for linux-s390@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/1] s390: remove extra check of task_stack_page()
       [not found] <CGME20260428105633epcas5p37a93b7abb3d2ce0d1c2fbf6a72634ea7@epcas5p3.samsung.com>
@ 2026-04-28 10:56 ` Maninder Singh
  2026-04-28 12:54   ` Heiko Carstens
  2026-04-28 13:31   ` Alexander Gordeev
  0 siblings, 2 replies; 3+ messages in thread
From: Maninder Singh @ 2026-04-28 10:56 UTC (permalink / raw)
  To: hca, gor, agordeev, borntraeger, svens
  Cc: linux-s390, linux-kernel, Maninder Singh

There is no need to call task_stack_page(),
because try_get_task_stack() already takes care of that.

Signed-off-by: Maninder Singh <maninder1.s@samsung.com>
---
 arch/s390/kernel/process.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/arch/s390/kernel/process.c b/arch/s390/kernel/process.c
index 0df95dcb2101..dabd569dbba8 100644
--- a/arch/s390/kernel/process.c
+++ b/arch/s390/kernel/process.c
@@ -203,9 +203,6 @@ unsigned long __get_wchan(struct task_struct *p)
 	struct unwind_state state;
 	unsigned long ip = 0;
 
-	if (!task_stack_page(p))
-		return 0;
-
 	if (!try_get_task_stack(p))
 		return 0;
 
-- 
2.34.1


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

* Re: [PATCH 1/1] s390: remove extra check of task_stack_page()
  2026-04-28 10:56 ` [PATCH 1/1] s390: remove extra check of task_stack_page() Maninder Singh
@ 2026-04-28 12:54   ` Heiko Carstens
  2026-04-28 13:31   ` Alexander Gordeev
  1 sibling, 0 replies; 3+ messages in thread
From: Heiko Carstens @ 2026-04-28 12:54 UTC (permalink / raw)
  To: Maninder Singh
  Cc: gor, agordeev, borntraeger, svens, linux-s390, linux-kernel

On Tue, Apr 28, 2026 at 04:26:07PM +0530, Maninder Singh wrote:
> There is no need to call task_stack_page(),
> because try_get_task_stack() already takes care of that.
> 
> Signed-off-by: Maninder Singh <maninder1.s@samsung.com>
> ---
>  arch/s390/kernel/process.c | 3 ---
>  1 file changed, 3 deletions(-)

Reviewed-by: Heiko Carstens <hca@linux.ibm.com>

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

* Re: [PATCH 1/1] s390: remove extra check of task_stack_page()
  2026-04-28 10:56 ` [PATCH 1/1] s390: remove extra check of task_stack_page() Maninder Singh
  2026-04-28 12:54   ` Heiko Carstens
@ 2026-04-28 13:31   ` Alexander Gordeev
  1 sibling, 0 replies; 3+ messages in thread
From: Alexander Gordeev @ 2026-04-28 13:31 UTC (permalink / raw)
  To: Maninder Singh; +Cc: hca, gor, borntraeger, svens, linux-s390, linux-kernel

On Tue, Apr 28, 2026 at 04:26:07PM +0530, Maninder Singh wrote:
> There is no need to call task_stack_page(),
> because try_get_task_stack() already takes care of that.
> 
> Signed-off-by: Maninder Singh <maninder1.s@samsung.com>
> ---
>  arch/s390/kernel/process.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/arch/s390/kernel/process.c b/arch/s390/kernel/process.c
> index 0df95dcb2101..dabd569dbba8 100644
> --- a/arch/s390/kernel/process.c
> +++ b/arch/s390/kernel/process.c
> @@ -203,9 +203,6 @@ unsigned long __get_wchan(struct task_struct *p)
>  	struct unwind_state state;
>  	unsigned long ip = 0;
>  
> -	if (!task_stack_page(p))
> -		return 0;
> -
>  	if (!try_get_task_stack(p))
>  		return 0;

Applied, thanks!

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

end of thread, other threads:[~2026-04-28 13:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <CGME20260428105633epcas5p37a93b7abb3d2ce0d1c2fbf6a72634ea7@epcas5p3.samsung.com>
2026-04-28 10:56 ` [PATCH 1/1] s390: remove extra check of task_stack_page() Maninder Singh
2026-04-28 12:54   ` Heiko Carstens
2026-04-28 13:31   ` Alexander Gordeev

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox