Date: Thu, 30 Aug 2012 12:12:54 -0600 From: Ian Lepore <freebsd@damnhippie.dyndns.org> To: Alan Cox <alc@rice.edu> Cc: "arm@freebsd.org" <arm@freebsd.org> Subject: Re: arm pmap locking Message-ID: <1346350374.1140.525.camel@revolution.hippie.lan> In-Reply-To: <503D12AE.1050705@rice.edu> References: <502FD67A.7030003@rice.edu> <1345315508.27688.260.camel@revolution.hippie.lan> <503D12AE.1050705@rice.edu>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 2012-08-28 at 13:49 -0500, Alan Cox wrote: > Can you please retry with the attached patch? For the time being, I > decided to address the above problem by simply enabling recursion on the > new pmap lock. As I mentioned in my prior message, the lock recursion > in the arm pmap is a mistake. However, I'd rather not change two things > at once, i.e., replace the page queues lock and fix the lock recursion. > I'll take a look at eliminating the lock recursion later this week. > > Thanks, > Alan > Sorry for the delay, I finally got around to trying this today, and it seems to be working well initially -- it boots to multiuser and the only difference in the dmesg.boot with and without the patch is the compile date, and the kernel image is 128 bytes smaller with the patch. I've got DIAGNOSTIC and INVARIANTS enabled; I'll run with the patch in place and let you know if anything glitches. -- Ian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1346350374.1140.525.camel>