From 76f50ad1ce0f1aecca89a7c55c5fcd3c4293d8d2 Mon Sep 17 00:00:00 2001 From: Daniel Jacobowitz Date: Thu, 3 Nov 2005 19:31:38 +0000 Subject: [PATCH] * linux-nat.c (linux_nat_resume): Add more debugging messages. Do not short-circuit resuming all threads if the signal will be ignored in linux_nat_wait. --- gdb/ChangeLog | 6 ++++++ gdb/linux-nat.c | 43 ++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 48 insertions(+), 1 deletion(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 0a62f48e3ae..e31c2723ce0 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2006-11-03 Daniel Jacobowitz + + * linux-nat.c (linux_nat_resume): Add more debugging messages. Do + not short-circuit resuming all threads if the signal will be ignored + in linux_nat_wait. + 2005-11-02 Andrew Stubbs * monitor.c (monitor_xfer_memory): Change char to gdb_byte. diff --git a/gdb/linux-nat.c b/gdb/linux-nat.c index 61c0effa52c..84ee5d4771e 100644 --- a/gdb/linux-nat.c +++ b/gdb/linux-nat.c @@ -1072,6 +1072,14 @@ linux_nat_resume (ptid_t ptid, int step, enum target_signal signo) struct lwp_info *lp; int resume_all; + if (debug_linux_nat) + fprintf_unfiltered (gdb_stdlog, + "LLR: Preparing to %s %s, %s, inferior_ptid %s\n", + step ? "step" : "resume", + target_pid_to_str (ptid), + signo ? strsignal (signo) : "0", + target_pid_to_str (inferior_ptid)); + /* A specific PTID means `step only this process id'. */ resume_all = (PIDGET (ptid) == -1); @@ -1097,12 +1105,45 @@ linux_nat_resume (ptid_t ptid, int step, enum target_signal signo) lp->resumed = 1; /* If we have a pending wait status for this thread, there is no - point in resuming the process. */ + point in resuming the process. But first make sure that + linux_nat_wait won't preemptively handle the event - we + should never take this short-circuit if we are going to + leave LP running, since we have skipped resuming all the + other threads. This bit of code needs to be synchronized + with linux_nat_wait. */ + + if (lp->status && WIFSTOPPED (lp->status)) + { + int saved_signo = target_signal_from_host (WSTOPSIG (lp->status)); + + if (signal_stop_state (saved_signo) == 0 + && signal_print_state (saved_signo) == 0 + && signal_pass_state (saved_signo) == 1) + { + if (debug_linux_nat) + fprintf_unfiltered (gdb_stdlog, + "LLR: Not short circuiting for ignored " + "status 0x%x\n", lp->status); + + /* FIXME: What should we do if we are supposed to continue + this thread with a signal? */ + gdb_assert (signo == TARGET_SIGNAL_0); + signo = saved_signo; + lp->status = 0; + } + } + if (lp->status) { /* FIXME: What should we do if we are supposed to continue this thread with a signal? */ gdb_assert (signo == TARGET_SIGNAL_0); + + if (debug_linux_nat) + fprintf_unfiltered (gdb_stdlog, + "LLR: Short circuiting for status 0x%x\n", + lp->status); + return; } -- 2.30.2