* Kris Kennaway <[EMAIL PROTECTED]> [021118 12:01] wrote:
> On Mon, Nov 18, 2002 at 11:39:01AM -0800, Alfred Perlstein wrote:
>
> > Well now they are, I will investigate as time permits.
>
> Thanks.
>
> > I'm still in a holding pattern about adding more debugging info to
> > lockd now that we're in release candidate mode. I may ask for a
> > branch to be done, but i'm not sure yet.
>
> Feel free to give me non-committed patches to test :-)
GRR, witless always gives me the locations of the correct lock
order, never the locations of the incorrect ones, so here, try this
and give me a backtrace, it should nail you with extreme prejudice
when you get what i think is the wrong order:
Index: pipe.h
===================================================================
RCS file: /home/ncvs/src/sys/sys/pipe.h,v
retrieving revision 1.20
diff -u -r1.20 pipe.h
--- pipe.h 14 Aug 2002 01:04:43 -0000 1.20
+++ pipe.h 18 Nov 2002 20:13:13 -0000
@@ -112,7 +112,11 @@
};
#define PIPE_MTX(pipe) (pipe)->pipe_mtxp
-#define PIPE_LOCK(pipe) mtx_lock(PIPE_MTX(pipe))
+#define PIPE_LOCK(pipe) \
+ do { \
+ mtx_lock(PIPE_MTX(pipe)); \
+ mtx_assert(&sigio_lock, MA_NOTOWNED); \
+ } while (0)
#define PIPE_UNLOCK(pipe) mtx_unlock(PIPE_MTX(pipe))
#define PIPE_LOCK_ASSERT(pipe, type) mtx_assert(PIPE_MTX(pipe), (type))
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
