Date: Mon, 03 Sep 2012 19:44:01 -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: <1346723041.1140.602.camel@revolution.hippie.lan> In-Reply-To: <5045351F.6060201@rice.edu> References: <502FD67A.7030003@rice.edu> <1345315508.27688.260.camel@revolution.hippie.lan> <503D12AE.1050705@rice.edu> <1346350374.1140.525.camel@revolution.hippie.lan> <5045351F.6060201@rice.edu>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 2012-09-03 at 17:54 -0500, Alan Cox wrote: > On 08/30/2012 13:12, Ian Lepore wrote: > > 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. > > > > Could you please test the attached patch? This is a small step toward > disentangling the arm pmap locking. > > Alan > Applied the patch, it's running just fine. -- Ian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1346723041.1140.602.camel>