Re: [PATCH 27/29] [AARCH64] Fix up ucontext for ILP32

[Mike Frysinger <vapier at gentoo dot org>]

On 27 Oct 2014 00:59, Andrew Pinski wrote:
> --- a/sysdeps/unix/sysv/linux/aarch64/sys/ucontext.h
> +++ b/sysdeps/unix/sysv/linux/aarch64/sys/ucontext.h
> @@ -46,8 +46,14 @@ typedef struct sigcontext mcontext_t;
>  /* Userlevel context.  */
>  typedef struct ucontext
>    {
> -    unsigned long uc_flags;
> -    struct ucontext *uc_link;
> +    __SYSCALL_ULONG_TYPE uc_flags;
> +#if defined(__ILP32__) && defined(__AARCH64EB__)
> +    int __pad_uc_link;
> +#endif
> +     struct ucontext *uc_link;
> +#if defined(__ILP32__) && !defined(__AARCH64EB__)
> +    int __pad_uc_link;
> +#endif

would it be better to leverage the common __BYTE_ORDER ?  would make it easier 
for other people to make changes and be confident on not screwing up if we used 
the same set of defines ...
-mike

Attachment: signature.asc
Description: Digital signature