fix riscv64 elf_fpregset_t type and member names mismatch
authorRich Felker <dalias@aerifal.cx>
Mon, 30 Sep 2019 03:45:47 +0000 (23:45 -0400)
committerRich Felker <dalias@aerifal.cx>
Mon, 30 Sep 2019 03:45:47 +0000 (23:45 -0400)
in order for sys/procfs.h (provided by sys/user.h) to be useful, it
needs to match the API its consumers (gdb, etc.) expect, including the
member names established by glibc.

this partly reverts commit 29e8737f81ccc9fbadcf61a75318aa3d0516aafa,
which partly reverted d493206de7df4db07ad34f24701539ba0a6ed38c,
eliminating struct user_fpregs_struct which seems to have had no
precedent and using union __riscv_mc_fp_state for elf_fpregset_t. this
requires indirect inclusion of signal.h to make union
__riscv_mc_fp_state visible, but being that these are nonstandard
"junk" headers with no official restrictions on what they can pull in,
that's no big deal.

split off and expanded from patch by Khem Raj.

arch/riscv64/bits/user.h

index d2e383b24e53b9e5ebf99b46986d81cb46ec494d..2da743eaf8fe9b437f3f6183968124c532083553 100644 (file)
@@ -1,8 +1,5 @@
-struct user_fpregs_struct {
-       double f[32];
-       unsigned int fcsr;
-};
+#include <signal.h>
 
 #define ELF_NGREG 32
 typedef unsigned long elf_greg_t, elf_gregset_t[ELF_NGREG];
-typedef struct user_fpregs_struct elf_fpregset_t;
+typedef union __riscv_mc_fp_state elf_fpregset_t;