From patchwork Wed Oct 11 14:56:32 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anton Kolesov X-Patchwork-Id: 23479 Received: (qmail 108626 invoked by alias); 11 Oct 2017 14:56:45 -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 108608 invoked by uid 89); 11 Oct 2017 14:56:44 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-23.8 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_LAZY_DOMAIN_SECURITY, RCVD_IN_DNSWL_NONE, RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=UD:DE, BTA, bta X-HELO: smtprelay.synopsys.com Received: from us01smtprelay-2.synopsys.com (HELO smtprelay.synopsys.com) (198.182.60.111) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 11 Oct 2017 14:56:42 +0000 Received: from mailhost.synopsys.com (mailhost2.synopsys.com [10.13.184.66]) by smtprelay.synopsys.com (Postfix) with ESMTP id 38D0910C133B for ; Wed, 11 Oct 2017 07:56:41 -0700 (PDT) Received: from mailhost.synopsys.com (localhost [127.0.0.1]) by mailhost.synopsys.com (Postfix) with ESMTP id 204E2147; Wed, 11 Oct 2017 07:56:41 -0700 (PDT) Received: from akolesov-lab.internal.synopsys.com (akolesov-lab.internal.synopsys.com [10.121.14.106]) by mailhost.synopsys.com (Postfix) with ESMTP id 2B10A138; Wed, 11 Oct 2017 07:56:39 -0700 (PDT) From: Anton Kolesov To: gdb-patches@sourceware.org Cc: Anton Kolesov , Francois Bedard Subject: [PATCH 2/3] arc: Recognize registers available on Linux targets Date: Wed, 11 Oct 2017 17:56:32 +0300 Message-Id: <20171011145633.19343-2-Anton.Kolesov@synopsys.com> In-Reply-To: <20171011145633.19343-1-Anton.Kolesov@synopsys.com> References: <20171011145633.19343-1-Anton.Kolesov@synopsys.com> For ARC there are registers that are not part of a required set in XML target descriptions by default, but which are almost always present on ARC targets and are universally exposed by the ptrace interface. This patch adds those registers to those recognized by GDB: - R30 - core register available in user-space on ARC HS processors only (used to be a privileged only ILINK2 on ARC700). - R58, R59 - accumulator register pair for a 64-multiplier and double-precision FPU - only on ARC HS. - LP_START, LP_END - AUX registers, hardware loop start and end. Formally optional, though it is hard to find ARC configuration that doesn't have them and is always present in processors that can run Linux. GDB needs to know about those registers to implement proper software single stepping, since they affect what instruction will be next. - BTA - AUX register that contains branch target address. Value of this register makes sense only when execution halts at the delay slot instruction - in this case branch instruction is already committed, STATUS32.DE is set to 1 and BTA contains address of next PC. GDB needs to understand this register to properly handle situations when breakpoint has been set in the delay slot (delay slot is stepped over when doing software single stepping). Unfortunately, right now this doesn't work very well, because Linux doesn't allow modifications of STATUS32 via ptrace and Linux uses TRAP_S instruction to implement software breakpoints - this instruction commits when executed, therefore when TRAP_S is set in the delay slot and execution halts at it, PC is already advanced to BTA value and STATUS32.DE is reset. BTA register will be more useful for debugger after support for SWI instruction will be added to ARC Linux - this breakpoint instruction doesn't commit, hence it doesn't change processor state. gdb/ChangeLog: yyyy-mm-dd Anton Kolesov * arc-tdep.c (core_v2_register_names): Fix names of R58 and R59. (aux_minimal_register_names): Add LP_START, LP_END and BTA. (arc_tdesc_init): Recognize those registers. * arc-tdep.h (arc_regnum): Add R58, R59, LP_START, LP_END and BTA. (gdbarch_tdep): New field has_hw_loops. gdb/doc/ChangeLog: yyyy-mm-dd Anton Kolesov * gdb.texinfo (Synopsys ARC): Document LP_START, LP_END and BTA. --- gdb/arc-tdep.c | 61 ++++++++++++++++++++++++++++++++++++++++++++++------- gdb/arc-tdep.h | 15 +++++++++++-- gdb/doc/gdb.texinfo | 3 ++- 3 files changed, 68 insertions(+), 11 deletions(-) diff --git a/gdb/arc-tdep.c b/gdb/arc-tdep.c index a825917..1643912 100644 --- a/gdb/arc-tdep.c +++ b/gdb/arc-tdep.c @@ -120,12 +120,12 @@ static const char *const core_v2_register_names[] = { "r44", "r45", "r46", "r47", "r48", "r49", "r50", "r51", "r52", "r53", "r54", "r55", - "r56", "r57", "accl", "acch", + "r56", "r57", "r58", "r59", "lp_count", "reserved", "limm", "pcl", }; static const char *const aux_minimal_register_names[] = { - "pc", "status32", + "pc", "status32", "lp_start", "lp_end", "bta" }; static const char *const core_arcompact_register_names[] = { @@ -1768,7 +1768,7 @@ static const struct frame_base arc_normal_base = { static int arc_tdesc_init (struct gdbarch_info info, const struct target_desc **tdesc, - struct tdesc_arch_data **tdesc_data) + struct tdesc_arch_data **tdesc_data, struct gdbarch_tdep *tdep) { if (arc_debug) debug_printf ("arc: Target description initialization.\n"); @@ -1919,8 +1919,35 @@ arc_tdesc_init (struct gdbarch_info info, const struct target_desc **tdesc, || (i >= ARC_R16_REGNUM && i <= ARC_R25_REGNUM))) continue; - valid_p = tdesc_numbered_register (feature, tdesc_data_loc, i, + /* R58 and R59 can have special names: ACCL and ACCH, however which + one is which depends on target endianness - for little endian R58 + is ACCL, R59 is ACCH; vice versa for big endian. */ + + const char *const r58_names[] = { + "r58", + (info.byte_order == BFD_ENDIAN_LITTLE ? "accl" : "acch"), + NULL + }; + const char *const r59_names[] = { + "r59", + (info.byte_order == BFD_ENDIAN_LITTLE ? "acch" : "accl"), + NULL + }; + + switch (i) + { + case ARC_R58_REGNUM: + valid_p = tdesc_numbered_register_choices (feature, tdesc_data_loc, + i, r58_names); + break; + case ARC_R59_REGNUM: + valid_p = tdesc_numbered_register_choices (feature, tdesc_data_loc, + i, r59_names); + break; + default: + valid_p = tdesc_numbered_register (feature, tdesc_data_loc, i, core_regs[i]); + } /* - Ignore errors in extension registers - they are optional. - Ignore missing ILINK because it doesn't make sense for Linux. @@ -1956,7 +1983,9 @@ arc_tdesc_init (struct gdbarch_info info, const struct target_desc **tdesc, { const char *name = aux_minimal_register_names[i - ARC_FIRST_AUX_REGNUM]; valid_p = tdesc_numbered_register (feature, tdesc_data_loc, i, name); - if (!valid_p) + + /* Only STATUS32 and PC are mandatory. */ + if (!valid_p && (i == ARC_PC_REGNUM || i == ARC_STATUS32_REGNUM)) { arc_print (_("Error: Cannot find required register `%s' " "in feature `%s'.\n"), @@ -1964,6 +1993,11 @@ arc_tdesc_init (struct gdbarch_info info, const struct target_desc **tdesc, tdesc_data_cleanup (tdesc_data_loc); return FALSE; } + /* Hardware loops present if both its registers are. */ + else if (ARC_LP_START_REGNUM == i) + tdep->has_hw_loops = valid_p; + else if (ARC_LP_END_REGNUM == i) + tdep->has_hw_loops &= valid_p; } *tdesc = tdesc_loc; @@ -1983,13 +2017,17 @@ arc_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) if (arc_debug) debug_printf ("arc: Architecture initialization.\n"); - if (!arc_tdesc_init (info, &tdesc, &tdesc_data)) - return NULL; - /* Allocate the ARC-private target-dependent information structure, and the GDB target-independent information structure. */ struct gdbarch_tdep *tdep = XCNEW (struct gdbarch_tdep); tdep->jb_pc = -1; /* No longjmp support by default. */ + + if (!arc_tdesc_init (info, &tdesc, &tdesc_data, tdep)) + { + XDELETE (tdep); + return NULL; + } + struct gdbarch *gdbarch = gdbarch_alloc (&info, tdep); /* Data types. */ @@ -2020,6 +2058,13 @@ arc_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) set_gdbarch_ps_regnum (gdbarch, ARC_STATUS32_REGNUM); set_gdbarch_fp0_regnum (gdbarch, -1); /* No FPU registers. */ + /* Confirm that register name lists have proper length. */ + gdb_static_assert (ARC_LAST_REGNUM + 1 + == (ARRAY_SIZE (core_v2_register_names) + + ARRAY_SIZE (aux_minimal_register_names))); + gdb_static_assert (ARRAY_SIZE (core_v2_register_names) + == ARRAY_SIZE (core_arcompact_register_names)); + set_gdbarch_dummy_id (gdbarch, arc_dummy_id); set_gdbarch_push_dummy_call (gdbarch, arc_push_dummy_call); set_gdbarch_push_dummy_code (gdbarch, arc_push_dummy_code); diff --git a/gdb/arc-tdep.h b/gdb/arc-tdep.h index 580ccb7..4ea06f6 100644 --- a/gdb/arc-tdep.h +++ b/gdb/arc-tdep.h @@ -53,6 +53,8 @@ enum arc_regnum ARC_R30_REGNUM, /* Return address from function. */ ARC_BLINK_REGNUM, + ARC_R58_REGNUM = 58, + ARC_R59_REGNUM, /* Zero-delay loop counter. */ ARC_LP_COUNT_REGNUM = 60, /* Reserved register number. There should never be a register with such @@ -74,8 +76,14 @@ enum arc_regnum ARC_FIRST_AUX_REGNUM = ARC_PC_REGNUM, /* Status register. */ ARC_STATUS32_REGNUM, - ARC_LAST_REGNUM = ARC_STATUS32_REGNUM, - ARC_LAST_AUX_REGNUM = ARC_STATUS32_REGNUM, + /* Zero-delay loop start instruction. */ + ARC_LP_START_REGNUM, + /* Zero-delay loop next-after-last instruction. */ + ARC_LP_END_REGNUM, + /* Branch target address. */ + ARC_BTA_REGNUM, + ARC_LAST_AUX_REGNUM = ARC_BTA_REGNUM, + ARC_LAST_REGNUM = ARC_LAST_AUX_REGNUM, /* Additional ABI constants. */ ARC_FIRST_ARG_REGNUM = ARC_R0_REGNUM, @@ -100,6 +108,9 @@ struct gdbarch_tdep /* Offset to PC value in jump buffer. If this is negative, longjmp support will be disabled. */ int jb_pc; + + /* Whether target has hardware (aka zero-delay) loops. */ + bool has_hw_loops; }; /* Utility functions used by other ARC-specific modules. */ diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index bfeb7a9..69a4a70 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -41614,7 +41614,8 @@ difference with @samp{org.gnu.gdb.arc.core.v2} feature is in the names of ARC v2, but @samp{ilink2} is optional on ARCompact. The @samp{org.gnu.gdb.arc.aux-minimal} feature is required for all ARC -targets. It should contain registers @samp{pc} and @samp{status32}. +targets. It should contain registers @samp{pc} and @samp{status32}. It may +contain registers @samp{lp_start}, @samp{lp_end} and @samp{bta}. @node ARM Features @subsection ARM Features