On Saturday, December 17, 2011 10:41:15 pm m...@freebsd.org wrote:
> On Sat, Dec 17, 2011 at 5:45 PM, Alexander Kabaev <kab...@gmail.com> wrote:
> > On Sun, 18 Dec 2011 01:09:00 +0100
> > "O. Hartmann" <ohart...@zedat.fu-berlin.de> wrote:
> >
> >> Sleeping thread (tid 100033, pid 16) owns a non sleepable lock
> >> panic: sleeping thread
> >> cpuid = 0
> >>
> >> PID 16 is always USB on my box.
> >
> > You really need to give us a backtrace when you quote panics. It is
> > impossible to make any sense of the above panic message without more
> > context.
>
> In the case of this panic, the stack of the thread which panics is
> useless; it's someone trying to propagate priority that discovered it.
> A backtrace on tid 100033 would be useful.
>
> With WITNESS enabled, it's possible to have this panic display the
> stack of the incorrectly sleeping thread at the time it acquired the
> lock, as well, but this code isn't in CURRENT or any release. I have
> a patch at $WORK I can dig up on Monday.
Huh? The stock kernel dumps a stack trace of the offending thread if you have
DDB enabled:
/*
* If the thread is asleep, then we are probably about
* to deadlock. To make debugging this easier, just
* panic and tell the user which thread misbehaved so
* they can hopefully get a stack trace from the truly
* misbehaving thread.
*/
if (TD_IS_SLEEPING(td)) {
printf(
"Sleeping thread (tid %d, pid %d) owns a non-sleepable lock\n",
td->td_tid, td->td_proc->p_pid);
#ifdef DDB
db_trace_thread(td, -1);
#endif
panic("sleeping thread");
}
It may be that we can make use of the STACK API here instead to output this
trace even when DDB isn't enabled. The patch below tries to do that
(untested). It does some odd thigns though since it is effectively running
from a panic context already, so it uses a statically allocated 'struct stack'
rather than using stack_create() and uses stack_print_ddb() since it is
holding spin locks and can't possibly grab an sx lock:
Index: subr_turnstile.c
===================================================================
--- subr_turnstile.c (revision 228534)
+++ subr_turnstile.c (working copy)
@@ -72,6 +72,7 @@ __FBSDID("$FreeBSD$");
#include <sys/proc.h>
#include <sys/queue.h>
#include <sys/sched.h>
+#include <sys/stack.h>
#include <sys/sysctl.h>
#include <sys/turnstile.h>
@@ -175,6 +176,7 @@ static void turnstile_fini(void *mem, int size);
static void
propagate_priority(struct thread *td)
{
+ static struct stack st;
struct turnstile *ts;
int pri;
@@ -217,8 +219,10 @@ propagate_priority(struct thread *td)
printf(
"Sleeping thread (tid %d, pid %d) owns a non-sleepable lock\n",
td->td_tid, td->td_proc->p_pid);
-#ifdef DDB
- db_trace_thread(td, -1);
+#ifdef STACK
+ stack_zero(&st);
+ stack_save_td(&st, td);
+ stack_print_ddb(&st);
#endif
panic("sleeping thread");
}
--
John Baldwin
_______________________________________________
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"
