Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 26 Jun 2009 16:46:39 +0400
From:      Stanislav Sedov <stas@FreeBSD.org>
To:        Thomas Backman <serenity@exscape.org>
Cc:        freebsd-current@freebsd.org, Artem Belevich <fbsdlist@src.cx>
Subject:   Re: ZFS : panic("sleeping thread")
Message-ID:  <20090626164639.71a34f62.stas@FreeBSD.org>
In-Reply-To: <993B7B5B-1B6B-48A5-8425-6A1D071335A9@exscape.org>
References:  <ed91d4a80905271058mbc1fc68g184def5097080cbc@mail.gmail.com> <993B7B5B-1B6B-48A5-8425-6A1D071335A9@exscape.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 18 Jun 2009 13:49:46 +0200
Thomas Backman <serenity@exscape.org> mentioned:

> Anyone have any updates on this? I just got a "sleeping thread" panic  
> in ZFS after doing a zfs rollback. Unfortunately, "panic" in the  
> debugger resulted in "dump device too small" (despite being RAM-sized)  
> so I don't have a BT... However the BT I got in the debugger was *not*  
> the same as yours. There was no _sx_xlock in it, but that's pretty  
> much all I know about it. :(
> 

Hi, Thomas!

Can you, please, try with the following patch? Thanks!

---------------------------------------------------------------------------

diff -r a707b4dd87cf src/sys/kern/kern_event.c
--- a/src/sys/kern/kern_event.c	Fri Jun 26 13:49:09 2009 +0400
+++ b/src/sys/kern/kern_event.c	Fri Jun 26 16:45:15 2009 +0400
@@ -1608,7 +1608,7 @@
  * first.
  */
 void
-knote(struct knlist *list, long hint, int islocked)
+knote(struct knlist *list, long hint, int lockflags)
 {
 	struct kqueue *kq;
 	struct knote *kn;
@@ -1617,9 +1617,9 @@
 	if (list == NULL)
 		return;
 
-	KNL_ASSERT_LOCK(list, islocked);
+	KNL_ASSERT_LOCK(list, lockflags & KNF_LISTLOCKED);
 
-	if (!islocked) 
+	if ((lockflags & KNF_LISTLOCKED) == 0)
 		list->kl_lock(list->kl_lockarg); 
 
 	/*
@@ -1636,7 +1636,7 @@
 			KQ_LOCK(kq);
 			if ((kn->kn_status & KN_INFLUX) == KN_INFLUX) {
 				KQ_UNLOCK(kq);
-			} else {
+			} else if ((lockflags & KNF_NOKQLOCK) != 0) {
 				kn->kn_status |= KN_INFLUX;
 				KQ_UNLOCK(kq);
 				error = kn->kn_fop->f_event(kn, hint);
@@ -1645,11 +1645,17 @@
 				if (error)
 					KNOTE_ACTIVATE(kn, 1);
 				KQ_UNLOCK_FLUX(kq);
+			} else {
+				kn->kn_status |= KN_HASKQLOCK;
+				if (kn->kn_fop->f_event(kn, hint))
+					KNOTE_ACTIVATE(kn, 1);
+				kn->kn_status &= ~KN_HASKQLOCK;
+				KQ_UNLOCK(kq);
 			}
 		}
 		kq = NULL;
 	}
-	if (!islocked)
+	if ((lockflags & KNF_LISTLOCKED) == 0)
 		list->kl_unlock(list->kl_lockarg); 
 }
 
diff -r a707b4dd87cf src/sys/sys/event.h
--- a/src/sys/sys/event.h	Fri Jun 26 13:49:09 2009 +0400
+++ b/src/sys/sys/event.h	Fri Jun 26 16:45:15 2009 +0400
@@ -135,8 +135,14 @@
 MALLOC_DECLARE(M_KQUEUE);
 #endif
 
-#define KNOTE(list, hist, lock)		knote(list, hist, lock)
-#define KNOTE_LOCKED(list, hint)	knote(list, hint, 1)
+/*
+ * Flags for knote call
+ */
+#define	KNF_LISTLOCKED	0x0001			/* knlist is locked */
+#define	KNF_NOKQLOCK	0x0002			/* do not keep KQ_LOCK */
+
+#define KNOTE(list, hist, flags)	knote(list, hist, flags)
+#define KNOTE_LOCKED(list, hint)	knote(list, hint, KNF_LISTLOCKED)
 #define KNOTE_UNLOCKED(list, hint)	knote(list, hint, 0)
 
 #define	KNLIST_EMPTY(list)		SLIST_EMPTY(&(list)->kl_list)
@@ -204,7 +210,7 @@
 struct proc;
 struct knlist;
 
-extern void	knote(struct knlist *list, long hint, int islocked);
+extern void	knote(struct knlist *list, long hint, int lockflags);
 extern void	knote_fork(struct knlist *list, int pid);
 extern void	knlist_add(struct knlist *knl, struct knote *kn, int islocked);
 extern void	knlist_remove(struct knlist *knl, struct knote *kn, int islocked);
diff -r a707b4dd87cf src/sys/sys/mount.h
--- a/src/sys/sys/mount.h	Fri Jun 26 13:49:09 2009 +0400
+++ b/src/sys/sys/mount.h	Fri Jun 26 16:45:15 2009 +0400
@@ -633,7 +633,7 @@
 #define VFS_KNOTE_LOCKED(vp, hint) do					\
 {									\
 	if (((vp)->v_vflag & VV_NOKNOTE) == 0)				\
-		VN_KNOTE((vp), (hint), 1);				\
+		VN_KNOTE((vp), (hint), KNF_LISTLOCKED);			\
 } while (0)
 
 #define VFS_KNOTE_UNLOCKED(vp, hint) do					\
diff -r a707b4dd87cf src/sys/sys/vnode.h
--- a/src/sys/sys/vnode.h	Fri Jun 26 13:49:09 2009 +0400
+++ b/src/sys/sys/vnode.h	Fri Jun 26 16:45:15 2009 +0400
@@ -222,9 +222,10 @@
 #define VN_KNOTE(vp, b, a)					\
 	do {							\
 		if (!VN_KNLIST_EMPTY(vp))			\
-			KNOTE(&vp->v_pollinfo->vpi_selinfo.si_note, (b), (a)); \
+			KNOTE(&vp->v_pollinfo->vpi_selinfo.si_note, (b), \
+			    (a) | KNF_NOKQLOCK);		\
 	} while (0)
-#define	VN_KNOTE_LOCKED(vp, b)		VN_KNOTE(vp, b, 1)
+#define	VN_KNOTE_LOCKED(vp, b)		VN_KNOTE(vp, b, KNF_LISTLOCKED)
 #define	VN_KNOTE_UNLOCKED(vp, b)	VN_KNOTE(vp, b, 0)
 
 /*

---------------------------------------------------------------------------------

-- 
Stanislav Sedov
ST4096-RIPE



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20090626164639.71a34f62.stas>