From: H.J. Lu Date: Thu, 24 Feb 2022 15:34:01 +0000 (-0800) Subject: gdb: Consolidate 32bit-pkeys.xml and 64bit-pkeys.xml X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=0653f01479ecbcbf3c4dfa6083187a5b2c2258c2;p=binutils-gdb.git gdb: Consolidate 32bit-pkeys.xml and 64bit-pkeys.xml 1. Since 32bit-pkeys.xml and 64bit-pkeys.xml are identical, consolidate them into a single keys.xml. 2. Enable PKU for x32 to fix: $ gdbserver :123456 x32-program ... .../gdbserver/regcache.cc:255: A problem internal to GDBserver has been detected . Unknown register pkru requested on Tiger Lake. --- diff --git a/gdb/amd64-linux-tdep.c b/gdb/amd64-linux-tdep.c index b61428d0e84..88a24c176e8 100644 --- a/gdb/amd64-linux-tdep.c +++ b/gdb/amd64-linux-tdep.c @@ -1578,14 +1578,16 @@ amd64_linux_read_description (uint64_t xcr0_features_bit, bool is_x32) { static target_desc *amd64_linux_tdescs \ [2/*AVX*/][2/*MPX*/][2/*AVX512*/][2/*PKRU*/] = {}; - static target_desc *x32_linux_tdescs[2/*AVX*/][2/*AVX512*/] = {}; + static target_desc *x32_linux_tdescs \ + [2/*AVX*/][2/*AVX512*/][2/*PKRU*/] = {}; target_desc **tdesc; if (is_x32) { tdesc = &x32_linux_tdescs[(xcr0_features_bit & X86_XSTATE_AVX) ? 1 : 0 ] - [(xcr0_features_bit & X86_XSTATE_AVX512) ? 1 : 0]; + [(xcr0_features_bit & X86_XSTATE_AVX512) ? 1 : 0] + [(xcr0_features_bit & X86_XSTATE_PKRU) ? 1 : 0]; } else { diff --git a/gdb/arch/amd64.c b/gdb/arch/amd64.c index 20716524e62..559f678d356 100644 --- a/gdb/arch/amd64.c +++ b/gdb/arch/amd64.c @@ -25,9 +25,9 @@ #include "../features/i386/64bit-core.c" #include "../features/i386/64bit-linux.c" #include "../features/i386/64bit-mpx.c" -#include "../features/i386/64bit-pkeys.c" #include "../features/i386/64bit-segments.c" #include "../features/i386/64bit-sse.c" +#include "../features/i386/pkeys.c" #include "../features/i386/x32-core.c" @@ -72,8 +72,8 @@ amd64_create_target_description (uint64_t xcr0, bool is_x32, bool is_linux, if (xcr0 & X86_XSTATE_AVX512) regnum = create_feature_i386_64bit_avx512 (tdesc.get (), regnum); - if ((xcr0 & X86_XSTATE_PKRU) && !is_x32) - regnum = create_feature_i386_64bit_pkeys (tdesc.get (), regnum); + if (xcr0 & X86_XSTATE_PKRU) + regnum = create_feature_i386_pkeys (tdesc.get (), regnum); return tdesc.release (); } diff --git a/gdb/arch/i386.c b/gdb/arch/i386.c index 6241053783b..f5b33a800ad 100644 --- a/gdb/arch/i386.c +++ b/gdb/arch/i386.c @@ -27,8 +27,8 @@ #include "../features/i386/32bit-avx.c" #include "../features/i386/32bit-avx512.c" #include "../features/i386/32bit-mpx.c" -#include "../features/i386/32bit-pkeys.c" #include "../features/i386/32bit-segments.c" +#include "../features/i386/pkeys.c" /* Create i386 target descriptions according to XCR0. */ @@ -67,7 +67,7 @@ i386_create_target_description (uint64_t xcr0, bool is_linux, bool segments) regnum = create_feature_i386_32bit_avx512 (tdesc.get (), regnum); if (xcr0 & X86_XSTATE_PKRU) - regnum = create_feature_i386_32bit_pkeys (tdesc.get (), regnum); + regnum = create_feature_i386_pkeys (tdesc.get (), regnum); return tdesc.release (); } diff --git a/gdb/features/Makefile b/gdb/features/Makefile index 68e17d0085d..a2bb2a5922f 100644 --- a/gdb/features/Makefile +++ b/gdb/features/Makefile @@ -216,7 +216,6 @@ FEATURE_XMLFILES = aarch64-core.xml \ i386/32bit-avx.xml \ i386/32bit-mpx.xml \ i386/32bit-avx512.xml \ - i386/32bit-pkeys.xml \ i386/32bit-segments.xml \ i386/64bit-avx512.xml \ i386/64bit-core.xml \ @@ -224,8 +223,8 @@ FEATURE_XMLFILES = aarch64-core.xml \ i386/64bit-segments.xml \ i386/64bit-avx.xml \ i386/64bit-linux.xml \ - i386/64bit-pkeys.xml \ i386/64bit-sse.xml \ + i386/pkeys.xml \ i386/x32-core.xml \ loongarch/base32.xml \ loongarch/base64.xml \ diff --git a/gdb/features/i386/32bit-pkeys.c b/gdb/features/i386/32bit-pkeys.c deleted file mode 100644 index 5e9087e4303..00000000000 --- a/gdb/features/i386/32bit-pkeys.c +++ /dev/null @@ -1,14 +0,0 @@ -/* THIS FILE IS GENERATED. -*- buffer-read-only: t -*- vi:set ro: - Original: 32bit-pkeys.xml */ - -#include "gdbsupport/tdesc.h" - -static int -create_feature_i386_32bit_pkeys (struct target_desc *result, long regnum) -{ - struct tdesc_feature *feature; - - feature = tdesc_create_feature (result, "org.gnu.gdb.i386.pkeys"); - tdesc_create_reg (feature, "pkru", regnum++, 1, NULL, 32, "uint32"); - return regnum; -} diff --git a/gdb/features/i386/32bit-pkeys.xml b/gdb/features/i386/32bit-pkeys.xml deleted file mode 100644 index db6c10af65f..00000000000 --- a/gdb/features/i386/32bit-pkeys.xml +++ /dev/null @@ -1,13 +0,0 @@ - - - - - - - - - diff --git a/gdb/features/i386/64bit-pkeys.c b/gdb/features/i386/64bit-pkeys.c deleted file mode 100644 index 83001d4a876..00000000000 --- a/gdb/features/i386/64bit-pkeys.c +++ /dev/null @@ -1,14 +0,0 @@ -/* THIS FILE IS GENERATED. -*- buffer-read-only: t -*- vi:set ro: - Original: 64bit-pkeys.xml */ - -#include "gdbsupport/tdesc.h" - -static int -create_feature_i386_64bit_pkeys (struct target_desc *result, long regnum) -{ - struct tdesc_feature *feature; - - feature = tdesc_create_feature (result, "org.gnu.gdb.i386.pkeys"); - tdesc_create_reg (feature, "pkru", regnum++, 1, NULL, 32, "uint32"); - return regnum; -} diff --git a/gdb/features/i386/64bit-pkeys.xml b/gdb/features/i386/64bit-pkeys.xml deleted file mode 100644 index db6c10af65f..00000000000 --- a/gdb/features/i386/64bit-pkeys.xml +++ /dev/null @@ -1,13 +0,0 @@ - - - - - - - - - diff --git a/gdb/features/i386/pkeys.c b/gdb/features/i386/pkeys.c new file mode 100644 index 00000000000..955d563176a --- /dev/null +++ b/gdb/features/i386/pkeys.c @@ -0,0 +1,14 @@ +/* THIS FILE IS GENERATED. -*- buffer-read-only: t -*- vi:set ro: + Original: pkeys.xml */ + +#include "gdbsupport/tdesc.h" + +static int +create_feature_i386_pkeys (struct target_desc *result, long regnum) +{ + struct tdesc_feature *feature; + + feature = tdesc_create_feature (result, "org.gnu.gdb.i386.pkeys"); + tdesc_create_reg (feature, "pkru", regnum++, 1, NULL, 32, "uint32"); + return regnum; +} diff --git a/gdb/features/i386/pkeys.xml b/gdb/features/i386/pkeys.xml new file mode 100644 index 00000000000..db6c10af65f --- /dev/null +++ b/gdb/features/i386/pkeys.xml @@ -0,0 +1,13 @@ + + + + + + + + +