Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 07 Jul 2002 13:50:10 -0700
From:      Peter Wemm <peter@wemm.org>
To:        Alan Cox <alc@cs.rice.edu>
Cc:        Alan Cox <alc@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/vm vm_zeroidle.c 
Message-ID:  <20020707205010.508423808@overcee.wemm.org>
In-Reply-To: <20020707193742.GX11391@cs.rice.edu> 

next in thread | previous in thread | raw e-mail | index | archive | help
Alan Cox wrote:
> On Sun, Jul 07, 2002 at 12:27:57PM -0700, Alan Cox wrote:
> > alc         2002/07/07 12:27:57 PDT
> > 
> >   Modified files:
> >     sys/vm               vm_zeroidle.c 
> >   Log:
> >    o Lock accesses to the free queue(s) in vm_page_zero_idle().
> >   
> >   Revision  Changes    Path
> >   1.10      +4 -0      src/sys/vm/vm_zeroidle.c
> 
> I believe that Giant could be removed from vm_page_zero_idle()
> on alpha and ia64.  i386's pmap_zero_page() is not MPSAFE.
> I'm not sure about sparc64.

I have patches for this too.. :-)  Including a MPSAFE page zero system for
i386.  The problem with SMP though is that I am unsure if the tlb shootdown
system is reentrant.

http://people.freebsd.org/~peter/idlezero.diff

Note that I do not yet have
#ifdef SMP
   mtx_lock(&Giant)
#endif
etc around the tlb shootdown stuff in the sparc64 or x86 functions in this
version.

Cheers,
-Peter
--
Peter Wemm - peter@wemm.org; peter@FreeBSD.org; peter@yahoo-inc.com
"All of this is for nothing if we don't go to the stars" - JMS/B5


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?20020707205010.508423808>