lkml.org 
[lkml]   [2014]   [Jun]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 12/24] ARM64: Add is_ilp32_compat_task and is_ilp32_compat_thread and TIF_32BIT_AARCH64.
On Sat, May 24, 2014 at 12:02:07AM -0700, Andrew Pinski wrote:
> diff --git a/arch/arm64/include/asm/compat.h b/arch/arm64/include/asm/compat.h
> index fad7612..85f945c 100644
> --- a/arch/arm64/include/asm/compat.h
> +++ b/arch/arm64/include/asm/compat.h
> @@ -313,6 +313,26 @@ static inline int is_a32_compat_thread(struct thread_info *thread)
> }
> #endif
>
> +#ifdef CONFIG_ARM64_ILP32
> +static inline int is_ilp32_compat_task(void)
> +{
> + return test_thread_flag(TIF_32BIT_AARCH64);
> +}
> +static inline int is_ilp32_compat_thread(struct thread_info *thread)
> +{
> + return test_ti_thread_flag(thread, TIF_32BIT_AARCH64);
> +}
> +#else
> +static inline int is_ilp32_compat_task(void)
> +{
> + return 0;
> +}
> +static inline int is_ilp32_compat_thread(struct thread_info *thread)
> +{
> + return 0;
> +}
> +#endif
> +
> #else /* !CONFIG_COMPAT */
>
> static inline int is_a32_compat_task(void)
> @@ -323,17 +343,25 @@ static inline int is_a32_compat_thread(struct thread_info *thread)
> {
> return 0;
> }
> +static inline int is_ilp32_compat_task(void)
> +{
> + return 0;
> +}
> +static inline int is_ilp32_compat_thread(struct thread_info *thread)
> +{
> + return 0;
> +}

As for a previous patch, it looks like we have too many dummy
definitions for is_ilp32_compat_*().

>
> #endif /* CONFIG_COMPAT */
>
> static inline int is_compat_task(void)
> {
> - return is_a32_compat_task();
> + return is_a32_compat_task() || is_ilp32_compat_task();
> }
>
> static inline int is_compat_thread(struct thread_info *thread)
> {
> - return is_a32_compat_thread(thread);
> + return is_a32_compat_thread(thread) || is_ilp32_compat_thread(thread);
> }
> #endif /* __KERNEL__ */
> #endif /* __ASM_COMPAT_H */
> diff --git a/arch/arm64/include/asm/thread_info.h b/arch/arm64/include/asm/thread_info.h
> index b2bf728..98baf65 100644
> --- a/arch/arm64/include/asm/thread_info.h
> +++ b/arch/arm64/include/asm/thread_info.h
> @@ -108,6 +108,7 @@ static inline struct thread_info *current_thread_info(void)
> #define TIF_SINGLESTEP 21
> #define TIF_32BIT 22 /* AARCH32 process */
> #define TIF_SWITCH_MM 23 /* deferred switch_mm */
> +#define TIF_32BIT_AARCH64 24 /* 32 bit process on AArch64(ILP32) */

The is_compat_task() code could be simplified if we keep the TIF_32BIT
flag for both 32 and 64-bit applications (ILP32) and define TIF_AARCH32.

--
Catalin


\
 
 \ /
  Last update: 2014-06-19 15:41    [W:0.271 / U:0.804 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site