2021-02-03 05:33:28

by Tiezhu Yang

Subject: [PATCH] parisc: Replace test_ti_thread_flag() with test_tsk_thread_flag()

Use test_tsk_thread_flag() directly instead of test_ti_thread_flag() to
improve readability when the argument type is struct task_struct, it is
similar with commit 5afc78551bf5 ("arm64: Use test_tsk_thread_flag() for
checking TIF_SINGLESTEP").

Signed-off-by: Tiezhu Yang <[email protected]>
---
arch/parisc/include/asm/compat.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/parisc/include/asm/compat.h b/arch/parisc/include/asm/compat.h
index 8f33085..1a609d3 100644
--- a/arch/parisc/include/asm/compat.h
+++ b/arch/parisc/include/asm/compat.h
@@ -179,7 +179,7 @@ static __inline__ void __user *arch_compat_alloc_user_space(long len)

static inline int __is_compat_task(struct task_struct *t)
{
- return test_ti_thread_flag(task_thread_info(t), TIF_32BIT);
+ return test_tsk_thread_flag(t, TIF_32BIT);
}

static inline int is_compat_task(void)
--
2.1.0

2021-02-03 14:11:36

by Helge Deller

Subject: Re: [PATCH] parisc: Replace test_ti_thread_flag() with test_tsk_thread_flag()

On 2/3/21 6:27 AM, Tiezhu Yang wrote:
> Use test_tsk_thread_flag() directly instead of test_ti_thread_flag() to
> improve readability when the argument type is struct task_struct, it is
> similar with commit 5afc78551bf5 ("arm64: Use test_tsk_thread_flag() for
> checking TIF_SINGLESTEP").
>
> Signed-off-by: Tiezhu Yang <[email protected]>

Thanks!
Applied to the parisc git tree.

Helge

> ---
> arch/parisc/include/asm/compat.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/parisc/include/asm/compat.h b/arch/parisc/include/asm/compat.h
> index 8f33085..1a609d3 100644
> --- a/arch/parisc/include/asm/compat.h
> +++ b/arch/parisc/include/asm/compat.h
> @@ -179,7 +179,7 @@ static __inline__ void __user *arch_compat_alloc_user_space(long len)
>
> static inline int __is_compat_task(struct task_struct *t)
> {
> - return test_ti_thread_flag(task_thread_info(t), TIF_32BIT);
> + return test_tsk_thread_flag(t, TIF_32BIT);
> }
>
> static inline int is_compat_task(void)
>