fix vector types in aarch64 register file structures
authorRich Felker <dalias@aerifal.cx>
Tue, 3 Nov 2020 04:25:12 +0000 (23:25 -0500)
committerRich Felker <dalias@aerifal.cx>
Wed, 11 Nov 2020 15:54:58 +0000 (10:54 -0500)
the ABI type for the vector registers in fpregset_t, struct
fpsimd_context, and struct user_fpsimd_struct is __uint128_t, which
was presumably originally not used because it's a nonstandard type,
but its existence is mandated by the aarch64 psABI. use of the wrong
type here broke software using these structures, and encouraged
incorrect fixes with casts rather than reinterpretation of
representation.

arch/aarch64/bits/signal.h
arch/aarch64/bits/user.h

index b71261f..5098c73 100644 (file)
@@ -11,7 +11,7 @@ typedef unsigned long greg_t;
 typedef unsigned long gregset_t[34];
 
 typedef struct {
-       long double vregs[32];
+       __uint128_t vregs[32];
        unsigned int fpsr;
        unsigned int fpcr;
 } fpregset_t;
@@ -34,7 +34,7 @@ struct fpsimd_context {
        struct _aarch64_ctx head;
        unsigned int fpsr;
        unsigned int fpcr;
-       long double vregs[32];
+       __uint128_t vregs[32];
 };
 struct esr_context {
        struct _aarch64_ctx head;
index d12cdf7..8a1002a 100644 (file)
@@ -6,7 +6,7 @@ struct user_regs_struct {
 };
 
 struct user_fpsimd_struct {
-       long double vregs[32];
+       __uint128_t vregs[32];
        unsigned int fpsr;
        unsigned int fpcr;
 };