Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 6 Aug 2003 16:22:43 -0400
From:      Andrew Gallatin <gallatin@cs.duke.edu>
To:        Alan Cox <alc@FreeBSD.org>
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/i386/i386 pmap.c
Message-ID:  <20030806162243.A91712@grasshopper.cs.duke.edu>
In-Reply-To: <200308061846.h76IkmYp068237@repoman.freebsd.org>; from alc@FreeBSD.org on Wed, Aug 06, 2003 at 11:46:48AM -0700
References:  <200308061846.h76IkmYp068237@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Alan Cox [alc@FreeBSD.org] wrote:
> alc         2003/08/06 11:46:48 PDT
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/i386/i386        pmap.c 
>   Log:
>   Acquire the page queues lock in pmap_insert_entry().  (I used to believe
>   that the page's busy flag could be relied upon to synchronize access to the
>   pv list.  I don't any longer.  See, for example, the call to
>   pmap_insert_entry() from pmap_copy().)

Any chance this is what's causing the pmap corruption on alpha and ia64?
We're also doing those same 3 lines there:

      TAILQ_INSERT_TAIL(&pmap->pm_pvlist, pv, pv_plist);
      TAILQ_INSERT_TAIL(&m->md.pv_list, pv, pv_list);
      m->md.pv_list_count++;

FWIW,  I've managed to get a j16 buildworld to complete on a July 4
kernel, and fail with a fault-on-nofault-entry panic with a July 10
kernel.


Drew



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