From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 89260 invoked by alias); 1 Jul 2015 13:58:51 -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 89026 invoked by uid 89); 1 Jul 2015 13:58:49 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-pa0-f41.google.com Received: from mail-pa0-f41.google.com (HELO mail-pa0-f41.google.com) (209.85.220.41) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Wed, 01 Jul 2015 13:58:44 +0000 Received: by pactm7 with SMTP id tm7so23601024pac.2 for ; Wed, 01 Jul 2015 06:58:42 -0700 (PDT) X-Received: by 10.68.247.166 with SMTP id yf6mr55044670pbc.3.1435759122606; Wed, 01 Jul 2015 06:58:42 -0700 (PDT) Received: from E107787-LIN.cambridge.arm.com (gcc1-power7.osuosl.org. [140.211.15.137]) by mx.google.com with ESMTPSA id sc7sm2417503pbb.85.2015.07.01.06.58.41 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 01 Jul 2015 06:58:42 -0700 (PDT) From: Yao Qi X-Google-Original-From: Yao Qi To: gdb-patches@sourceware.org Subject: [PATCH 04/11] Set architecture to arm in arm-*.xml files Date: Wed, 01 Jul 2015 13:58:00 -0000 Message-Id: <1435759111-22856-5-git-send-email-yao.qi@linaro.org> In-Reply-To: <1435759111-22856-1-git-send-email-yao.qi@linaro.org> References: <1435759111-22856-1-git-send-email-yao.qi@linaro.org> X-IsSubscribed: yes X-SW-Source: 2015-07/txt/msg00032.txt.bz2 This patch is to add the following line to various arm target description xml files, arm in order to fix problems I've seen on aarch64 multi-arch debugging, detach^M Detaching from program: build-gdb/gdb/testsuite/gdb.base/attach, process 17145^M (gdb) PASS: gdb.base/attach.exp: attach1 detach file^M No executable file now.^M Architecture of file not recognized.^M (gdb) FAIL: gdb.base/attach.exp: attach1, purging symbols after detach Without this patch, struct target_desc *tdesc_* are not initialised properly, that is, fields arch and osabi in 'struct target_desc' are not set properly. This doesn't cause any problems on single arch debugging, because arch-utils.c:gdbarch_info_fill will guess correctly. However, in multi-arch debugging, gdbarch_info_fill gets the aarch64 arch, but the target description is for arm (because the current inferior is 32-bit arm). It is a surprise to me we didn't set architecture to "arm" before in *.xml files, and I didn't find out why didn't do so. AFAICS, gdb/features/arm-with-iwmmxt.xml was added firstly (in patch https://sourceware.org/ml/gdb-patches/2007-01/msg00593.html) which had iwmmxt, however, afterwards, architecture isn't set anymore in features/arm-*.xml files (in patches https://sourceware.org/ml/gdb-patches/2009-07/msg00689.html and https://sourceware.org/ml/gdb-patches/2010-08/msg00225.html). gdb: 2015-06-29 Yao Qi * features/arm-with-m-fpa-layout.xml: Set architecture to arm. * features/arm-with-m-fpa-layout.c: Regenerated. * features/arm-with-m-vfp-d16.xml: Likewise. * features/arm-with-m-vfp-d16.c: Regenerated. * features/arm-with-m.xml: Likewise. * features/arm-with-m.c: Regenerated. * features/arm-with-neon.xml: Likewise. * features/arm-with-neon.c: Regenerated. * features/arm-with-vfpv2.xml: Likewise. * features/arm-with-vfpv2.c: Regenerated. * features/arm-with-vfpv3.xml: Likewise. * features/arm-with-vfpv3.c: Regenerated. --- gdb/features/arm-with-m-fpa-layout.c | 2 ++ gdb/features/arm-with-m-fpa-layout.xml | 1 + gdb/features/arm-with-m-vfp-d16.c | 2 ++ gdb/features/arm-with-m-vfp-d16.xml | 1 + gdb/features/arm-with-m.c | 2 ++ gdb/features/arm-with-m.xml | 1 + gdb/features/arm-with-neon.c | 2 ++ gdb/features/arm-with-neon.xml | 1 + gdb/features/arm-with-vfpv2.c | 2 ++ gdb/features/arm-with-vfpv2.xml | 1 + gdb/features/arm-with-vfpv3.c | 2 ++ gdb/features/arm-with-vfpv3.xml | 1 + 12 files changed, 18 insertions(+) diff --git a/gdb/features/arm-with-m-fpa-layout.c b/gdb/features/arm-with-m-fpa-layout.c index bc51f05..f720614 100644 --- a/gdb/features/arm-with-m-fpa-layout.c +++ b/gdb/features/arm-with-m-fpa-layout.c @@ -12,6 +12,8 @@ initialize_tdesc_arm_with_m_fpa_layout (void) struct target_desc *result = allocate_target_description (); struct tdesc_feature *feature; + set_tdesc_architecture (result, bfd_scan_arch ("arm")); + feature = tdesc_create_feature (result, "org.gnu.gdb.arm.m-profile"); tdesc_create_reg (feature, "r0", 0, 1, NULL, 32, "int"); tdesc_create_reg (feature, "r1", 1, 1, NULL, 32, "int"); diff --git a/gdb/features/arm-with-m-fpa-layout.xml b/gdb/features/arm-with-m-fpa-layout.xml index 7700666..aab1806 100644 --- a/gdb/features/arm-with-m-fpa-layout.xml +++ b/gdb/features/arm-with-m-fpa-layout.xml @@ -10,6 +10,7 @@ +arm diff --git a/gdb/features/arm-with-m-vfp-d16.c b/gdb/features/arm-with-m-vfp-d16.c index a0da583..069baac 100644 --- a/gdb/features/arm-with-m-vfp-d16.c +++ b/gdb/features/arm-with-m-vfp-d16.c @@ -12,6 +12,8 @@ initialize_tdesc_arm_with_m_vfp_d16 (void) struct target_desc *result = allocate_target_description (); struct tdesc_feature *feature; + set_tdesc_architecture (result, bfd_scan_arch ("arm")); + feature = tdesc_create_feature (result, "org.gnu.gdb.arm.m-profile"); tdesc_create_reg (feature, "r0", 0, 1, NULL, 32, "int"); tdesc_create_reg (feature, "r1", 1, 1, NULL, 32, "int"); diff --git a/gdb/features/arm-with-m-vfp-d16.xml b/gdb/features/arm-with-m-vfp-d16.xml index 315d1a8..0586017 100644 --- a/gdb/features/arm-with-m-vfp-d16.xml +++ b/gdb/features/arm-with-m-vfp-d16.xml @@ -7,6 +7,7 @@ + arm diff --git a/gdb/features/arm-with-m.c b/gdb/features/arm-with-m.c index 8e27a80..64d31bb 100644 --- a/gdb/features/arm-with-m.c +++ b/gdb/features/arm-with-m.c @@ -12,6 +12,8 @@ initialize_tdesc_arm_with_m (void) struct target_desc *result = allocate_target_description (); struct tdesc_feature *feature; + set_tdesc_architecture (result, bfd_scan_arch ("arm")); + feature = tdesc_create_feature (result, "org.gnu.gdb.arm.m-profile"); tdesc_create_reg (feature, "r0", 0, 1, NULL, 32, "int"); tdesc_create_reg (feature, "r1", 1, 1, NULL, 32, "int"); diff --git a/gdb/features/arm-with-m.xml b/gdb/features/arm-with-m.xml index 6289472..a7aa2d2 100644 --- a/gdb/features/arm-with-m.xml +++ b/gdb/features/arm-with-m.xml @@ -7,5 +7,6 @@ + arm diff --git a/gdb/features/arm-with-neon.c b/gdb/features/arm-with-neon.c index 61c9a3a..d365c0f 100644 --- a/gdb/features/arm-with-neon.c +++ b/gdb/features/arm-with-neon.c @@ -12,6 +12,8 @@ initialize_tdesc_arm_with_neon (void) struct target_desc *result = allocate_target_description (); struct tdesc_feature *feature; + set_tdesc_architecture (result, bfd_scan_arch ("arm")); + feature = tdesc_create_feature (result, "org.gnu.gdb.arm.core"); tdesc_create_reg (feature, "r0", 0, 1, NULL, 32, "uint32"); tdesc_create_reg (feature, "r1", 1, 1, NULL, 32, "uint32"); diff --git a/gdb/features/arm-with-neon.xml b/gdb/features/arm-with-neon.xml index b537337..777b826 100644 --- a/gdb/features/arm-with-neon.xml +++ b/gdb/features/arm-with-neon.xml @@ -7,6 +7,7 @@ + arm diff --git a/gdb/features/arm-with-vfpv2.c b/gdb/features/arm-with-vfpv2.c index 78e3c81..0ebbfef 100644 --- a/gdb/features/arm-with-vfpv2.c +++ b/gdb/features/arm-with-vfpv2.c @@ -12,6 +12,8 @@ initialize_tdesc_arm_with_vfpv2 (void) struct target_desc *result = allocate_target_description (); struct tdesc_feature *feature; + set_tdesc_architecture (result, bfd_scan_arch ("arm")); + feature = tdesc_create_feature (result, "org.gnu.gdb.arm.core"); tdesc_create_reg (feature, "r0", 0, 1, NULL, 32, "uint32"); tdesc_create_reg (feature, "r1", 1, 1, NULL, 32, "uint32"); diff --git a/gdb/features/arm-with-vfpv2.xml b/gdb/features/arm-with-vfpv2.xml index 6008124..fe7941a 100644 --- a/gdb/features/arm-with-vfpv2.xml +++ b/gdb/features/arm-with-vfpv2.xml @@ -7,6 +7,7 @@ + arm diff --git a/gdb/features/arm-with-vfpv3.c b/gdb/features/arm-with-vfpv3.c index e48ca79..e235dfa 100644 --- a/gdb/features/arm-with-vfpv3.c +++ b/gdb/features/arm-with-vfpv3.c @@ -12,6 +12,8 @@ initialize_tdesc_arm_with_vfpv3 (void) struct target_desc *result = allocate_target_description (); struct tdesc_feature *feature; + set_tdesc_architecture (result, bfd_scan_arch ("arm")); + feature = tdesc_create_feature (result, "org.gnu.gdb.arm.core"); tdesc_create_reg (feature, "r0", 0, 1, NULL, 32, "uint32"); tdesc_create_reg (feature, "r1", 1, 1, NULL, 32, "uint32"); diff --git a/gdb/features/arm-with-vfpv3.xml b/gdb/features/arm-with-vfpv3.xml index e4eeb6a..66d98ec 100644 --- a/gdb/features/arm-with-vfpv3.xml +++ b/gdb/features/arm-with-vfpv3.xml @@ -7,6 +7,7 @@ + arm -- 1.9.1