From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by sourceware.org (Postfix) with ESMTPS id 7C152384640E for ; Thu, 20 Aug 2020 21:24:51 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 7C152384640E IronPort-SDR: kn8G38TDm2VTVObq8AExd7eCemHd6smFgzYzwQ46nxl28HkzFMwyFOBzk3h9K7VdaL66cZyBQ+ MSW7E/WLyNyw== X-IronPort-AV: E=McAfee;i="6000,8403,9719"; a="155396644" X-IronPort-AV: E=Sophos;i="5.76,334,1592895600"; d="scan'208";a="155396644" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Aug 2020 14:24:49 -0700 IronPort-SDR: UB3XNa50iQjhttAygT15NoKPBlhhUcx9ayFjOmuo4j8BNzXcI7C4UfV4ZjscUuRsRZoQIUPNxB AiNPIYjTmFYw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.76,334,1592895600"; d="scan'208";a="297726138" Received: from irvmail001.ir.intel.com ([163.33.26.43]) by orsmga006.jf.intel.com with ESMTP; 20 Aug 2020 14:24:49 -0700 Received: from ulvlx001.iul.intel.com (ulvlx001.iul.intel.com [172.28.207.17]) by irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id 07KLOmgm005749; Thu, 20 Aug 2020 22:24:48 +0100 Received: from ulvlx001.iul.intel.com (localhost [127.0.0.1]) by ulvlx001.iul.intel.com with ESMTP id 07KLOmNk022866; Thu, 20 Aug 2020 23:24:48 +0200 Received: (from taktemur@localhost) by ulvlx001.iul.intel.com with LOCAL id 07KLOmss022862; Thu, 20 Aug 2020 23:24:48 +0200 From: Tankut Baris Aktemur To: gdb-patches@sourceware.org Subject: [PATCH v2 2/2] gdb/breakpoint: add flags to 'condition' and 'break' commands to force condition Date: Thu, 20 Aug 2020 23:24:39 +0200 Message-Id: <47d8fcdb736b717ec984b9dfcf4035c5d797e034.1597957849.git.tankut.baris.aktemur@intel.com> X-Mailer: git-send-email 1.7.0.7 In-Reply-To: References: In-Reply-To: References: X-Spam-Status: No, score=-18.5 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_NONE, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Aug 2020 21:24:54 -0000 The previous patch made it possible to define a condition if it's valid at some locations. If the condition is invalid at all of the locations, it's rejected. However, there may be cases where the user knows the condition *will* be valid at a location in the future, e.g. due to a shared library load. To make it possible that such condition can be defined, this patch adds an optional '-force' flag to the 'condition' command, and, respectively, a '-force-condition' flag to the 'break'command. When the force flag is passed, the condition is not rejected even when it is invalid for all the current locations (note that all the locations would be internally disabled in this case). For instance: (gdb) break test.c:5 Breakpoint 1 at 0x1155: file test.c, line 5. (gdb) cond 1 foo == 42 No symbol "foo" in current context. Defining the condition was not possible because 'foo' is not available. The user can override this behavior with the '-force' flag: (gdb) cond -force 1 foo == 42 warning: disabling breakpoint 1.1: No symbol "foo" in current context. (gdb) info breakpoints Num Type Disp Enb Address What 1 breakpoint keep y stop only if foo == 42 1.1 n 0x0000000000001155 in main at test.c:5 Now the condition is accepted, but the location is automatically disabled. If a future location has a context in which 'foo' is available, that location would be enabled. For the 'break' command, -force-condition has the same result: (gdb) break test.c:5 -force-condition if foo == 42 warning: disabling breakpoint location 1: No symbol "foo" in current context. Breakpoint 1 at 0x1169: file test.c, line 5. gdb/ChangeLog: 2020-07-31 Tankut Baris Aktemur * breakpoint.h (set_breakpoint_condition): Add a new bool parameter. * breakpoint.c: Update the help text of the 'condition' and 'break' commands. (set_breakpoint_condition): Take a new bool parameter to control whether condition definition should be forced even when the condition expression is invalid in all of the current locations. (condition_command): Update the call to 'set_breakpoint_condition'. (find_condition_and_thread): Take the "-force-condition" flag into account. * linespec.c (linespec_keywords): Add "-force-condition" as an element. (FORCE_KEYWORD_INDEX): New #define. (linespec_lexer_lex_keyword): Update to consider "-force-condition" as a keyword. * ada-lang.c (create_ada_exception_catchpoint): Ditto. * guile/scm-breakpoint.c (gdbscm_set_breakpoint_condition_x): Ditto. * python/py-breakpoint.c (bppy_set_condition): Ditto. gdb/testsuite/ChangeLog: 2020-08-20 Tankut Baris Aktemur * gdb.base/condbreak-multi-context.exp: Expand to test forcing the condition. * gdb.linespec/cpcompletion.exp: Update to consider the '-force-condition' keyword. * gdb.linespec/explicit.exp: Ditto. * lib/completion-support.exp: Ditto. gdb/doc/ChangeLog: 2020-07-31 Tankut Baris Aktemur * gdb.texinfo: Document the '-force-condition' flag of the 'break' command, and the '-force' flag of the 'condition' command. --- gdb/ada-lang.c | 2 +- gdb/breakpoint.c | 52 ++++++++++++++++--- gdb/breakpoint.h | 6 ++- gdb/doc/gdb.texinfo | 50 ++++++++++++++++++ gdb/guile/scm-breakpoint.c | 2 +- gdb/linespec.c | 18 +++++-- gdb/python/py-breakpoint.c | 2 +- .../gdb.base/condbreak-multi-context.exp | 22 ++++++++ gdb/testsuite/gdb.linespec/cpcompletion.exp | 12 +++-- gdb/testsuite/gdb.linespec/explicit.exp | 1 + gdb/testsuite/lib/completion-support.exp | 2 +- 11 files changed, 148 insertions(+), 21 deletions(-) diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c index 29951528e5e..69b3c3c63cd 100644 --- a/gdb/ada-lang.c +++ b/gdb/ada-lang.c @@ -12702,7 +12702,7 @@ create_ada_exception_catchpoint (struct gdbarch *gdbarch, c->excep_string = excep_string; create_excep_cond_exprs (c.get (), ex_kind); if (!cond_string.empty ()) - set_breakpoint_condition (c.get (), cond_string.c_str (), from_tty); + set_breakpoint_condition (c.get (), cond_string.c_str (), from_tty, false); install_breakpoint (0, std::move (c), 1); } diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index 7abfd510abc..239420f4c0f 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -874,7 +874,7 @@ set_breakpoint_location_condition (const char *cond_string, bp_location *loc, void set_breakpoint_condition (struct breakpoint *b, const char *exp, - int from_tty) + int from_tty, bool force) { if (*exp == 0) { @@ -941,9 +941,11 @@ set_breakpoint_condition (struct breakpoint *b, const char *exp, catch (const gdb_exception_error &e) { /* Condition string is invalid. If this happens to - be the last loc, abandon. */ + be the last loc, abandon (if not forced) or continue + (if forced). */ if (loc->next == nullptr) - throw; + if (!force) + throw; } } @@ -1023,6 +1025,18 @@ condition_command (const char *arg, int from_tty) error_no_arg (_("breakpoint number")); p = arg; + + /* Check if the "-force" flag was passed. */ + bool force = false; + const char *tok = skip_spaces (p); + const char *end_tok = skip_to_space (tok); + int toklen = end_tok - tok; + if (toklen >= 1 && strncmp (tok, "-force", toklen) == 0) + { + force = true; + p = end_tok + 1; + } + bnum = get_number (&p); if (bnum == 0) error (_("Bad breakpoint argument: '%s'"), arg); @@ -1042,7 +1056,7 @@ condition_command (const char *arg, int from_tty) " a %s stop condition defined for this breakpoint."), ext_lang_capitalized_name (extlang)); } - set_breakpoint_condition (b, p, from_tty); + set_breakpoint_condition (b, p, from_tty, force); if (is_breakpoint (b)) update_global_location_list (UGLL_MAY_INSERT); @@ -9150,6 +9164,7 @@ find_condition_and_thread (const char *tok, CORE_ADDR pc, *thread = -1; *task = 0; *rest = NULL; + bool force = false; while (tok && *tok) { @@ -9173,10 +9188,25 @@ find_condition_and_thread (const char *tok, CORE_ADDR pc, if (toklen >= 1 && strncmp (tok, "if", toklen) == 0) { tok = cond_start = end_tok + 1; - parse_exp_1 (&tok, pc, block_for_pc (pc), 0); + try + { + parse_exp_1 (&tok, pc, block_for_pc (pc), 0); + } + catch (const gdb_exception_error &) + { + if (!force) + throw; + else + tok = tok + strlen (tok); + } cond_end = tok; *cond_string = savestring (cond_start, cond_end - cond_start); } + else if (toklen >= 1 && strncmp (tok, "-force-condition", toklen) == 0) + { + tok = cond_start = end_tok + 1; + force = true; + } else if (toklen >= 1 && strncmp (tok, "thread", toklen) == 0) { const char *tmptok; @@ -15242,7 +15272,8 @@ specified name as a complete fully-qualified name instead." command. */ #define BREAK_ARGS_HELP(command) \ -command" [PROBE_MODIFIER] [LOCATION] [thread THREADNUM] [if CONDITION]\n\ +command" [PROBE_MODIFIER] [LOCATION] [thread THREADNUM]\n\ +\t[-force-condition] [if CONDITION]\n\ PROBE_MODIFIER shall be present if the command is to be placed in a\n\ probe point. Accepted values are `-probe' (for a generic, automatically\n\ guessed probe type), `-probe-stap' (for a SystemTap probe) or \n\ @@ -15255,6 +15286,9 @@ stack frame. This is useful for breaking on return to a stack frame.\n\ \n\ THREADNUM is the number from \"info threads\".\n\ CONDITION is a boolean expression.\n\ +\n\ +With the \"-force-condition\" flag, the condition is defined even when\n\ +it is invalid for all current locations.\n\ \n" LOCATION_HELP_STRING "\n\n\ Multiple breakpoints at one place are permitted, and useful if their\n\ conditions are different.\n\ @@ -15576,8 +15610,10 @@ then no output is printed when it is hit, except what the commands print.")); c = add_com ("condition", class_breakpoint, condition_command, _("\ Specify breakpoint number N to break only if COND is true.\n\ -Usage is `condition N COND', where N is an integer and COND is an\n\ -expression to be evaluated whenever breakpoint N is reached.")); +Usage is `condition [-force] N COND', where N is an integer and COND\n\ +is an expression to be evaluated whenever breakpoint N is reached.\n\ +With the \"-force\" flag, the condition is defined even when it is\n\ +invalid for all current locations.")); set_cmd_completer (c, condition_completer); c = add_com ("tbreak", class_breakpoint, tbreak_command, _("\ diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h index 145bc8a397a..6acec428a9a 100644 --- a/gdb/breakpoint.h +++ b/gdb/breakpoint.h @@ -1617,9 +1617,11 @@ extern int breakpoints_should_be_inserted_now (void); in our opinion won't ever trigger. */ extern void breakpoint_retire_moribund (void); -/* Set break condition of breakpoint B to EXP. */ +/* Set break condition of breakpoint B to EXP. + If FORCE, define the condition even if it is invalid in + all of the breakpoint locations. */ extern void set_breakpoint_condition (struct breakpoint *b, const char *exp, - int from_tty); + int from_tty, bool force); /* Checks if we are catching syscalls or not. Returns 0 if not, greater than 0 if we are. */ diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index 77013648b2f..1d3c1f29f94 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -4278,6 +4278,50 @@ value is nonzero---that is, if @var{cond} evaluates as true. above (or no argument) specifying where to break. @xref{Conditions, ,Break Conditions}, for more information on breakpoint conditions. +The breakpoint may be mapped to multiple locations. If the breakpoint +condition @var{cond} is invalid at some but not all of the locations, +the locations for which the condition is invalid are disabled. For +example, @value{GDBN} reports below that two of the three locations +are disabled. + +@smallexample +(@value{GDBP}) break func if a == 42 +warning: disabling breakpoint location 2: No symbol "a" in current context. +warning: disabling breakpoint location 3: No symbol "a" in current context. +Breakpoint 1 at 0x401129: test.c:18. (3 locations) +@end smallexample + +If the breakpoint condition @var{cond} is invalid in the context of +@emph{all} the locations of the breakpoint, @value{GDBN} refuses to +define the breakpoint. For example, if variable @code{foo} is an +undefined variable: + +@smallexample +(@value{GDBP}) break func if foo +No symbol "foo" in current context. +@end smallexample + +@item break @dots{} -force-condition if @var{cond} +There may be cases where the condition @var{cond} is invalid at all +the current locations, but the user knows that it will be valid at a +future location; for example, because of a library load. In such +cases, by using the @code{-force-condition} keyword before @samp{if}, +@value{GDBN} can be forced to define the breakpoint with the given +condition expression instead of refusing it. + +@smallexample +(@value{GDBP}) break func -force-condition if foo +warning: disabling breakpoint location 1: No symbol "foo" in current context. +warning: disabling breakpoint location 2: No symbol "foo" in current context. +warning: disabling breakpoint location 3: No symbol "foo" in current context. +Breakpoint 1 at 0x401129: test.c:18. (3 locations) +@end smallexample + +This causes all the present locations where the breakpoint would +otherwise be inserted, to be disabled, as seen in the example above. +However, if there exist locations at which the condition is valid, the +@code{-force-condition} keyword has no effect. + @kindex tbreak @item tbreak @var{args} Set a breakpoint enabled only for one stop. The @var{args} are the @@ -5457,6 +5501,12 @@ prints an error message: No symbol "foo" in current context. @end smallexample +@item condition -force @var{bnum} @var{expression} +When the @code{-force} flag is used, define the condition even if +@var{expression} is invalid in all the current locations of breakpoint +@var{bpnum}. This is similar to the @code{-force-condition} option +of the @code{break} command. + @noindent @value{GDBN} does not actually evaluate @var{expression} at the time the @code{condition} diff --git a/gdb/guile/scm-breakpoint.c b/gdb/guile/scm-breakpoint.c index 96b6ca91f8d..7c9707235ec 100644 --- a/gdb/guile/scm-breakpoint.c +++ b/gdb/guile/scm-breakpoint.c @@ -905,7 +905,7 @@ gdbscm_set_breakpoint_condition_x (SCM self, SCM newvalue) ? nullptr : gdbscm_scm_to_c_string (newvalue)); - set_breakpoint_condition (bp_smob->bp, exp ? exp.get () : "", 0); + set_breakpoint_condition (bp_smob->bp, exp ? exp.get () : "", 0, false); return SCM_UNSPECIFIED; }); diff --git a/gdb/linespec.c b/gdb/linespec.c index 4a634e3aff9..3bfcf41ae06 100644 --- a/gdb/linespec.c +++ b/gdb/linespec.c @@ -72,7 +72,7 @@ enum class linespec_complete_what /* An expression. E.g., "break foo if EXPR", or "break *EXPR". */ EXPRESSION, - /* A linespec keyword ("if"/"thread"/"task"). + /* A linespec keyword ("if"/"thread"/"task"/"-force-condition"). E.g., "break func threa". */ KEYWORD, }; @@ -254,8 +254,9 @@ typedef enum ls_token_type linespec_token_type; /* List of keywords. This is NULL-terminated so that it can be used as enum completer. */ -const char * const linespec_keywords[] = { "if", "thread", "task", NULL }; +const char * const linespec_keywords[] = { "if", "thread", "task", "-force-condition", NULL }; #define IF_KEYWORD_INDEX 0 +#define FORCE_KEYWORD_INDEX 3 /* A token of the linespec lexer */ @@ -486,11 +487,22 @@ linespec_lexer_lex_keyword (const char *p) { int j; + /* Special case: "-force" is always followed by an "if". */ + if (i == FORCE_KEYWORD_INDEX) + { + p += len; + p = skip_spaces (p); + int nextlen = strlen (linespec_keywords[IF_KEYWORD_INDEX]); + if (!(strncmp (p, linespec_keywords[IF_KEYWORD_INDEX], nextlen) == 0 + && isspace (p[nextlen]))) + return NULL; + } + /* Special case: "if" ALWAYS stops the lexer, since it is not possible to predict what is going to appear in the condition, which can only be parsed after SaLs have been found. */ - if (i != IF_KEYWORD_INDEX) + else if (i != IF_KEYWORD_INDEX) { p += len; p = skip_spaces (p); diff --git a/gdb/python/py-breakpoint.c b/gdb/python/py-breakpoint.c index 11345ad3052..56640421994 100644 --- a/gdb/python/py-breakpoint.c +++ b/gdb/python/py-breakpoint.c @@ -461,7 +461,7 @@ bppy_set_condition (PyObject *self, PyObject *newvalue, void *closure) try { - set_breakpoint_condition (self_bp->bp, exp, 0); + set_breakpoint_condition (self_bp->bp, exp, 0, false); } catch (gdb_exception &ex) { diff --git a/gdb/testsuite/gdb.base/condbreak-multi-context.exp b/gdb/testsuite/gdb.base/condbreak-multi-context.exp index b02a0a2e40c..30bde8adf7c 100644 --- a/gdb/testsuite/gdb.base/condbreak-multi-context.exp +++ b/gdb/testsuite/gdb.base/condbreak-multi-context.exp @@ -216,3 +216,25 @@ with_test_prefix "scenario 3" { # The second BP's locations are all disabled. No more hits! gdb_continue_to_end } + +# Scenario 4: Test the '-force'/'-force-condition' flag. + +with_test_prefix "force" { + clean_restart ${binfile} + + gdb_breakpoint "$srcfile2:$bp_location" + # Pick a condition that is invalid at every location. + set bpnum1 [get_integer_valueof "\$bpnum" 0 "get bpnum1"] + gdb_test "cond -force $bpnum1 foo" \ + [multi_line \ + "${warning} 1.1: No symbol \"foo\" in current context." \ + "${warning} 1.2: No symbol \"foo\" in current context." \ + "${warning} 1.3: No symbol \"foo\" in current context."] \ + "force the condition of bp 1" + check_bp_locations $bpnum1 {n n n} "after forcing the condition" + + # Now with the 'break' command. + gdb_breakpoint "$srcfile2:$bp_location -force-condition if baz" + set bpnum2 [get_integer_valueof "\$bpnum" 0 "get bpnum2"] + check_bp_locations $bpnum2 {n n n} "set using the break command" +} diff --git a/gdb/testsuite/gdb.linespec/cpcompletion.exp b/gdb/testsuite/gdb.linespec/cpcompletion.exp index 2c95e2b4048..de96556f22d 100644 --- a/gdb/testsuite/gdb.linespec/cpcompletion.exp +++ b/gdb/testsuite/gdb.linespec/cpcompletion.exp @@ -836,12 +836,14 @@ proc_with_prefix function-labels {} { } # Test that completion after a function name offers keyword -# (if/task/thread) matches in linespec mode, and also the explicit -# location options in explicit locations mode. +# (if/task/thread/-force-condition) matches in linespec mode, and also +# the explicit location options in explicit locations mode. proc_with_prefix keywords-after-function {} { set explicit_list \ - [concat $completion::explicit_opts_list $completion::keyword_list] + [lsort [concat \ + $completion::explicit_opts_list \ + $completion::keyword_list]] # Test without a source file, with a known source file, and with # and unknown source file. @@ -865,7 +867,9 @@ proc_with_prefix keywords-after-function {} { proc_with_prefix keywords-after-label {} { set explicit_list \ - [concat $completion::explicit_opts_list $completion::keyword_list] + [lsort [concat \ + $completion::explicit_opts_list \ + $completion::keyword_list]] foreach_location_labels \ { "" "cpls.cc" } \ diff --git a/gdb/testsuite/gdb.linespec/explicit.exp b/gdb/testsuite/gdb.linespec/explicit.exp index 4f457dc372f..5e3d352918e 100644 --- a/gdb/testsuite/gdb.linespec/explicit.exp +++ b/gdb/testsuite/gdb.linespec/explicit.exp @@ -405,6 +405,7 @@ namespace eval $testfile { # completion matches both the explicit location options and # the linespec keywords. set completions_list { + "-force-condition" "-function" "-label" "-line" diff --git a/gdb/testsuite/lib/completion-support.exp b/gdb/testsuite/lib/completion-support.exp index 51436cc6713..dbb958157ed 100644 --- a/gdb/testsuite/lib/completion-support.exp +++ b/gdb/testsuite/lib/completion-support.exp @@ -27,7 +27,7 @@ namespace eval completion { # List of all quote chars, including no-quote at all. variable maybe_quoted_list {"" "'" "\""} - variable keyword_list {"if" "task" "thread"} + variable keyword_list {"-force-condition" "if" "task" "thread"} variable explicit_opts_list \ {"-function" "-label" "-line" "-qualified" "-source"} -- 2.17.1