aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYao Qi <yao.qi@linaro.org>2015-07-07 16:58:19 +0100
committerYao Qi <yao.qi@linaro.org>2015-07-07 16:58:19 +0100
commit82c40d4b1deb252f2d17ff74f3fae6ae8cdd0383 (patch)
tree35a027ba1b343d5ff5c1a474e0c6763ba8cece3f
parent607685ecee1015d6c37e0d800d40453dc0aadc8c (diff)
downloadgdb-82c40d4b1deb252f2d17ff74f3fae6ae8cdd0383.zip
gdb-82c40d4b1deb252f2d17ff74f3fae6ae8cdd0383.tar.gz
gdb-82c40d4b1deb252f2d17ff74f3fae6ae8cdd0383.tar.bz2
Set architecture to arm in arm-*.xml files
This patch is to add the following line to various arm target description xml files, <architecture>arm</architecture> 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 <architecture>iwmmxt</architecture>, 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-07-07 Yao Qi <yao.qi@linaro.org> * 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.
-rw-r--r--gdb/ChangeLog15
-rw-r--r--gdb/features/arm-with-m-fpa-layout.c2
-rw-r--r--gdb/features/arm-with-m-fpa-layout.xml1
-rw-r--r--gdb/features/arm-with-m-vfp-d16.c2
-rw-r--r--gdb/features/arm-with-m-vfp-d16.xml1
-rw-r--r--gdb/features/arm-with-m.c2
-rw-r--r--gdb/features/arm-with-m.xml1
-rw-r--r--gdb/features/arm-with-neon.c2
-rw-r--r--gdb/features/arm-with-neon.xml1
-rw-r--r--gdb/features/arm-with-vfpv2.c2
-rw-r--r--gdb/features/arm-with-vfpv2.xml1
-rw-r--r--gdb/features/arm-with-vfpv3.c2
-rw-r--r--gdb/features/arm-with-vfpv3.xml1
13 files changed, 33 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index c1f3994..680f96c 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,20 @@
2015-07-07 Yao Qi <yao.qi@linaro.org>
+ * 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.
+
+2015-07-07 Yao Qi <yao.qi@linaro.org>
+
* aarch32-linux-nat.h (VFP_REGS_SIZE): New macro, moved from
arm-linux-nat.c.
* aarch64-linux-nat.c: Include aarch32-linux-nat.h and
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 @@
<!DOCTYPE target SYSTEM "gdb-target.dtd">
<target>
+<architecture>arm</architecture>
<feature name="org.gnu.gdb.arm.m-profile">
<reg name="r0" bitsize="32"/>
<reg name="r1" bitsize="32"/>
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 @@
<!DOCTYPE target SYSTEM "gdb-target.dtd">
<target>
+ <architecture>arm</architecture>
<xi:include href="arm-m-profile.xml"/>
<!-- Layout of vfpv4-sp-d16 is identical to vfpv2 -->
<xi:include href="arm-vfpv2.xml"/>
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 @@
<!DOCTYPE target SYSTEM "gdb-target.dtd">
<target>
+ <architecture>arm</architecture>
<xi:include href="arm-m-profile.xml"/>
</target>
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 @@
<!DOCTYPE target SYSTEM "gdb-target.dtd">
<target>
+ <architecture>arm</architecture>
<xi:include href="arm-core.xml"/>
<xi:include href="arm-vfpv3.xml"/>
<feature name="org.gnu.gdb.arm.neon"/>
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 @@
<!DOCTYPE target SYSTEM "gdb-target.dtd">
<target>
+ <architecture>arm</architecture>
<xi:include href="arm-core.xml"/>
<xi:include href="arm-vfpv2.xml"/>
</target>
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 @@
<!DOCTYPE target SYSTEM "gdb-target.dtd">
<target>
+ <architecture>arm</architecture>
<xi:include href="arm-core.xml"/>
<xi:include href="arm-vfpv3.xml"/>
</target>