From patchwork Tue Apr 10 14:33:35 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alan Hayward X-Patchwork-Id: 26672 Received: (qmail 54024 invoked by alias); 10 Apr 2018 14:34:21 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 53923 invoked by uid 89); 10 Apr 2018 14:34:20 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-26.0 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_HELO_PASS, SPF_PASS autolearn=ham version=3.3.2 spammy=visitor X-HELO: EUR02-HE1-obe.outbound.protection.outlook.com Received: from mail-eopbgr10088.outbound.protection.outlook.com (HELO EUR02-HE1-obe.outbound.protection.outlook.com) (40.107.1.88) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 10 Apr 2018 14:34:15 +0000 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Alan.Hayward@arm.com; Received: from c02tf0u7hf1t.manchester.arm.com (217.140.96.140) by HE1PR0802MB2138.eurprd08.prod.outlook.com (2603:10a6:3:c2::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.653.12; Tue, 10 Apr 2018 14:33:56 +0000 From: Alan Hayward To: gdb-patches@sourceware.org Cc: nd@arm.com, Alan Hayward Subject: [PATCH v5 6/8] Create xml from target descriptions Date: Tue, 10 Apr 2018 15:33:35 +0100 Message-Id: <20180410143337.71768-7-alan.hayward@arm.com> In-Reply-To: <20180410143337.71768-1-alan.hayward@arm.com> References: <20180410143337.71768-1-alan.hayward@arm.com> MIME-Version: 1.0 X-ClientProxiedBy: CWXP265CA0019.GBRP265.PROD.OUTLOOK.COM (2603:10a6:400:2e::31) To HE1PR0802MB2138.eurprd08.prod.outlook.com (2603:10a6:3:c2::10) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 73dd4e41-86b0-40e1-c781-08d59ef017e6 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(48565401081)(5600026)(4604075)(4534165)(4627221)(201703031133081)(201702281549075)(2017052603328)(7153060)(7193020); SRVR:HE1PR0802MB2138; X-Microsoft-Exchange-Diagnostics: 1; HE1PR0802MB2138; 3:9KV+rZhD411s3/721s1MpLt7NEPIK1PxplxmhJYyGArAAa85jiv4DwewilQGLay4n4z8dKjgP36xu1WLOvXh5aqgqSlN9/s92nhpgjnJhUUNPiZi9JaUxqPY1GSezC9TH6QeAGnzFy2KW//xidOQ7A0QqXsEms8yQ98xpiW6M9pKbvv4UiD2gtlNfPbZw52TW9im3j29nxirL5wS+1QXeuNzbqi7vZDZygQW831xrfroFuxV89Wu2o0ZVhWe4mKA; 25:6SUXZSdTV33wmoj5vcVAE7xuU91KbARLHhO5hzb6quN3R3nIDAdt4zXXd8HePekWrFC/nUmBm3P5F1BXgNk8aNrDi0ilddEWTlm4cNoX4KWZKgVHPcwshioeir0wPasXT132H0c0E7vrQHAlO1bIeh/r8bRf3rcoPG73s3vzq8ymbC+Gj9alDPgS0VhxduYs4yJH+jpzcBC/yhyKslmAbDEID0BzJT91KvN4oUTTDAMSoGESoexcpxy36NojCxiUGq/tJKKzBEIMiwNgU3jcjvChJ5E7ByyG/XxcYVtlqQClERa9k0iu1cXE2HyGRQcMmkcvDmHsvv3gGDVvDIub0Q==; 31:0MUs/wcdUVCYoGNzN6Nax1Ok0m/+Oe/I5XKqzZeWuy2sz5AkomKdm0PkQgJqp7KFvo399wz80oLHda84ERVwjsxy6B3wG1MTgGTEejd7FdsAJnV/s9BNvmpBTIHKak+wiR2e0omq05mM4VHYz57Juwpneug5Wb82A+IGKDA0AC373jL3ZHV1ztAVVJi/DF2BMimYgUD+xGayHBndIg8Kkf71AOz8g5WXPv8qkI027BU= X-MS-TrafficTypeDiagnostic: HE1PR0802MB2138: NoDisclaimer: True X-Microsoft-Exchange-Diagnostics: 1; HE1PR0802MB2138; 20:6S7mhKwiAqLycGVC0Ft8/q1hcvgIYJ5AeUwSKYcNmElTZlhW+4SFU/CfjSWYnPFJmWuoAo29aco4wyvlQgmnGc2j9cSYpD4nI9BhqNhYapzxotUkk+BXwDjxuXv5QMerzv0yV9x8xOUI7MrnbuPfGAcZ+vqPKwmaDCV+pM6Krtg=; 4:Id77nBjkjQ+ABwc0xl3iHsWMT0WUXIO6XMpZbfmb/4w/IxycTSuJHMI/zoSDFbYN8KiBzD9T6MZyDbIvXpBqynDG+s1qOKEhyxNG7r98TEWE1U2NBQGTdHSTgcoRCFKG3sSk9wntafsgnU5ZHmAtbedaqar4REIRlmrKtSWdFUBYLZIAqeiCXVgELReQhR8zbBRsnUpxF9lvMKk2x9uZ1BFQfcl+CgFuWICYXwr0Om9U80p0ZjLRZdj9NutscuCzoa6G+5OaSc03oo1rSoPQgqMdAwhYvzq+aQAkJ2DS7LU5BH+fExD1+dwLfpWDGjMcTjwoMHPyLBMpgLcA8dSQ1RRKQA9/qySNu7REdTTC/qI= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(180628864354917)(131327999870524); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(93006095)(93001095)(3231221)(944501327)(52105095)(3002001)(10201501046)(6055026)(6041310)(20161123560045)(20161123564045)(20161123558120)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(6072148)(201708071742011); SRVR:HE1PR0802MB2138; BCL:0; PCL:0; RULEID:; SRVR:HE1PR0802MB2138; X-Forefront-PRVS: 0638FD5066 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(39380400002)(396003)(346002)(39860400002)(366004)(376002)(377424004)(189003)(199004)(76176011)(386003)(52116002)(7696005)(36756003)(59450400001)(51416003)(6486002)(105586002)(486006)(6666003)(11346002)(6916009)(476003)(2906002)(3846002)(2616005)(6116002)(956004)(2361001)(446003)(1076002)(2351001)(186003)(316002)(26005)(86362001)(16526019)(8676002)(68736007)(16586007)(81166006)(81156014)(5660300001)(7736002)(8936002)(53936002)(106356001)(4326008)(478600001)(66066001)(305945005)(25786009)(48376002)(97736004)(72206003)(575784001)(50226002)(50466002)(47776003); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR0802MB2138; H:c02tf0u7hf1t.manchester.arm.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; Received-SPF: None (protection.outlook.com: arm.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; HE1PR0802MB2138; 23:NE3uIFxdXrnq37HcEvPHtFDFwQ9nVYwkykruVEZ?= =?us-ascii?Q?tYZ6/yphfYWwq/1nogCGzWwE2sU3+OPmHOWOcsxFJInICGJtgNwl0MUzC/TE?= =?us-ascii?Q?YpTqqU+2V+np5beGKrXNZj5kqxk8BJbqJung90Q9soNhdZdl66INBMQn7hlD?= =?us-ascii?Q?nBOWqAKgzap6bo0BpXy7YL/GWk5CPVQQgpurjx+mZxd77pPdbiMhzLVumvnp?= =?us-ascii?Q?4qEhA1Dng+fv+Aropf/PAH+YVO0im/mtNx2Sh9Pr/RACG4kA1poC9eK+9YXx?= =?us-ascii?Q?0HpCn+rgHHe4gsb7hGGIDjJ+BITeXGcQ1FIRjptKiVDr4XB043GYnBDYQhC2?= =?us-ascii?Q?LxPrB+FaFwWJ4BvvoSFIMPq2BJdAwa5IJKD+s36tI7fPTqmxyI84maK8QxmG?= =?us-ascii?Q?4g9NkC2bNnVPejV6lM4iuY/6JrnOFmHcinXen85hNsftaN6eZxZwdmnnWRVs?= =?us-ascii?Q?p/DgUAxCXNyRufF9FVioEKxuhUVOXU/PnJ2dDEg4FH6+7lyJaV4g6QEJJdsn?= =?us-ascii?Q?aDD57B7c/ynPiwbqEsLsmbEQeR0HEiRSBw/OUeg8aNw7elWoK75QaJevoXK1?= =?us-ascii?Q?Y4jws0VaZLkcwSeWrXhr0wJFI4LeH7bOi0qDJtOJAPLIE38Ieb8y2PdEJhCl?= =?us-ascii?Q?RxmQaavz1m8rI1TyUC+B3KwK4MDF0OvIN1ycEjR5W2fP5+S0dprSxja6vlhh?= =?us-ascii?Q?hnTCpgwekH2qwr8gbZwEdQgu3ZQgWwMlq8dsiz/oiDUp/TVK7TVCC7u89YFx?= =?us-ascii?Q?655JnzSTrtL3fh1vM4SXUjPwLnmwH+mk1gQf5BVA98fLRuWFZ+o+nSJ2eUpm?= =?us-ascii?Q?s7k40ZjspBvXT3phwpeBAzFuuVM2XB3oj7An3sRC0X1vlSyK82ZHRy9Bx3Xy?= =?us-ascii?Q?ySifFvZN1oNImjMC2WL+oZMoCSm+UnoRjTEcwFgf5sTG/vOeCUHHgZKy2XdP?= =?us-ascii?Q?t+YTVI6YbunBDp4H3OE87iQKNfFpIcjWrx1q9gEvSW3PprDmkgo5J1yn5Sya?= =?us-ascii?Q?CfbNBuyHgWt2o0suuL4VraMYOnlUrT9AX9xFVQCl5koPHUY9rLVQU9E8tmSM?= =?us-ascii?Q?K1yysVMkyY0qI038+v2BHUPzIZqDsYkW+yuau5bp2rENHOBP9jG4w2lltZXu?= =?us-ascii?Q?TjA/HqkQD1IBIg3wxwrBanXWXwIGo29Gn/PmFvK4+gjwXYfo02C2b5oKmH1O?= =?us-ascii?Q?CKc77gceyaiGrC2IIxkwzqIVscT446SqDBAKTImtUl5wZ1DvAp0ZyHflxi5K?= =?us-ascii?Q?Qw++8W9zjRTQuj/174vNLUIImD5zk0bvxP6dzQYNm723tHJwQOIOUp58xBIa?= =?us-ascii?Q?tc3/N92I8VEYy0Sc46GUP4J0=3D?= X-Microsoft-Antispam-Message-Info: cpbRlz6A4PCDWHnNDzL/oBVK8+F3SGtxopgECmbJV0G07L+z6t6gSX0H5lkIg0GOj69vHhWqI5AzcG9NATSN/ntuUWDiUHKzw/LBZ4gSzrGYZ1Mn7wdVmpVGGzKX6AQPH/NrCKIsv2gfu051uE0UlBngqzwA+61NXb/OxTCeOwJnQEXNMcE2KMqpm3FMT3/Z X-Microsoft-Exchange-Diagnostics: 1; HE1PR0802MB2138; 6:ZA6JcbQtZNjYRmrZQ8ananXwdi3GGY5DM9OSZ+gG58Ivb3D8gB3isssT9SUCoRNl9eDG/Xd0KeZFDFrjBIUli8Z5C1ctonXXaCclM79YALvpDGjnpHjJF6sLLJIfTa/HX1/RrIWQ9/u8TIk3g3bUtl9hVYjGLZC60H022Fs8sq4zuT7ApPsJw0DoIsxod74dc54zk3dy7TqK8s+gbW1xZPKHx7jE8v+o7HdbCR3p5EEyUNizt/urB8rbW87JL5r5TRIwhorPy07nH8CT+84dqPAjRYOvVPkCiBhLPZG8PcmL+yWjOGvPQuu+5wFoh2WvcAPpZ9e0nEHW2d7JoO1Tkc7S4vlVgzZhH930/WR9D9PAyPPeHvNaLfSzULMZTvgIjVHZ7ClJVv5596sebLxJBdoh0XhHdpWPFjtUH8qWiIW9gbkoNVS1tOVUCvUGbbUNELV3s8smHmHrsx0Hoc9GZg==; 5:nn0hHSh9k6k3wi8h4hb8pPae5Pt1Lw9jjVSGHAagMzFU161kZq9gPExEvb6tojBumYXEyaHea7ZsC5Hm78Q3ALQHZqmqNwFvy8i885w5t9IGqJ/wj08iVPy9XQEEf0GMZucJMOkKzyGA3Xl8ozE85r6+nLtY7kLy2DIle36FAYQ=; 24:Ey5E5v7+RcvoDsJH2rMCkxu7LldyuD4ta1wtHVxTlGB2wvaNzZfsYmywipjopkmj7be/lJdrwaAiCLkGAWMOo5XY5ESTcC1TLWe4UWeVMS4= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; HE1PR0802MB2138; 7:VfdAJ8nxs12SRbhkiJO/kpWfJawbni8SMJqluOffmEp9xvoRMoRMGwPZfc+2FJ/yUR/8FetuGp82eZw/BJ1HqrN4QTpXDE1mAVsyFAuU0YLUII5sV0m8tj1eNS7OabzjUxhu+8UkcF88pmLQIAFK4QaaEbZOoZDi7Taq3G1P0/b5ISThQDXujr8rGwVzk82nnL532vEvRq+UGXo8MMBEaLeWIgGK4/jL2KgqlWaK/oqGoVeQdqSoZ4kbJr03Buvm X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Apr 2018 14:33:56.9226 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 73dd4e41-86b0-40e1-c781-08d59ef017e6 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0802MB2138 X-IsSubscribed: yes This patch adds a print_xml_feature visitor class which turns a C target description into xml. Both gdb and gdbserver can do this. An accept function is added to gdbserver tdesc to allow it to use vistor classes. Tests are added to maintenance_check_xml_descriptions which takes each pair of tested descriptions, turns them both into xml, then back again, and confirms the descriptions still match. Differences from V4 are: print_xml_feature now uses a ptr. tdesc xmltarget is now mutable. Added maintenance_check_tdesc_xml_convert test function. Improved string_read_description_xml. This patch requires the next patch in the series, otherwise target description will be rejected by gdb when using gdbserver. I'll make sure the two patches get committed in a single commit. Alan. 2018-04-10 Alan Hayward gdb/ * common/tdesc.c (print_xml_feature::visit_pre): Add xml parsing. (print_xml_feature::visit_post): Likewise. (print_xml_feature::visit): Likewise. * common/tdesc.h (tdesc_get_features_xml): Use const tdesc. (print_xml_feature): Add new class. * regformats/regdat.sh: Null xmltarget on feature targets. * target-descriptions.c (struct target_desc): Add xmltarget. (maintenance_check_tdesc_xml_convert): Add unittest function. (tdesc_get_features_xml): Add function to get xml. (maintenance_check_xml_descriptions): Test xml generation. * xml-tdesc.c (string_read_description_xml): Add function. * xml-tdesc.h (string_read_description_xml): Add declaration. gdbserver/ * gdb/gdbserver/server.c (get_features_xml): Remove cast. * tdesc.c (void target_desc::accept): Fill in function. (tdesc_get_features_xml): Remove old xml creation. (print_xml_feature::visit_pre): Add xml vistor. * tdesc.h (struct target_desc): Make xmltarget mutable. (tdesc_get_features_xml): Remove declaration. --- gdb/common/tdesc.c | 108 ++++++++++++++++++++++++++++++++++++++++++++++ gdb/common/tdesc.h | 29 +++++++++++++ gdb/gdbserver/server.c | 2 +- gdb/gdbserver/tdesc.c | 45 ++++++++----------- gdb/gdbserver/tdesc.h | 10 ++--- gdb/regformats/regdat.sh | 4 +- gdb/target-descriptions.c | 56 ++++++++++++++++++++++++ gdb/xml-tdesc.c | 11 +++++ gdb/xml-tdesc.h | 5 +++ 9 files changed, 234 insertions(+), 36 deletions(-) diff --git a/gdb/common/tdesc.c b/gdb/common/tdesc.c index b9e9ddb3fa..b4bd7bfb00 100644 --- a/gdb/common/tdesc.c +++ b/gdb/common/tdesc.c @@ -290,3 +290,111 @@ tdesc_add_enum_value (tdesc_type_with_fields *type, int value, tdesc_predefined_type (TDESC_TYPE_INT32), value, -1); } + +void print_xml_feature::visit_pre (const tdesc_feature *e) +{ + string_appendf (*m_buffer, "\n", e->name.c_str ()); +} + +void print_xml_feature::visit_post (const tdesc_feature *e) +{ + string_appendf (*m_buffer, "\n"); +} + +void print_xml_feature::visit (const tdesc_type_builtin *t) +{ + error (_("xml output is not supported type \"%s\"."), t->name.c_str ()); +} + +void print_xml_feature::visit (const tdesc_type_vector *t) +{ + string_appendf (*m_buffer, "\n", + t->name.c_str (), t->element_type->name.c_str (), t->count); +} + +void print_xml_feature::visit (const tdesc_type_with_fields *t) +{ + struct tdesc_type_field *f; + const static char *types[] = { "struct", "union", "flags", "enum" }; + + gdb_assert (t->kind >= TDESC_TYPE_STRUCT && t->kind <= TDESC_TYPE_ENUM); + + string_appendf (*m_buffer, + "<%s id=\"%s\"", types[t->kind - TDESC_TYPE_STRUCT], + t->name.c_str ()); + + switch (t->kind) + { + case TDESC_TYPE_STRUCT: + case TDESC_TYPE_FLAGS: + if (t->size > 0) + string_appendf (*m_buffer, " size=\"%d\"", t->size); + string_appendf (*m_buffer, ">\n"); + + for (const tdesc_type_field &f : t->fields) + { + string_appendf (*m_buffer, " \n", + f.type->name.c_str ()); + else + string_appendf (*m_buffer, "start=\"%d\" end=\"%d\"/>\n", f.start, + f.end); + } + break; + + case TDESC_TYPE_ENUM: + string_appendf (*m_buffer, ">\n"); + for (const tdesc_type_field &f : t->fields) + string_appendf (*m_buffer, " \n", + f.name.c_str (), f.start); + break; + + case TDESC_TYPE_UNION: + string_appendf (*m_buffer, ">\n"); + for (const tdesc_type_field &f : t->fields) + string_appendf (*m_buffer, " \n", + f.name.c_str (), f.type->name.c_str ()); + break; + + default: + error (_("xml output is not supported type \"%s\"."), t->name.c_str ()); + } + + string_appendf (*m_buffer, "\n", types[t->kind - TDESC_TYPE_STRUCT]); +} + +void print_xml_feature::visit (const tdesc_reg *r) +{ + string_appendf (*m_buffer, + "name.c_str (), r->bitsize, r->type.c_str (), + r->target_regnum); + + if (r->group.length () > 0) + string_appendf (*m_buffer, " group=\"%s\"", r->group.c_str ()); + + if (r->save_restore == 0) + string_appendf (*m_buffer, " save-restore=\"no\""); + + string_appendf (*m_buffer, "/>\n"); +} + +void print_xml_feature::visit_pre (const target_desc *e) +{ +#ifndef IN_PROCESS_AGENT + string_appendf (*m_buffer, "\n"); + string_appendf (*m_buffer, "\n"); + string_appendf (*m_buffer, "\n%s\n", + tdesc_architecture_name (e)); + + const char *osabi = tdesc_osabi_name (e); + if (osabi != nullptr) + string_appendf (*m_buffer, "%s", osabi); +#endif +} + +void print_xml_feature::visit_post (const target_desc *e) +{ + string_appendf (*m_buffer, "\n"); +} diff --git a/gdb/common/tdesc.h b/gdb/common/tdesc.h index 311341da0d..00eaada072 100644 --- a/gdb/common/tdesc.h +++ b/gdb/common/tdesc.h @@ -381,4 +381,33 @@ void tdesc_create_reg (struct tdesc_feature *feature, const char *name, int regnum, int save_restore, const char *group, int bitsize, const char *type); +/* Return the tdesc in string XML format. */ + +const char *tdesc_get_features_xml (const target_desc *tdesc); + +/* Print target description as xml. */ + +class print_xml_feature : public tdesc_element_visitor +{ +public: + print_xml_feature (std::string *buffer_) + : m_buffer (buffer_) + {} + + ~print_xml_feature () + {} + + void visit_pre (const target_desc *e) override; + void visit_post (const target_desc *e) override; + void visit_pre (const tdesc_feature *e) override; + void visit_post (const tdesc_feature *e) override; + void visit (const tdesc_type_builtin *type) override; + void visit (const tdesc_type_vector *type) override; + void visit (const tdesc_type_with_fields *type) override; + void visit (const tdesc_reg *reg) override; + +private: + std::string *m_buffer; +}; + #endif /* ARCH_TDESC_H */ diff --git a/gdb/gdbserver/server.c b/gdb/gdbserver/server.c index 64c72bdd58..5027df5e10 100644 --- a/gdb/gdbserver/server.c +++ b/gdb/gdbserver/server.c @@ -940,7 +940,7 @@ get_features_xml (const char *annex) if (strcmp (annex, "target.xml") == 0) { - const char *ret = tdesc_get_features_xml ((target_desc*) desc); + const char *ret = tdesc_get_features_xml (desc); if (*ret == '@') return ret + 1; diff --git a/gdb/gdbserver/tdesc.c b/gdb/gdbserver/tdesc.c index 7603a90a59..126589f3e3 100644 --- a/gdb/gdbserver/tdesc.c +++ b/gdb/gdbserver/tdesc.c @@ -47,6 +47,18 @@ bool target_desc::operator== (const target_desc &other) const #endif +void target_desc::accept (tdesc_element_visitor &v) const +{ +#ifndef IN_PROCESS_AGENT + v.visit_pre (this); + + for (const tdesc_feature_up &feature : features) + feature->accept (v); + + v.visit_post (this); +#endif +} + void init_target_desc (struct target_desc *tdesc) { @@ -138,11 +150,10 @@ set_tdesc_osabi (struct target_desc *target_desc, const char *name) target_desc->osabi = xstrdup (name); } -/* Return a string which is of XML format, including XML target - description to be sent to GDB. */ +/* See common/tdesc.h. */ const char * -tdesc_get_features_xml (target_desc *tdesc) +tdesc_get_features_xml (const target_desc *tdesc) { /* Either .xmltarget or .features is not NULL. */ gdb_assert (tdesc->xmltarget != NULL @@ -151,31 +162,9 @@ tdesc_get_features_xml (target_desc *tdesc) if (tdesc->xmltarget == NULL) { - std::string buffer ("@"); - - buffer += ""; - buffer += ""; - buffer += ""; - buffer += tdesc_architecture_name (tdesc); - buffer += ""; - - const char *osabi = tdesc_osabi_name (tdesc); - if (osabi != nullptr) - { - buffer += ""; - buffer += osabi; - buffer += ""; - } - - for (const tdesc_feature_up &feature : tdesc->features) - { - buffer += "name; - buffer += "\"/>"; - } - - buffer += ""; - + std::string buffer ("@"); + print_xml_feature v (&buffer); + tdesc->accept (v); tdesc->xmltarget = xstrdup (buffer.c_str ()); } diff --git a/gdb/gdbserver/tdesc.h b/gdb/gdbserver/tdesc.h index 197fb59127..2152b17947 100644 --- a/gdb/gdbserver/tdesc.h +++ b/gdb/gdbserver/tdesc.h @@ -27,7 +27,7 @@ /* A target description. Inherit from tdesc_feature so that target_desc can be used as tdesc_feature. */ -struct target_desc +struct target_desc : tdesc_element { /* A vector of elements of register definitions that describe the inferior's register set. */ @@ -51,7 +51,7 @@ struct target_desc It can be NULL, then, its content is got from the following three fields features, arch, and osabi in tdesc_get_features_xml. */ - const char *xmltarget = NULL; + mutable const char *xmltarget = NULL; /* The value of element in the XML, replying GDB. */ const char *arch = NULL; @@ -73,6 +73,8 @@ public: return !(*this == other); } #endif + + void accept (tdesc_element_visitor &v) const; }; /* Copy target description SRC to DEST. */ @@ -89,8 +91,4 @@ void init_target_desc (struct target_desc *tdesc); const struct target_desc *current_target_desc (void); -#ifndef IN_PROCESS_AGENT -const char *tdesc_get_features_xml (struct target_desc *tdesc); -#endif - #endif /* TDESC_H */ diff --git a/gdb/regformats/regdat.sh b/gdb/regformats/regdat.sh index 18108d77eb..8f546fe276 100755 --- a/gdb/regformats/regdat.sh +++ b/gdb/regformats/regdat.sh @@ -163,7 +163,9 @@ done echo echo "static const char *expedite_regs_${name}[] = { \"`echo ${expedite} | sed 's/,/", "/g'`\", 0 };" -if test "${xmltarget}" = x; then +if test "${feature}" != x; then + echo "static const char *xmltarget_${name} = 0;" +elif test "${xmltarget}" = x; then if test "${xmlarch}" = x && test "${xmlosabi}" = x; then echo "static const char *xmltarget_${name} = 0;" else diff --git a/gdb/target-descriptions.c b/gdb/target-descriptions.c index da2c1ce345..b96ef961a4 100644 --- a/gdb/target-descriptions.c +++ b/gdb/target-descriptions.c @@ -333,6 +333,8 @@ struct target_desc : tdesc_element /* The features associated with this target. */ std::vector features; + mutable char *xmltarget = nullptr; + void accept (tdesc_element_visitor &v) const override { v.visit_pre (this); @@ -1667,6 +1669,21 @@ private: int m_next_regnum = 0; }; +/* See common/tdesc.h. */ + +const char * +tdesc_get_features_xml (const target_desc *tdesc) +{ + if (tdesc->xmltarget == nullptr) + { + std::string buffer ("@"); + print_xml_feature v (&buffer); + tdesc->accept (v); + tdesc->xmltarget = xstrdup (buffer.c_str ()); + } + return tdesc->xmltarget; +} + static void maint_print_c_tdesc_cmd (const char *args, int from_tty) { @@ -1739,6 +1756,39 @@ record_xml_tdesc (const char *xml_file, const struct target_desc *tdesc) } +/* Test the convesion process of a target description to/from xml: Take a target + description TDESC, convert to xml, back to a description, and confirm the new + tdesc is identical to the original. */ +static bool +maintenance_check_tdesc_xml_convert (const target_desc *tdesc, const char *name) +{ + const char *xml = tdesc_get_features_xml (tdesc); + + if (xml == nullptr && *xml != '@') + { + printf_filtered (_("Could not convert description for %s to xml.\n"), + name); + return false; + } + + const target_desc *tdesc_trans = string_read_description_xml (xml + 1); + + if (tdesc_trans == nullptr) + { + printf_filtered (_("Could not convert description for %s from xml.\n"), + name); + return false; + } + else if (*tdesc != *tdesc_trans) + { + printf_filtered (_("Converted description for %s does not match.\n"), + name); + return false; + } + return true; +} + + /* Check that the target descriptions created dynamically by architecture-specific code equal the descriptions created from XML files found in the specified directory DIR. */ @@ -1760,6 +1810,12 @@ maintenance_check_xml_descriptions (const char *dir, int from_tty) = file_read_description_xml (tdesc_xml.data ()); if (tdesc == NULL || *tdesc != *e.second) + { + printf_filtered ( _("Descriptions for %s do not match.\n"), e.first); + failed++; + } + else if (!maintenance_check_tdesc_xml_convert (tdesc, e.first) + || !maintenance_check_tdesc_xml_convert (e.second, e.first)) failed++; } printf_filtered (_("Tested %lu XML files, %d failed\n"), diff --git a/gdb/xml-tdesc.c b/gdb/xml-tdesc.c index 9190d5f3c6..5c6ee4c8cd 100644 --- a/gdb/xml-tdesc.c +++ b/gdb/xml-tdesc.c @@ -752,3 +752,14 @@ target_fetch_description_xml (struct target_ops *ops) return output; #endif } + +/* See xml-tdesc.h. */ + +const struct target_desc * +string_read_description_xml (const char *xml_str) +{ + return tdesc_parse_xml (xml_str, [] (const char *href, void *baton) { + error (_("xincludes are unsupported with this method")); + return gdb::unique_xmalloc_ptr (); + }, nullptr); +} \ No newline at end of file diff --git a/gdb/xml-tdesc.h b/gdb/xml-tdesc.h index 8f0679707a..e39d5580bb 100644 --- a/gdb/xml-tdesc.h +++ b/gdb/xml-tdesc.h @@ -44,5 +44,10 @@ const struct target_desc *target_read_description_xml (struct target_ops *); otherwise. */ gdb::optional target_fetch_description_xml (target_ops *ops); +/* Take an xml string, parse it, and return the parsed description. Does not + handle a string containing includes. */ + +const struct target_desc *string_read_description_xml (const char *); + #endif /* XML_TDESC_H */