From c912f608be7bc2598b919da2b11721b3c262d154 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Sat, 7 Apr 2018 13:24:58 -0400 Subject: [PATCH] Fix generation of x86-64 gdbarch with osabi none (PR 22979) When a 64-bits (x86-64) gdbarch is created, it is first born as a 32-bits gdbarch in i386_gdbarch_init. The call gdbarch_init_osabi will call the handler register for the selected (arch, osabi) pair, such as amd64_linux_init_abi. The various amd64 handlers call amd64_init_abi, which turns the gdbarch into a 64-bits one. When selecting the i386:x86-64 architecture with no osabi, no such handler is ever called, so the gdbarch stays (wrongfully) a 32-bits one. My first idea was to manually call amd64_init_abi & al in i386_gdbarch_init when the osabi is GDB_OSABI_NONE. However, this doesn't work in a build of GDB where i386 is included as a target but not amd64. My next option (implemented in this patch), is to allow registering handlers for GDB_OSABI_NONE. I added two such handlers in amd64-tdep.c, so now it works the same as for the "normal" osabis. It required re-ordering things in gdbarch_init_osabi to allow running handlers for GDB_OSABI_NONE. Without this patch applied (but with the previous one*) : (gdb) set osabi none (gdb) set architecture i386:x86-64 The target architecture is assumed to be i386:x86-64 (gdb) p sizeof(void*) $1 = 4 and now: (gdb) set osabi none (gdb) set architecture i386:x86-64 The target architecture is assumed to be i386:x86-64 (gdb) p sizeof(void*) $1 = 8 * Before the previous patch, which fixed "set osabi none", this bug was hidden because we didn't actually try to generate a gdbarch for no osabi, it would always fall back on Linux. Generating the gdbarch for amd64/linux did work. gdb/ChangeLog: PR gdb/22979 * amd64-tdep.c (amd64_none_init_abi): New function. (amd64_x32_none_init_abi): New function. (_initialize_amd64_tdep): Register handlers for x86-64 and x64_32 with GDB_OSABI_NONE. * osabi.c (gdbarch_init_osabi): Allow running handlers for the GDB_OSABI_NONE osabi. gdb/testsuite/ChangeLog: PR gdb/22979 * gdb.arch/amd64-osabi.exp: New file. --- gdb/ChangeLog | 10 ++++++ gdb/amd64-tdep.c | 24 +++++++++++++- gdb/osabi.c | 14 ++++----- gdb/testsuite/ChangeLog | 5 +++ gdb/testsuite/gdb.arch/amd64-osabi.exp | 43 ++++++++++++++++++++++++++ 5 files changed, 88 insertions(+), 8 deletions(-) create mode 100644 gdb/testsuite/gdb.arch/amd64-osabi.exp diff --git a/gdb/ChangeLog b/gdb/ChangeLog index a5773e8f2f..59ca3d711a 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,13 @@ +2018-04-07 Simon Marchi + + PR gdb/22979 + * amd64-tdep.c (amd64_none_init_abi): New function. + (amd64_x32_none_init_abi): New function. + (_initialize_amd64_tdep): Register handlers for x86-64 and + x64_32 with GDB_OSABI_NONE. + * osabi.c (gdbarch_init_osabi): Allow running handlers for the + GDB_OSABI_NONE osabi. + 2018-04-07 Simon Marchi PR gdb/22980 diff --git a/gdb/amd64-tdep.c b/gdb/amd64-tdep.c index 07eef5ea9f..bceb6e1427 100644 --- a/gdb/amd64-tdep.c +++ b/gdb/amd64-tdep.c @@ -47,6 +47,7 @@ #include "ax.h" #include "ax-gdb.h" #include "common/byte-vector.h" +#include "osabi.h" /* Note that the AMD64 architecture was previously known as x86-64. The latter is (forever) engraved into the canonical system name as @@ -3206,7 +3207,14 @@ amd64_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch, set_gdbarch_insn_is_ret (gdbarch, amd64_insn_is_ret); set_gdbarch_insn_is_jump (gdbarch, amd64_insn_is_jump); } - + +/* Initialize ARCH for x86-64, no osabi. */ + +static void +amd64_none_init_abi (gdbarch_info info, gdbarch *arch) +{ + amd64_init_abi (info, arch, amd64_target_description (X86_XSTATE_SSE_MASK)); +} static struct type * amd64_x32_pseudo_register_type (struct gdbarch *gdbarch, int regnum) @@ -3240,6 +3248,15 @@ amd64_x32_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch, set_gdbarch_ptr_bit (gdbarch, 32); } +/* Initialize ARCH for x64-32, no osabi. */ + +static void +amd64_x32_none_init_abi (gdbarch_info info, gdbarch *arch) +{ + amd64_x32_init_abi (info, arch, + amd64_target_description (X86_XSTATE_SSE_MASK)); +} + /* Return the target description for a specified XSAVE feature mask. */ const struct target_desc * @@ -3263,6 +3280,11 @@ amd64_target_description (uint64_t xcr0) void _initialize_amd64_tdep (void) { + gdbarch_register_osabi (bfd_arch_i386, bfd_mach_x86_64, GDB_OSABI_NONE, + amd64_none_init_abi); + gdbarch_register_osabi (bfd_arch_i386, bfd_mach_x64_32, GDB_OSABI_NONE, + amd64_x32_none_init_abi); + #if GDB_SELF_TEST struct { diff --git a/gdb/osabi.c b/gdb/osabi.c index ef1d9938c0..7d0540b181 100644 --- a/gdb/osabi.c +++ b/gdb/osabi.c @@ -338,13 +338,6 @@ gdbarch_init_osabi (struct gdbarch_info info, struct gdbarch *gdbarch) gdb_assert (info.osabi != GDB_OSABI_UNKNOWN); - if (info.osabi == GDB_OSABI_NONE) - { - /* Don't complain about no OSABI. Assume the user knows - what they are doing. */ - return; - } - for (handler = gdb_osabi_handler_list; handler != NULL; handler = handler->next) { @@ -378,6 +371,13 @@ gdbarch_init_osabi (struct gdbarch_info info, struct gdbarch *gdbarch) } } + if (info.osabi == GDB_OSABI_NONE) + { + /* Don't complain about no OSABI. Assume the user knows + what they are doing. */ + return; + } + warning ("A handler for the OS ABI \"%s\" is not built into this configuration\n" "of GDB. Attempting to continue with the default %s settings.\n", diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 5a885bf699..34e7610a09 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2018-04-07 Simon Marchi + + PR gdb/22979 + * gdb.arch/amd64-osabi.exp: New file. + 2018-04-07 Simon Marchi PR gdb/22980 diff --git a/gdb/testsuite/gdb.arch/amd64-osabi.exp b/gdb/testsuite/gdb.arch/amd64-osabi.exp new file mode 100644 index 0000000000..4f764d336e --- /dev/null +++ b/gdb/testsuite/gdb.arch/amd64-osabi.exp @@ -0,0 +1,43 @@ +# Copyright 2018 Free Software Foundation, Inc. + +# 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 . + +# Verify that gdbarches for i386 variants and osabi none are properly created. + +if { ![istarget x86_64-*-* ] } { + untested "skipping x86-64 specific test" + return +} + + +proc test_osabi_none { arch void_ptr_size long_double_size } { + clean_restart + + gdb_test "set architecture i386:x86-64" "The target architecture is assumed to be i386:x86-64" + gdb_test_no_output "set osabi none" "set osabi none" + gdb_test "print sizeof (void*)" " = 8" + gdb_test "print sizeof (long double)" " = 16" +} + +set infos { \ + { "i386:x86-64" 8 16 } \ + { "i386:x64-32" 4 16 } \ + { "i386" 4 12 } } + +foreach info $infos { + lassign $info arch void_ptr_size long_double_size + with_test_prefix "arch=$arch" { + test_osabi_none $arch $void_ptr_size $long_double_size + } +} -- 2.34.1