diff options
author | Pedro Alves <palves@redhat.com> | 2015-09-14 15:45:14 +0100 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2015-09-14 15:45:14 +0100 |
commit | 4c2f2a792a5971fcc7fe6511725eaf50d19d302b (patch) | |
tree | 1bcd3950962a71534108444d1a97e2d77de378f6 /gdb/testsuite/gdb.base/hook-stop-continue.c | |
parent | 919e6dbe9b61a27e8f7f89121ba182907df461a3 (diff) | |
download | gdb-4c2f2a792a5971fcc7fe6511725eaf50d19d302b.zip gdb-4c2f2a792a5971fcc7fe6511725eaf50d19d302b.tar.gz gdb-4c2f2a792a5971fcc7fe6511725eaf50d19d302b.tar.bz2 |
Bail out of processing stop if hook-stop resumes target / changes context
This patch, relative to a tree with
https://sourceware.org/ml/gdb-patches/2015-08/msg00295.html, fixes
issues/crashes that trigger if something unexpected happens during a
hook-stop.
E.g., if the inferior disappears while running the hook-stop, we hit
failed assertions:
(gdb) define hook-stop
Type commands for definition of "hook-stop".
End with a line saying just "end".
>kill
>end
(gdb) si
Kill the program being debugged? (y or n) [answered Y; input not from terminal]
/home/pedro/gdb/mygit/build/../src/gdb/thread.c:88: internal-error: inferior_thread: Assertion `tp' failed.
A problem internal to GDB has been detected,
further debugging may prove unreliable.
Quit this debugging session? (y or n)
I noticed that if a hook-stop issues a synchronous execution command,
we print the same stop event twice:
(gdb) define hook-stop
Type commands for definition of "hook-stop".
End with a line saying just "end".
>si
>end
(gdb) si
0x000000000040074a 42 args[i] = 1; /* Init value. */ <<<<<<< once
0x000000000040074a 42 args[i] = 1; /* Init value. */ <<<<<<< twice
(gdb)
In MI:
*stopped,reason="end-stepping-range",frame={addr="0x000000000040074a",func="main",args=[],file="threads.c",fullname="/home/pedro/gdb/tests/threads.c",line="42"},thread-id="1",stopped-threads="all",core="0"
*stopped,reason="end-stepping-range",frame={addr="0x000000000040074a",func="main",args=[],file="threads.c",fullname="/home/pedro/gdb/tests/threads.c",line="42"},thread-id="1",stopped-threads="all",core="0"
(gdb)
The fix has GDB stop processing the event if the context changed. I
don't expect people to be doing crazy things from the hook-stop.
E.g., it gives me headaches to try to come up a proper behavior for
handling a thread change from a hook-stop... (E.g., imagine the
hook-stop does thread N; step, with scheduler-locing on). I think the
most important bit here is preventing crashes.
The patch adds a new hook-stop.exp test that covers the above and also
merges in the old hook-stop-continue.exp and hook-stop-frame.exp into
the same framework.
gdb/ChangeLog:
2015-09-14 Pedro Alves <palves@redhat.com>
* infrun.c (current_stop_id): New global.
(get_stop_id, new_stop_id): New functions.
(fetch_inferior_event): Handle normal_stop proceeding the target.
(struct stop_context): New.
(save_stop_context, release_stop_context_cleanup)
(stop_context_changed): New functions.
(normal_stop): Return true if the hook-stop changes the stop
context.
* infrun.h (get_stop_id): Declare.
(normal_stop): Now returns int. Add documentation.
gdb/testsuite/ChangeLog:
2015-09-14 Pedro Alves <palves@redhat.com>
* gdb.base/hook-stop-continue.c: Delete.
* gdb.base/hook-stop-continue.exp: Delete.
* gdb.base/hook-stop-frame.c: Delete.
* gdb.base/hook-stop-frame.exp: Delete.
* gdb.base/hook-stop.c: New file.
* gdb.base/hook-stop.exp: New file.
Diffstat (limited to 'gdb/testsuite/gdb.base/hook-stop-continue.c')
-rw-r--r-- | gdb/testsuite/gdb.base/hook-stop-continue.c | 42 |
1 files changed, 0 insertions, 42 deletions
diff --git a/gdb/testsuite/gdb.base/hook-stop-continue.c b/gdb/testsuite/gdb.base/hook-stop-continue.c deleted file mode 100644 index 47090fc..0000000 --- a/gdb/testsuite/gdb.base/hook-stop-continue.c +++ /dev/null @@ -1,42 +0,0 @@ -/* This testcase is part of GDB, the GNU debugger. - - Copyright 2008-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 <http://www.gnu.org/licenses/>. */ - -int -funbreak (int i) -{ - i = i * 2; /* set breakpoint here */ - i = i + 10; - return i; -} - -int -func (int i) -{ - return i * 2; -} - -int -main (int argc, char **argv, char **envp) -{ - func (1); - func (2); - func (3); - func (4); - funbreak (5); - - return 0; -} |