From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1039) id 6B74F3857C42; Thu, 31 Mar 2022 17:25:34 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 6B74F3857C42 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: H.J. Lu To: gdb-cvs@sourceware.org Subject: [binutils-gdb] gdb: Consolidate 32bit-pkeys.xml and 64bit-pkeys.xml X-Act-Checkin: binutils-gdb X-Git-Author: H.J. Lu X-Git-Refname: refs/heads/master X-Git-Oldrev: 5a0c4a06eb4f4afcf885596ee97706c1045a8476 X-Git-Newrev: 0653f01479ecbcbf3c4dfa6083187a5b2c2258c2 Message-Id: <20220331172534.6B74F3857C42@sourceware.org> Date: Thu, 31 Mar 2022 17:25:34 +0000 (GMT) X-BeenThere: gdb-cvs@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 31 Mar 2022 17:25:34 -0000 https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3D0653f01479ec= bcbf3c4dfa6083187a5b2c2258c2 commit 0653f01479ecbcbf3c4dfa6083187a5b2c2258c2 Author: H.J. Lu Date: Thu Feb 24 07:34:01 2022 -0800 gdb: Consolidate 32bit-pkeys.xml and 64bit-pkeys.xml =20 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: =20 $ gdbserver :123456 x32-program ... .../gdbserver/regcache.cc:255: A problem internal to GDBserver has been= detected . Unknown register pkru requested =20 on Tiger Lake. Diff: --- gdb/amd64-linux-tdep.c | 6 ++++-- gdb/arch/amd64.c | 6 +++--- gdb/arch/i386.c | 4 ++-- gdb/features/Makefile | 3 +-- gdb/features/i386/64bit-pkeys.c | 14 -------------- gdb/features/i386/64bit-pkeys.xml | 13 ------------- gdb/features/i386/{32bit-pkeys.c =3D> pkeys.c} | 4 ++-- gdb/features/i386/{32bit-pkeys.xml =3D> pkeys.xml} | 0 8 files changed, 12 insertions(+), 38 deletions(-) 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_feature= s_bit, bool is_x32) { static target_desc *amd64_linux_tdescs \ [2/*AVX*/][2/*MPX*/][2/*AVX512*/][2/*PKRU*/] =3D {}; - static target_desc *x32_linux_tdescs[2/*AVX*/][2/*AVX512*/] =3D {}; + static target_desc *x32_linux_tdescs \ + [2/*AVX*/][2/*AVX512*/][2/*PKRU*/] =3D {}; =20 target_desc **tdesc; =20 if (is_x32) { tdesc =3D &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" =20 #include "../features/i386/x32-core.c" =20 @@ -72,8 +72,8 @@ amd64_create_target_description (uint64_t xcr0, bool is_x= 32, bool is_linux, if (xcr0 & X86_XSTATE_AVX512) regnum =3D create_feature_i386_64bit_avx512 (tdesc.get (), regnum); =20 - if ((xcr0 & X86_XSTATE_PKRU) && !is_x32) - regnum =3D create_feature_i386_64bit_pkeys (tdesc.get (), regnum); + if (xcr0 & X86_XSTATE_PKRU) + regnum =3D create_feature_i386_pkeys (tdesc.get (), regnum); =20 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" =20 /* Create i386 target descriptions according to XCR0. */ =20 @@ -67,7 +67,7 @@ i386_create_target_description (uint64_t xcr0, bool is_li= nux, bool segments) regnum =3D create_feature_i386_32bit_avx512 (tdesc.get (), regnum); =20 if (xcr0 & X86_XSTATE_PKRU) - regnum =3D create_feature_i386_32bit_pkeys (tdesc.get (), regnum); + regnum =3D create_feature_i386_pkeys (tdesc.get (), regnum); =20 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 =3D 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 =3D 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/64bit-pkeys.c b/gdb/features/i386/64bit-pkey= s.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 =3D 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-pk= eys.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/32bit-pkeys.c b/gdb/features/i386/pkeys.c similarity index 74% rename from gdb/features/i386/32bit-pkeys.c rename to gdb/features/i386/pkeys.c index 5e9087e4303..955d563176a 100644 --- a/gdb/features/i386/32bit-pkeys.c +++ b/gdb/features/i386/pkeys.c @@ -1,10 +1,10 @@ /* THIS FILE IS GENERATED. -*- buffer-read-only: t -*- vi:set ro: - Original: 32bit-pkeys.xml */ + Original: pkeys.xml */ =20 #include "gdbsupport/tdesc.h" =20 static int -create_feature_i386_32bit_pkeys (struct target_desc *result, long regnum) +create_feature_i386_pkeys (struct target_desc *result, long regnum) { struct tdesc_feature *feature; =20 diff --git a/gdb/features/i386/32bit-pkeys.xml b/gdb/features/i386/pkeys.xml similarity index 100% rename from gdb/features/i386/32bit-pkeys.xml rename to gdb/features/i386/pkeys.xml