From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15331 invoked by alias); 24 Jan 2018 09:27:46 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 62254 invoked by uid 89); 24 Jan 2018 09:27:28 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-25.7 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,KAM_LOTSOFHASH,KAM_SHORT,RCVD_IN_DNSWL_NONE,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 spammy=Hx-exchange-antispam-report-test:22074186197030, hyphens X-HELO: EUR01-HE1-obe.outbound.protection.outlook.com Received: from mail-he1eur01on0086.outbound.protection.outlook.com (HELO EUR01-HE1-obe.outbound.protection.outlook.com) (104.47.0.86) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 24 Jan 2018 09:27:25 +0000 Received: from DB3PR08MB0106.eurprd08.prod.outlook.com (10.161.56.20) by DB3PR08MB0108.eurprd08.prod.outlook.com (10.161.56.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.428.17; Wed, 24 Jan 2018 09:27:21 +0000 Received: from DB3PR08MB0106.eurprd08.prod.outlook.com ([fe80::958e:90c2:6108:af75]) by DB3PR08MB0106.eurprd08.prod.outlook.com ([fe80::958e:90c2:6108:af75%14]) with mapi id 15.20.0428.019; Wed, 24 Jan 2018 09:27:21 +0000 From: Alan Hayward To: "gdb-patches@sourceware.org" CC: nd Subject: [PATCH v2 2/8] Use tdesc_reg in gxdbserver tdesc Date: Wed, 24 Jan 2018 09:27:00 -0000 Message-ID: References: <7C97CC6A-92CB-4702-820D-206022F07102@arm.com> In-Reply-To: <7C97CC6A-92CB-4702-820D-206022F07102@arm.com> authentication-results: spf=none (sender IP is ) smtp.mailfrom=Alan.Hayward@arm.com; x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1;DB3PR08MB0108;7:Zmln6iuokaJFrcqXUNGBsO9kdl+XFWkwFyzi25GPueBi6vL+W8d4u7VxB4r+sSiGJ9/VS3L7L2AXYl5oJ8wf46fet12ux14cbWZNRMsEVdLv+LQyNJsl74rNUgQGjCmXNLj+VZbaP1MXwHKV3WzCGsOu5ovTqhgR8b1xzN/MG89gMTojrFKHY7jSzpU52K36WDn4rx8QdERkr+JS0kYe+Npy8zkESdUa1BcVAIRDFmZCt/rejc9+gypOdaJ8utD5 x-ms-exchange-antispam-srfa-diagnostics: SSOS; x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: ab3c8979-98bb-43f3-f59d-08d5630cabaa x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(7020095)(4652020)(48565401081)(4534165)(4627221)(201703031133081)(201702281549075)(5600026)(4604075)(3008032)(2017052603307)(7153060)(7193020);SRVR:DB3PR08MB0108; x-ms-traffictypediagnostic: DB3PR08MB0108: nodisclaimer: True x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(250305191791016)(180628864354917)(22074186197030)(131327999870524); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(6040501)(2401047)(5005006)(8121501046)(3231023)(2400081)(944501161)(93006095)(93001095)(10201501046)(3002001)(6055026)(6041288)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(20161123562045)(20161123560045)(6072148)(201708071742011);SRVR:DB3PR08MB0108;BCL:0;PCL:0;RULEID:;SRVR:DB3PR08MB0108; x-forefront-prvs: 056297E276 x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(39860400002)(366004)(376002)(346002)(396003)(39380400002)(377424004)(189003)(199004)(2351001)(3280700002)(4326008)(25786009)(82746002)(3660700001)(83716003)(6506007)(106356001)(97736004)(76176011)(6916009)(305945005)(99286004)(105586002)(6436002)(86362001)(575784001)(478600001)(59450400001)(5640700003)(72206003)(2950100002)(2900100001)(5250100002)(81166006)(2501003)(8936002)(8676002)(2906002)(7736002)(6306002)(5660300001)(81156014)(26005)(36756003)(4743002)(68736007)(6486002)(14454004)(33656002)(316002)(66066001)(6116002)(102836004)(3846002)(53936002)(6512007)(2004002);DIR:OUT;SFP:1101;SCL:1;SRVR:DB3PR08MB0108;H:DB3PR08MB0106.eurprd08.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; received-spf: None (protection.outlook.com: arm.com does not designate permitted sender hosts) x-microsoft-antispam-message-info: L9ytz4uebLyCKdeYJeRJdq0gemxWQ/tWjXU13h7KYDTUtWwIQc2xnQk+eRdwW0sctA/lWCEmIDm7ZiVlm6g+YA== spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-ID: Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-Network-Message-Id: ab3c8979-98bb-43f3-f59d-08d5630cabaa X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Jan 2018 09:27:21.2877 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB3PR08MB0108 X-IsSubscribed: yes X-SW-Source: 2018-01/txt/msg00464.txt.bz2 This patch commonises tdesc_reg and makes use of it in gdbserver tdesc. tdesc_create_reg is changed to create a tdesc_reg and place it in tdesc_feature. After registers have been parsed, init_target_desc is called. I have expanded this function so that it now populates the reg_defs directly from the tdesc_reg registers. The long term goal is to change the other files within gdbserver to access member functions instead of reg_defs directly. reg_defs can then be removed. This is out of the scope of this patch series. I wanted to make tdesc_create_reg common, but I cannot do that until the next patch. I also had to commonise tdesc_element_visitor and tdesc_element. The gdbserver makefile change is a little strange. I had to add using "..". I think there is a rule that needs changing, but I have no idea where it is. Note that this is the first file in arch which is common for all builds. I'm also wondering if tdesc.c/h should be in arch/ ? Maybe it should be moved to common? Alan. 2018-01-24 Alan Hayward gdb/ * Makefile.in: Add arch/tdesc.c * arch/tdesc.c: New file. * arch/tdesc.h (tdesc_element_visitor): Move to here. (tdesc_element): Likewise. (tdesc_reg): Likewise. (tdesc_reg_up): Likewise.=09 * target-descriptions.c: (tdesc_element_visitor): Move from here.=20 (tdesc_element): Likewise. (tdesc_reg): Likewise. (tdesc_reg_up): Likewise.=09 gdbserver/ * Makefile.in: Add arch/tdesc.c * tdesc.c (init_target_desc): init all reg_defs from register vector. (tdesc_create_reg): Create tdesc_reg. * tdesc.h (tdesc_feature): Add register vector. diff --git a/gdb/Makefile.in b/gdb/Makefile.in index 364ea7afc66d00d0be44c4369f3ddc924de7b6aa..05ecf64451de4df3a5593d15840= 4d526a71c0b29 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -911,6 +911,7 @@ COMMON_SFILES =3D \ agent.c \ annotate.c \ arch-utils.c \ + arch/tdesc.c \ auto-load.c \ auxv.c \ ax-gdb.c \ diff --git a/gdb/arch/tdesc.h b/gdb/arch/tdesc.h index cc11651dcaa7abe81598b69f509ef34ff0d94dbf..e957822b348182b8cae9809cd21= f72974ffe05c5 100644 --- a/gdb/arch/tdesc.h +++ b/gdb/arch/tdesc.h @@ -18,6 +18,12 @@ #ifndef ARCH_TDESC_H #define ARCH_TDESC_H 1 +#ifdef GDBSERVER +#include "server.h" +#else +#include "defs.h" +#endif + struct tdesc_feature; struct tdesc_type; struct tdesc_type_builtin; @@ -26,6 +32,99 @@ struct tdesc_type_with_fields; struct tdesc_reg; struct target_desc; +/* The interface to visit different elements of target description. */ + +class tdesc_element_visitor +{ +public: + virtual void visit_pre (const target_desc *e) =3D 0; + virtual void visit_post (const target_desc *e) =3D 0; + + virtual void visit_pre (const tdesc_feature *e) =3D 0; + virtual void visit_post (const tdesc_feature *e) =3D 0; + + virtual void visit (const tdesc_type_builtin *e) =3D 0; + virtual void visit (const tdesc_type_vector *e) =3D 0; + virtual void visit (const tdesc_type_with_fields *e) =3D 0; + + virtual void visit (const tdesc_reg *e) =3D 0; +}; + +class tdesc_element +{ +public: + virtual void accept (tdesc_element_visitor &v) const =3D 0; +}; + +/* An individual register from a target description. */ + +struct tdesc_reg : tdesc_element +{ + tdesc_reg (struct tdesc_feature *feature, const std::string &name_, + int regnum, int save_restore_, const char *group_, + int bitsize_, const char *type_); + + virtual ~tdesc_reg () =3D default; + + DISABLE_COPY_AND_ASSIGN (tdesc_reg); + + /* The name of this register. In standard features, it may be + recognized by the architecture support code, or it may be purely + for the user. */ + std::string name; + + /* The register number used by this target to refer to this + register. This is used for remote p/P packets and to determine + the ordering of registers in the remote g/G packets. */ + long target_regnum; + + /* If this flag is set, GDB should save and restore this register + around calls to an inferior function. */ + int save_restore; + + /* The name of the register group containing this register, or empty + if the group should be automatically determined from the + register's type. If this is "general", "float", or "vector", the + corresponding "info" command should display this register's + value. It can be an arbitrary string, but should be limited to + alphanumeric characters and internal hyphens. Currently other + strings are ignored (treated as empty). */ + std::string group; + + /* The size of the register, in bits. */ + int bitsize; + + /* The type of the register. This string corresponds to either + a named type from the target description or a predefined + type from GDB. */ + std::string type; + + /* The target-described type corresponding to TYPE, if found. */ + struct tdesc_type *tdesc_type; + + void accept (tdesc_element_visitor &v) const override + { + v.visit (this); + } + + bool operator=3D=3D (const tdesc_reg &other) const + { + return (name =3D=3D other.name + && target_regnum =3D=3D other.target_regnum + && save_restore =3D=3D other.save_restore + && bitsize =3D=3D other.bitsize + && group =3D=3D other.group + && type =3D=3D other.type); + } + + bool operator!=3D (const tdesc_reg &other) const + { + return !(*this =3D=3D other); + } +}; + +typedef std::unique_ptr tdesc_reg_up; + /* Allocate a new target_desc. */ target_desc *allocate_target_description (void); diff --git a/gdb/arch/tdesc.c b/gdb/arch/tdesc.c new file mode 100644 index 0000000000000000000000000000000000000000..3d84ad8d3fa2e017f4c6f7d137a= 3be2b58427370 --- /dev/null +++ b/gdb/arch/tdesc.c @@ -0,0 +1,40 @@ +/* Target description support for GDB. + + Copyright (C) 2017 Free Software Foundation, Inc. + + This file is part of GDB. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . = */ + +#ifdef GDBSERVER +#include "server.h" +#else +#include "defs.h" +#endif + +#include "tdesc.h" + +tdesc_reg::tdesc_reg (struct tdesc_feature *feature, const std::string &na= me_, + int regnum, int save_restore_, const char *group_, + int bitsize_, const char *type_) + : name (name_), target_regnum (regnum), + save_restore (save_restore_), + group (group_ !=3D NULL ? group_ : ""), + bitsize (bitsize_), + type (type_ !=3D NULL ? type_ : "") +{ + /* If the register's type is target-defined, look it up now. We may not + have easy access to the containing feature when we want it later. */ + tdesc_type =3D tdesc_named_type (feature, type.c_str ()); +} diff --git a/gdb/gdbserver/Makefile.in b/gdb/gdbserver/Makefile.in index 3ce086d70f23df445b174c49c489ec8415d7614a..8ae124875dd08f31e8424b18281= cb1454ed40edb 100644 --- a/gdb/gdbserver/Makefile.in +++ b/gdb/gdbserver/Makefile.in @@ -194,6 +194,7 @@ SFILES =3D \ $(srcdir)/arch/arm.c \ $(srcdir)/arch/arm-get-next-pcs.c \ $(srcdir)/arch/arm-linux.c \ + $(srcdir)/../arch/tdesc.c \ $(srcdir)/common/btrace-common.c \ $(srcdir)/common/buffer.c \ $(srcdir)/common/cleanups.c \ @@ -232,6 +233,7 @@ TAGFILES =3D $(SOURCES) ${HFILES} ${ALLPARAM} ${POSSLIB= S} OBS =3D \ agent.o \ + arch/tdesc.o \ ax.o \ btrace-common.o \ buffer.o \ @@ -391,6 +393,7 @@ gdbreplay$(EXEEXT): $(GDBREPLAY_OBS) $(LIBGNU) $(LIBIBE= RTY) $(XM_CLIBS) $(LIBGNU) $(LIBIBERTY) IPA_OBJS =3D \ + arch/tdesc-ipa.o \ ax-ipa.o \ common-utils-ipa.o \ errors-ipa.o \ diff --git a/gdb/gdbserver/tdesc.h b/gdb/gdbserver/tdesc.h index 705b968b98af9405682507acbcfb41356da62857..cfc1a194def019ecf899e918438= 5be94de39d08c 100644 --- a/gdb/gdbserver/tdesc.h +++ b/gdb/gdbserver/tdesc.h @@ -25,7 +25,10 @@ #include struct tdesc_feature -{}; +{ + /* The registers associated with this feature. */ + std::vector registers; +}; /* A target description. Inherit from tdesc_feature so that target_desc can be used as tdesc_feature. */ diff --git a/gdb/gdbserver/tdesc.c b/gdb/gdbserver/tdesc.c index 359ab97361c07ff2cbc2c8d5ff392c6233c329ff..5dec636baa6b899490e8c700eea= 21d363fb79fa3 100644 --- a/gdb/gdbserver/tdesc.c +++ b/gdb/gdbserver/tdesc.c @@ -72,9 +72,27 @@ init_target_desc (struct target_desc *tdesc) { int offset =3D 0; - for (reg *reg : tdesc->reg_defs) + /* Go through all the features and populate reg_defs. */ + for (const tdesc_reg_up &treg : tdesc->registers) { + /* Fill in any blank spaces. */ + while (tdesc->reg_defs.size () < treg->target_regnum) + { + struct reg *reg =3D XCNEW (struct reg); + reg->name =3D ""; + reg->size =3D 0; + reg->offset =3D offset; + tdesc->reg_defs.push_back (reg); + } + + gdb_assert (treg->target_regnum =3D=3D 0 + || treg->target_regnum =3D=3D tdesc->reg_defs.size ()); + + struct reg *reg =3D XCNEW (struct reg); + reg->name =3D treg->name.c_str (); + reg->size =3D treg->bitsize; reg->offset =3D offset; + tdesc->reg_defs.push_back (reg); offset +=3D reg->size; } @@ -241,23 +259,10 @@ tdesc_create_reg (struct tdesc_feature *feature, cons= t char *name, { struct target_desc *tdesc =3D (struct target_desc *) feature; - while (tdesc->reg_defs.size () < regnum) - { - struct reg *reg =3D XCNEW (struct reg); - - reg->name =3D ""; - reg->size =3D 0; - tdesc->reg_defs.push_back (reg); - } - - gdb_assert (regnum =3D=3D 0 - || regnum =3D=3D tdesc->reg_defs.size ()); - - struct reg *reg =3D XCNEW (struct reg); + tdesc_reg *reg =3D new tdesc_reg (feature, name, regnum, save_restore, + group, bitsize, type); - reg->name =3D name; - reg->size =3D bitsize; - tdesc->reg_defs.push_back (reg); + tdesc->registers.emplace_back (reg); } /* See arch/tdesc.h. */ diff --git a/gdb/target-descriptions.c b/gdb/target-descriptions.c index 1b20a12d769718e591dea6df8183c2e9ecfac990..881ef78d2e3adde409e8585deb6= 7c96e2f0594ed 100644 --- a/gdb/target-descriptions.c +++ b/gdb/target-descriptions.c @@ -38,30 +38,6 @@ #include "completer.h" #include "readline/tilde.h" /* tilde_expand */ -/* The interface to visit different elements of target description. */ - -class tdesc_element_visitor -{ -public: - virtual void visit_pre (const target_desc *e) =3D 0; - virtual void visit_post (const target_desc *e) =3D 0; - - virtual void visit_pre (const tdesc_feature *e) =3D 0; - virtual void visit_post (const tdesc_feature *e) =3D 0; - - virtual void visit (const tdesc_type_builtin *e) =3D 0; - virtual void visit (const tdesc_type_vector *e) =3D 0; - virtual void visit (const tdesc_type_with_fields *e) =3D 0; - - virtual void visit (const tdesc_reg *e) =3D 0; -}; - -class tdesc_element -{ -public: - virtual void accept (tdesc_element_visitor &v) const =3D 0; -}; - /* Types. */ struct property @@ -74,84 +50,6 @@ struct property std::string value; }; -/* An individual register from a target description. */ - -struct tdesc_reg : tdesc_element -{ - tdesc_reg (struct tdesc_feature *feature, const std::string &name_, - int regnum, int save_restore_, const char *group_, - int bitsize_, const char *type_) - : name (name_), target_regnum (regnum), - save_restore (save_restore_), - group (group_ !=3D NULL ? group_ : ""), - bitsize (bitsize_), - type (type_ !=3D NULL ? type_ : "") - { - /* If the register's type is target-defined, look it up now. We may n= ot - have easy access to the containing feature when we want it later. = */ - tdesc_type =3D tdesc_named_type (feature, type.c_str ()); - } - - virtual ~tdesc_reg () =3D default; - - DISABLE_COPY_AND_ASSIGN (tdesc_reg); - - /* The name of this register. In standard features, it may be - recognized by the architecture support code, or it may be purely - for the user. */ - std::string name; - - /* The register number used by this target to refer to this - register. This is used for remote p/P packets and to determine - the ordering of registers in the remote g/G packets. */ - long target_regnum; - - /* If this flag is set, GDB should save and restore this register - around calls to an inferior function. */ - int save_restore; - - /* The name of the register group containing this register, or empty - if the group should be automatically determined from the register's - type. This is traditionally "general", "float", "vector" but can - also be an arbitrary string. If defined the corresponding "info" - command should display this register's value. The string should be - limited to alphanumeric characters and internal hyphens. */ - std::string group; - - /* The size of the register, in bits. */ - int bitsize; - - /* The type of the register. This string corresponds to either - a named type from the target description or a predefined - type from GDB. */ - std::string type; - - /* The target-described type corresponding to TYPE, if found. */ - struct tdesc_type *tdesc_type; - - void accept (tdesc_element_visitor &v) const override - { - v.visit (this); - } - - bool operator=3D=3D (const tdesc_reg &other) const - { - return (name =3D=3D other.name - && target_regnum =3D=3D other.target_regnum - && save_restore =3D=3D other.save_restore - && bitsize =3D=3D other.bitsize - && group =3D=3D other.group - && type =3D=3D other.type); - } - - bool operator!=3D (const tdesc_reg &other) const - { - return !(*this =3D=3D other); - } -}; - -typedef std::unique_ptr tdesc_reg_up; - /* A named type from a target description. */ struct tdesc_type_field