From patchwork Mon Aug 21 15:28:56 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yao Qi X-Patchwork-Id: 22273 Received: (qmail 128594 invoked by alias); 21 Aug 2017 15:29:37 -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 128313 invoked by uid 89); 21 Aug 2017 15:29:36 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-26.4 required=5.0 tests=BAYES_00, FREEMAIL_FROM, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, RCVD_IN_SORBS_SPAM, SPF_PASS autolearn=ham version=3.3.2 spammy=FEATURE X-HELO: mail-it0-f68.google.com Received: from mail-it0-f68.google.com (HELO mail-it0-f68.google.com) (209.85.214.68) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 21 Aug 2017 15:29:30 +0000 Received: by mail-it0-f68.google.com with SMTP id m81so4982178itb.0 for ; Mon, 21 Aug 2017 08:29:30 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:in-reply-to :references; bh=T2KAfJdFbnNa0y8edXZObwoK2b+sCjLLXtJGTK6k3Fc=; b=UaLn5c22yVheLNvciMhN+Y2f8dihOqHHivDjrzW18zVkMm0ZVXv+GahP89txYHkHW/ aptBQKHezE58zS2ij39WIkyXmQVDghiOrsY4+/sVulL3LaeZFuZfOdDKCv3EltM8okEm 5PahV9rupIUjCKIRhEQ5eIrwjcxEGcG6GY7nns+L+fNr3hhN39vAMJwIfXHpgExklvDn ze62+1BcQXsSpJVDcdCmkoJBlVtsXA3hadOPhqiMFmeCGST1y5EKi+O41LfVRuF8EL50 DPoR7KKSqw4PXm500hE3YaDPA3CaFGViKRrA6izL0AmXHGYS8HtYAQdoX56MxTIFrqjq HJcA== X-Gm-Message-State: AHYfb5htDtO6P/IjJ6fFa8VsS6ZDKJ4ccVuqqxBMUzDjrUDQSSB7DkHy MruT0+ceaX5IsT+X X-Received: by 10.36.213.131 with SMTP id a125mr7261951itg.74.1503329368750; Mon, 21 Aug 2017 08:29:28 -0700 (PDT) Received: from E107787-LIN.cambridge.arm.com (static.42.136.251.148.clients.your-server.de. [148.251.136.42]) by smtp.gmail.com with ESMTPSA id p62sm5831589ioe.34.2017.08.21.08.29.27 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 21 Aug 2017 08:29:28 -0700 (PDT) From: Yao Qi X-Google-Original-From: Yao Qi To: gdb-patches@sourceware.org Subject: [PATCH 11/22] Share i386-linux target description between GDB and GDBserver Date: Mon, 21 Aug 2017 16:28:56 +0100 Message-Id: <1503329347-26711-12-git-send-email-yao.qi@linaro.org> In-Reply-To: <1503329347-26711-1-git-send-email-yao.qi@linaro.org> References: <1503329347-26711-1-git-send-email-yao.qi@linaro.org> X-IsSubscribed: yes The code on creating i386-linux target descriptions are quite similar between GDB and GDBserver, so this patch moves them into a shared file arch/i386.c. I didn't name it as i386-linux.c, because I want to reuse it to create other i386 non-linux target descriptions later. gdb: 2017-07-05 Yao Qi * Makefile.in (ALL_TARGET_OBS): Add i386.o. (SFILES): Add arch/i386.c. (HFILES_NO_SRCDIR): Add arch/i386.h. * arch/i386.c: New file. * arch/i386.h: New file. * arch/tdesc.h (allocate_target_description): Declare. (set_tdesc_architecture): Declare. (set_tdesc_osabi): Declare. * configure.tgt (i[34567]86-*-linux*): Add i386.o. * i386-linux-tdep.c: Don't include ../features/i386/32bit-XXX.c. include arch/i386.h. (i386_linux_read_description): Remove code and call i386_create_target_description. (set_tdesc_architecture): New function. (set_tdesc_osabi): New function. * target-descriptions.h (allocate_target_description): Remove. gdb/gdbserver: 2017-07-05 Yao Qi * Makefile.in (arch-i386.o): New rule. * configure.srv (i[34567]86-*-linux*): Add arch-i386.o. (x86_64-*-linux*): Likewise. * linux-x86-tdesc.c: Don't include ../features/i386/32bit-XXX.c, include arch/i386.h. (i386_linux_read_description): Remove code and call i386_create_target_description. * tdesc.c (allocate_target_description): New function. * tdesc.h (set_tdesc_architecture): Remove declaration. (set_tdesc_osabi): Likewise. --- gdb/Makefile.in | 3 ++ gdb/arch/i386.c | 66 +++++++++++++++++++++++++++++++++++++++++ gdb/arch/i386.h | 21 +++++++++++++ gdb/arch/tdesc.h | 10 +++++++ gdb/configure.tgt | 5 ++-- gdb/gdbserver/Makefile.in | 8 +++++ gdb/gdbserver/configure.srv | 3 ++ gdb/gdbserver/linux-x86-tdesc.c | 39 ++---------------------- gdb/gdbserver/tdesc.c | 10 +++++++ gdb/gdbserver/tdesc.h | 4 --- gdb/i386-linux-tdep.c | 36 ++-------------------- gdb/target-descriptions.c | 17 +++++++++++ gdb/target-descriptions.h | 1 - 13 files changed, 146 insertions(+), 77 deletions(-) create mode 100644 gdb/arch/i386.c create mode 100644 gdb/arch/i386.h diff --git a/gdb/Makefile.in b/gdb/Makefile.in index 85de646..40ae421 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -812,6 +812,7 @@ ALL_TARGET_OBS = \ hppa-nbsd-tdep.o \ hppa-obsd-tdep.o \ hppa-tdep.o \ + i386.o \ i386-bsd-tdep.o \ i386-cygwin-tdep.o \ i386-darwin-tdep.o \ @@ -1030,6 +1031,7 @@ SFILES = \ agent.c \ annotate.c \ arch-utils.c \ + arch/i386.c \ auto-load.c \ auxv.c \ ax-gdb.c \ @@ -1500,6 +1502,7 @@ HFILES_NO_SRCDIR = \ xtensa-tdep.h \ arch/aarch64-insn.h \ arch/arm.h \ + arch/i386.h \ cli/cli-cmds.h \ cli/cli-decode.h \ cli/cli-script.h \ diff --git a/gdb/arch/i386.c b/gdb/arch/i386.c new file mode 100644 index 0000000..60cf63b --- /dev/null +++ b/gdb/arch/i386.c @@ -0,0 +1,66 @@ +/* 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 . */ + +#include "i386.h" +#include "tdesc.h" +#include "x86-xstate.h" +#include + +#include "../features/i386/32bit-core.c" +#include "../features/i386/32bit-linux.c" +#include "../features/i386/32bit-sse.c" +#include "../features/i386/32bit-avx.c" +#include "../features/i386/32bit-avx512.c" +#include "../features/i386/32bit-mpx.c" +#include "../features/i386/32bit-pkeys.c" + +/* Create i386 target descriptions according to XCR0. */ + +target_desc * +i386_create_target_description (uint64_t xcr0) +{ + target_desc *tdesc = allocate_target_description (); + +#ifndef IN_PROCESS_AGENT + set_tdesc_architecture (tdesc, "i386"); + set_tdesc_osabi (tdesc, "GNU/Linux"); +#endif + + long regnum = 0; + + if (xcr0 & X86_XSTATE_X87) + regnum = create_feature_i386_32bit_core (tdesc, regnum); + + if (xcr0 & X86_XSTATE_SSE) + regnum = create_feature_i386_32bit_sse (tdesc, regnum); + + regnum = create_feature_i386_32bit_linux (tdesc, regnum); + + if (xcr0 & X86_XSTATE_AVX) + regnum = create_feature_i386_32bit_avx (tdesc, regnum); + + if (xcr0 & X86_XSTATE_MPX) + regnum = create_feature_i386_32bit_mpx (tdesc, regnum); + + if (xcr0 & X86_XSTATE_AVX512) + regnum = create_feature_i386_32bit_avx512 (tdesc, regnum); + + if (xcr0 & X86_XSTATE_PKRU) + regnum = create_feature_i386_32bit_pkeys (tdesc, regnum); + + return tdesc; +} diff --git a/gdb/arch/i386.h b/gdb/arch/i386.h new file mode 100644 index 0000000..0163372 --- /dev/null +++ b/gdb/arch/i386.h @@ -0,0 +1,21 @@ +/* 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 . */ + +#include "tdesc.h" +#include + +target_desc *i386_create_target_description (uint64_t xcr0); diff --git a/gdb/arch/tdesc.h b/gdb/arch/tdesc.h index bf50e24..78bb0fb 100644 --- a/gdb/arch/tdesc.h +++ b/gdb/arch/tdesc.h @@ -23,6 +23,16 @@ struct tdesc_type; struct tdesc_reg; struct target_desc; +/* Allocate a new target_desc. */ +target_desc *allocate_target_description (void); + +/* Set TARGET_DESC's architecture by NAME. */ +void set_tdesc_architecture (target_desc *target_desc, + const char *name); + +/* Set TARGET_DESC's osabi by NAME. */ +void set_tdesc_osabi (target_desc *target_desc, const char *name); + /* Return the type associated with ID in the context of FEATURE, or NULL if none. */ struct tdesc_type *tdesc_named_type (const struct tdesc_feature *feature, diff --git a/gdb/configure.tgt b/gdb/configure.tgt index fdcb7b1..1f263515 100644 --- a/gdb/configure.tgt +++ b/gdb/configure.tgt @@ -228,7 +228,8 @@ i[34567]86-*-solaris*) ;; i[34567]86-*-linux*) # Target: Intel 386 running GNU/Linux - gdb_target_obs="i386-tdep.o i386-linux-tdep.o glibc-tdep.o i387-tdep.o \ + gdb_target_obs="i386-tdep.o i386.o i386-linux-tdep.o \ + glibc-tdep.o i387-tdep.o \ solib-svr4.o symfile-mem.o \ linux-tdep.o linux-record.o" if test "x$enable_64_bit_bfd" = "xyes"; then @@ -678,7 +679,7 @@ x86_64-*-elf*) x86_64-*-linux*) # Target: GNU/Linux x86-64 gdb_target_obs="amd64-tdep.o amd64-linux-tdep.o i386-tdep.o \ - i387-tdep.o i386-linux-tdep.o glibc-tdep.o \ + i387-tdep.o i386.o i386-linux-tdep.o glibc-tdep.o \ solib-svr4.o symfile-mem.o linux-tdep.o linux-record.o" build_gdbserver=yes ;; diff --git a/gdb/gdbserver/Makefile.in b/gdb/gdbserver/Makefile.in index 9b5982b..5f9b2fe 100644 --- a/gdb/gdbserver/Makefile.in +++ b/gdb/gdbserver/Makefile.in @@ -528,6 +528,10 @@ ax.o: ax.c $(COMPILE) $(WARN_CFLAGS_NO_FORMAT) $< $(POSTCOMPILE) +arch-i386.o: ../arch/i386.c + $(COMPILE) $< + $(POSTCOMPILE) + # Rules for objects that go in the in-process agent. %-ipa.o: %-generated.c @@ -553,6 +557,10 @@ ax.o: ax.c $(IPAGENT_COMPILE) $< $(POSTCOMPILE) +%-ipa.o: ../arch/%.c + $(IPAGENT_COMPILE) $< + $(POSTCOMPILE) + # Rules for objects that go in the gdbserver binary. %.o: %-generated.c diff --git a/gdb/gdbserver/configure.srv b/gdb/gdbserver/configure.srv index 1a27012..876098b 100644 --- a/gdb/gdbserver/configure.srv +++ b/gdb/gdbserver/configure.srv @@ -122,6 +122,7 @@ case "${target}" in srv_xmlfiles="${srv_xmlfiles} $srv_amd64_linux_xmlfiles" srv_tgtobj="amd64-linux-siginfo.o" fi + srv_tgtobj="${srv_tgtobj} arch-i386.o" srv_tgtobj="${srv_tgtobj} $srv_linux_obj linux-x86-low.o x86-low.o x86-dregs.o i387-fp.o" srv_tgtobj="${srv_tgtobj} linux-x86-tdesc.o" srv_tgtobj="${srv_tgtobj} linux-btrace.o x86-linux.o" @@ -131,6 +132,7 @@ case "${target}" in srv_linux_thread_db=yes srv_linux_btrace=yes ipa_obj="${ipa_i386_linux_regobj} linux-i386-ipa.o linux-x86-tdesc-ipa.o" + ipa_obj="${ipa_obj} i386-ipa.o" ;; i[34567]86-*-lynxos*) srv_regobj="i386.o" srv_tgtobj="lynx-low.o lynx-i386-low.o fork-child.o fork-inferior.o" @@ -358,6 +360,7 @@ case "${target}" in ;; x86_64-*-linux*) srv_regobj="$srv_amd64_linux_regobj $srv_i386_linux_regobj" srv_tgtobj="$srv_linux_obj linux-x86-low.o x86-low.o x86-dregs.o i387-fp.o" + srv_tgtobj="${srv_tgtobj} arch-i386.o" srv_tgtobj="${srv_tgtobj} linux-x86-tdesc.o" srv_tgtobj="${srv_tgtobj} linux-btrace.o x86-linux.o" srv_tgtobj="${srv_tgtobj} x86-linux-dregs.o" diff --git a/gdb/gdbserver/linux-x86-tdesc.c b/gdb/gdbserver/linux-x86-tdesc.c index 3f63d8e..548e780 100644 --- a/gdb/gdbserver/linux-x86-tdesc.c +++ b/gdb/gdbserver/linux-x86-tdesc.c @@ -20,16 +20,10 @@ #include "server.h" #include "tdesc.h" #include "linux-x86-tdesc.h" -#include "x86-xstate.h" +#include "arch/i386.h" +#include "common/x86-xstate.h" #if defined __i386__ || !defined IN_PROCESS_AGENT -#include "../features/i386/32bit-core.c" -#include "../features/i386/32bit-linux.c" -#include "../features/i386/32bit-sse.c" -#include "../features/i386/32bit-avx.c" -#include "../features/i386/32bit-avx512.c" -#include "../features/i386/32bit-mpx.c" -#include "../features/i386/32bit-pkeys.c" /* Defined in auto-generated file i386-linux.c. */ void init_registers_i386_linux (void); @@ -142,34 +136,7 @@ i386_linux_read_description (uint64_t xcr0) if (*tdesc == NULL) { - *tdesc = new target_desc (); - -#ifndef IN_PROCESS_AGENT - set_tdesc_architecture (*tdesc, "i386"); - set_tdesc_osabi (*tdesc, "GNU/Linux"); -#endif - - long regnum = 0; - - if (xcr0 & X86_XSTATE_X87) - regnum = create_feature_i386_32bit_core (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_SSE) - regnum = create_feature_i386_32bit_sse (*tdesc, regnum); - - regnum = create_feature_i386_32bit_linux (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_AVX) - regnum = create_feature_i386_32bit_avx (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_MPX) - regnum = create_feature_i386_32bit_mpx (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_AVX512) - regnum = create_feature_i386_32bit_avx512 (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_PKRU) - regnum = create_feature_i386_32bit_pkeys (*tdesc, regnum); + *tdesc = i386_create_target_description (xcr0); init_target_desc (*tdesc); diff --git a/gdb/gdbserver/tdesc.c b/gdb/gdbserver/tdesc.c index 0b5096b..53f36d5 100644 --- a/gdb/gdbserver/tdesc.c +++ b/gdb/gdbserver/tdesc.c @@ -39,6 +39,12 @@ init_target_desc (struct target_desc *tdesc) gdb_assert (2 * tdesc->registers_size + 32 <= PBUFSIZ); } +struct target_desc * +allocate_target_description (void) +{ + return new target_desc (); +} + #ifndef IN_PROCESS_AGENT static const struct target_desc default_description {}; @@ -62,6 +68,8 @@ current_target_desc (void) return current_process ()->tdesc; } +/* See arch/tdesc.h. */ + void set_tdesc_architecture (struct target_desc *target_desc, const char *name) @@ -69,6 +77,8 @@ set_tdesc_architecture (struct target_desc *target_desc, target_desc->arch = xstrdup (name); } +/* See arch/tdesc.h. */ + void set_tdesc_osabi (struct target_desc *target_desc, const char *name) { diff --git a/gdb/gdbserver/tdesc.h b/gdb/gdbserver/tdesc.h index fe3c78f..71249e4 100644 --- a/gdb/gdbserver/tdesc.h +++ b/gdb/gdbserver/tdesc.h @@ -142,10 +142,6 @@ void init_target_desc (struct target_desc *tdesc); const struct target_desc *current_target_desc (void); #ifndef IN_PROCESS_AGENT -void set_tdesc_architecture (struct target_desc *target_desc, - const char *name); -void set_tdesc_osabi (struct target_desc *target_desc, const char *name); - const char *tdesc_get_features_xml (struct target_desc *tdesc); #endif diff --git a/gdb/i386-linux-tdep.c b/gdb/i386-linux-tdep.c index 792706c..c47aa6e 100644 --- a/gdb/i386-linux-tdep.c +++ b/gdb/i386-linux-tdep.c @@ -46,13 +46,7 @@ #include "record-full.h" #include "linux-record.h" -#include "features/i386/32bit-core.c" -#include "features/i386/32bit-sse.c" -#include "features/i386/32bit-linux.c" -#include "features/i386/32bit-avx.c" -#include "features/i386/32bit-mpx.c" -#include "features/i386/32bit-avx512.c" -#include "features/i386/32bit-pkeys.c" +#include "arch/i386.h" #include "target-descriptions.h" /* Return non-zero, when the register is in the corresponding register @@ -700,33 +694,7 @@ i386_linux_read_description (uint64_t xcr0) [(xcr0 & X86_XSTATE_PKRU) ? 1 : 0]; if (*tdesc == NULL) - { - *tdesc = allocate_target_description (); - set_tdesc_architecture (*tdesc, bfd_scan_arch ("i386")); - set_tdesc_osabi (*tdesc, osabi_from_tdesc_string ("GNU/Linux")); - - long regnum = 0; - - if (xcr0 & X86_XSTATE_X87) - regnum = create_feature_i386_32bit_core (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_SSE) - regnum = create_feature_i386_32bit_sse (*tdesc, regnum); - - regnum = create_feature_i386_32bit_linux (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_AVX) - regnum = create_feature_i386_32bit_avx (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_MPX) - regnum = create_feature_i386_32bit_mpx (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_AVX512) - regnum = create_feature_i386_32bit_avx512 (*tdesc, regnum); - - if (xcr0 & X86_XSTATE_PKRU) - regnum = create_feature_i386_32bit_pkeys (*tdesc, regnum); - } + *tdesc = i386_create_target_description (xcr0); return *tdesc; } diff --git a/gdb/target-descriptions.c b/gdb/target-descriptions.c index 6135e0d..7a8dac0 100644 --- a/gdb/target-descriptions.c +++ b/gdb/target-descriptions.c @@ -1823,6 +1823,15 @@ set_tdesc_property (struct target_desc *target_desc, VEC_safe_push (property_s, target_desc->properties, &new_prop); } +/* See arch/tdesc.h. */ + +void +set_tdesc_architecture (struct target_desc *target_desc, + const char *name) +{ + set_tdesc_architecture (target_desc, bfd_scan_arch (name)); +} + void set_tdesc_architecture (struct target_desc *target_desc, const struct bfd_arch_info *arch) @@ -1830,6 +1839,14 @@ set_tdesc_architecture (struct target_desc *target_desc, target_desc->arch = arch; } +/* See arch/tdesc.h. */ + +void +set_tdesc_osabi (struct target_desc *target_desc, const char *name) +{ + set_tdesc_osabi (target_desc, osabi_from_tdesc_string (name)); +} + void set_tdesc_osabi (struct target_desc *target_desc, enum gdb_osabi osabi) { diff --git a/gdb/target-descriptions.h b/gdb/target-descriptions.h index c4f9b92..e97a373 100644 --- a/gdb/target-descriptions.h +++ b/gdb/target-descriptions.h @@ -201,7 +201,6 @@ int tdesc_register_in_reggroup_p (struct gdbarch *gdbarch, int regno, /* Methods for constructing a target description. */ -struct target_desc *allocate_target_description (void); struct cleanup *make_cleanup_free_target_description (struct target_desc *); void set_tdesc_architecture (struct target_desc *, const struct bfd_arch_info *);