From patchwork Tue Jan 13 19:00:48 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Pedro Alves X-Patchwork-Id: 4650 Received: (qmail 27858 invoked by alias); 13 Jan 2015 19:00:55 -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 27845 invoked by uid 89); 13 Jan 2015 19:00:54 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.0 required=5.0 tests=AWL, BAYES_00, SPF_HELO_PASS, SPF_PASS, T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Tue, 13 Jan 2015 19:00:52 +0000 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id t0DJ0nJm004018 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL); Tue, 13 Jan 2015 14:00:50 -0500 Received: from [127.0.0.1] (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t0DJ0mHG029809; Tue, 13 Jan 2015 14:00:49 -0500 Message-ID: <54B56B60.6040406@redhat.com> Date: Tue, 13 Jan 2015 19:00:48 +0000 From: Pedro Alves User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: GDB Patches CC: Joel Brobecker Subject: [PATCH] PR17525 - breakpoint commands not executed when program run, from -x script Executing a gdb script that runs the inferior (from the command line with -x), and has it hit breakpoints with breakpoint commands that themselves run the target, is currently broken on async targets (Linux, remote). While we're executing a command list or a script, we force the interpreter to be sync, which results in some functions nesting an event loop and waiting for the target to stop, instead of returning immediately and having the top level event loop handle the stop. The issue with this bug is simply that bpstat_do_actions misses checking whether the interpreter is sync. When we get here, in the case of executing a script (or, when the interpreter is sync), the program has already advanced to the next breakpoint, through maybe_wait_sync_command_done. We need to process its breakpoints immediately, just like with a sync target. Tested on x86_64 Fedora 20. gdb/ 2015-01-12 Pedro Alves PR gdb/17525 * breakpoint.c: Include "interps.h". (bpstat_do_actions_1): Also check whether the interpreter is async. gdb/testsuite/ 2015-01-12 Pedro Alves Joel Brobecker PR gdb/17525 * gdb.base/bp-cmds-execution-x-script.c: New file. * gdb.base/bp-cmds-execution-x-script.exp: New file. * gdb.base/bp-cmds-execution-x-script.gdb: New file. --- gdb/breakpoint.c | 4 +- .../gdb.base/bp-cmds-execution-x-script.c | 35 ++++++++++ .../gdb.base/bp-cmds-execution-x-script.exp | 80 ++++++++++++++++++++++ .../gdb.base/bp-cmds-execution-x-script.gdb | 21 ++++++ 4 files changed, 138 insertions(+), 2 deletions(-) create mode 100644 gdb/testsuite/gdb.base/bp-cmds-execution-x-script.c create mode 100644 gdb/testsuite/gdb.base/bp-cmds-execution-x-script.exp create mode 100644 gdb/testsuite/gdb.base/bp-cmds-execution-x-script.gdb diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index c64c2a9..af0c2cf 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -66,7 +66,7 @@ #include "skip.h" #include "ax-gdb.h" #include "dummy-frame.h" - +#include "interps.h" #include "format.h" /* readline include files */ @@ -4635,7 +4635,7 @@ bpstat_do_actions_1 (bpstat *bsp) if (breakpoint_proceeded) { - if (target_can_async_p ()) + if (interpreter_async && target_can_async_p ()) /* If we are in async mode, then the target might be still running, not stopped at any breakpoint, so nothing for us to do here -- just return to the event loop. */ diff --git a/gdb/testsuite/gdb.base/bp-cmds-execution-x-script.c b/gdb/testsuite/gdb.base/bp-cmds-execution-x-script.c new file mode 100644 index 0000000..f7101b6 --- /dev/null +++ b/gdb/testsuite/gdb.base/bp-cmds-execution-x-script.c @@ -0,0 +1,35 @@ +/* This testcase is part of GDB, the GNU debugger. + + Copyright 2015 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 . */ + +int +increment (int i) +{ + return i + 1; +} + +#define N_INCR 20 + +int +main (void) +{ + int val = 1; + int j; + + for (j = 0; j < N_INCR; j++) + val = increment (val); + return val == N_INCR; +} diff --git a/gdb/testsuite/gdb.base/bp-cmds-execution-x-script.exp b/gdb/testsuite/gdb.base/bp-cmds-execution-x-script.exp new file mode 100644 index 0000000..19c9f1a --- /dev/null +++ b/gdb/testsuite/gdb.base/bp-cmds-execution-x-script.exp @@ -0,0 +1,80 @@ +# Copyright 2015 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 . */ + +# Test that GDB isn't silent if it fails to remove a breakpoint from +# the main program, independently of whether the program was loaded +# with "file PROGRAM" or directly from the command line with "gdb +# PROGRAM". + +# Test that execution commands in a GDB script file run through -x +# work as expected. Specifically, the scripts creates a breakpoint +# that has "continue" in its command list, and then does "run". + +standard_testfile + +if {[build_executable "failed to prepare" $testfile $srcfile debug]} { + return -1 +} + +set x_file ${srcdir}/${subdir}/$testfile.gdb + +global GDBFLAGS +set saved_gdbflags $GDBFLAGS + +# Run the test proper. + +proc test {} { + global inferior_exited_re + global binfile + global gdb_prompt + global saved_gdbflags GDBFLAGS + global x_file + + gdb_exit + + set GDBFLAGS "$GDBFLAGS -x $x_file $binfile" + + set GDBFLAGS $saved_gdbflags + append GDBFLAGS " -ex \"set height unlimited\"" + append GDBFLAGS " -x \"$x_file\"" + append GDBFLAGS " --args \"$binfile\"" + + set test "run to end" + + gdb_exit + set res [gdb_spawn] + if { $res != 0} { + fail $test + return -1 + } + + set res -1 + gdb_test_multiple "" $test { + -re "$inferior_exited_re normally.*$gdb_prompt $" { + pass $test + } + -re "Don't know how to run.*$gdb_prompt $" { + set res 0 + unsupported $test + } + } + + # In case the board file wants to send further commands. + return $res +} + +test + +set GDBFLAGS $saved_gdbflags diff --git a/gdb/testsuite/gdb.base/bp-cmds-execution-x-script.gdb b/gdb/testsuite/gdb.base/bp-cmds-execution-x-script.gdb new file mode 100644 index 0000000..b49c346 --- /dev/null +++ b/gdb/testsuite/gdb.base/bp-cmds-execution-x-script.gdb @@ -0,0 +1,21 @@ +# Copyright 2015 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 . */ + +set pagination off +break increment +commands + cont +end +run