Date: Tue, 16 Jan 2001 17:38:10 -0800 From: Alfred Perlstein <bright@wintelcom.net> To: John Baldwin <jhb@FreeBSD.org> Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org, Bosko Milekic <bmilekic@technokratis.com> Subject: Re: cvs commit: src/sys/kern uipc_mbuf.c Message-ID: <20010116173809.C7240@fw.wintelcom.net> In-Reply-To: <XFMail.010116172437.jhb@FreeBSD.org>; from jhb@FreeBSD.org on Tue, Jan 16, 2001 at 05:24:37PM -0800 References: <20010116171353.A7240@fw.wintelcom.net> <XFMail.010116172437.jhb@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
* John Baldwin <jhb@FreeBSD.org> [010116 17:24] wrote: > > > Your patch doesn't fix the deadlock, it only turns it into a panic. > > A panic is much easier to debug than a deadlock. We don't need to debug it, we know it's going to happen and why. :) > > > A fix would be to DROP_GIANT on entry into a driver, or make kmem_alloc > > not require Giant. > > The latter is preferable, but that would mean locking down vm. Are you > volunteering? :) Hmm, well, there's already a bunch of lockmgr locks protecting the map structures, I'm wondering if we need to lock down the entire thing, or only the map manipulation routines. -- -Alfred Perlstein - [bright@wintelcom.net|alfred@freebsd.org] "I have the heart of a child; I keep it in a jar on my desk." To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010116173809.C7240>