gdb: Don't ignore all SIGSTOP when the signal handler is set to pass
authorAndrew Burgess <andrew.burgess@embecosm.com>
Tue, 10 Sep 2019 14:24:28 +0000 (15:24 +0100)
committerAndrew Burgess <andrew.burgess@embecosm.com>
Thu, 3 Oct 2019 15:12:02 +0000 (16:12 +0100)
commitd8c06f22a38c849ae1d4488e6e4a7b0c780869b8
treef6b6660985dc458b2b62ce45ee3c7e5776aa19c5
parent3a56ed8668f57bbec6b769a862131d769d2fceb7
gdb: Don't ignore all SIGSTOP when the signal handler is set to pass

It was observed that in a multi-threaded application on GNU/Linux,
that if the user has set the SIGSTOP to be pass (using GDB's handle
command) then the inferior would hang upon hitting a breakpoint.

What happens is that when a thread hits the breakpoint GDB tries to
stop all of the other threads by sending them a SIGSTOP and setting
the stop_requested flag in the target_ops structure - this can be seen
in infrun.c:stop_all_threads.

GDB then waits for all of the other threads to stop.

When the SIGSTOP event arrives we eventually end up in
linux-nat.c:linux_nat_filter_event, which has the job of deciding if
the event we're looking at (the SIGSTOP arriving in this case) is
something that should be reported back to the core of GDB.

One of the final actions of this function is to check if we stopped
due to a signal, and if we did, and the signal has been set to 'pass'
by the user then we ignore the event and resume the thread.

This code already has some conditions in place that mean the event is
reported to GDB even if the signal is in the set of signals to be
passed to the inferior.

In this commit I extend this condition such that:

  If the signal is a SIGSTOP, and the thread's stop_requested flag is
  set (indicating we're waiting for the thread to stop with a SIGSTOP)
  then we should report this SIGSTOP to GDB and not pass it to the
  inferior.

With this change in place the test now passes.  Regression tested on
x86-64 GNU/Linux with no regressions.

gdb/ChangeLog:

* linux-nat.c (linux_nat_filter_event): Don't ignore SIGSTOP if we
have just sent the thread a SIGSTOP and are waiting for it to
arrive.

gdb/testsuite/ChangeLog:

* gdb.threads/stop-with-handle.c: New file.
* gdb.threads/stop-with-handle.exp: New file.
gdb/ChangeLog
gdb/linux-nat.c
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.threads/stop-with-handle.c [new file with mode: 0644]
gdb/testsuite/gdb.threads/stop-with-handle.exp [new file with mode: 0644]
This page took 0.025948 seconds and 4 git commands to generate.