cvs commit: src/sys/sparc64/sparc64 pmap.c

Alan L. Cox alc at imimic.com
Sat Sep 20 14:30:27 PDT 2003


Jake Burkholder wrote:
> 
> Apparently, On Sat, Sep 20, 2003 at 01:35:01PM -0500,
>         Alan L. Cox said words to the effect of;
> 
> > Jake Burkholder wrote:
> > >
> > > jake        2003/09/20 10:00:59 PDT
> > >
> > >   FreeBSD src repository
> > >
> > >   Modified files:
> > >     sys/sparc64/sparc64  pmap.c
> > >   Log:
> > >   Remove an invalid KASSERT.  Apparently pmap_remove_all gets called on
> > >   unmanaged pages.
> > >
> > >   Revision  Changes    Path
> > >   1.124     +0 -3      src/sys/sparc64/sparc64/pmap.c
> >
> > FYI, this is, perhaps, triggered by a recent change that I made to
> > kmem_malloc(), which declares the allocated pages as unmanaged.
> 
> Ahh, ok, thanks, I wondered.  The i386 pmap has similar assertions but
> they seem to never be compiled.
> 

In case it's not obvious, my rationale was to stop the creation of "pv"
entries for kmem_map (including mbuf-related) pages.  They serve no
purpose.

Regards,
Alan

P.S.  In fact, I would argue that changing the type of the kernel and
kmem vm objects to type PHYS would make a lot of sense.  Today, it
wouldn't have any functional effect, but it better describes the kind of
object they are.


More information about the cvs-src mailing list