From owner-cvs-all Sun Jul 7 14:15:36 2002 Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.FreeBSD.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6597337B400; Sun, 7 Jul 2002 14:15:32 -0700 (PDT) Received: from cs.rice.edu (cs.rice.edu [128.42.1.30]) by mx1.FreeBSD.org (Postfix) with ESMTP id 1287A43E54; Sun, 7 Jul 2002 14:15:32 -0700 (PDT) (envelope-from alc@cs.rice.edu) Received: from localhost (localhost [127.0.0.1]) by cs.rice.edu (Postfix) with ESMTP id 497284AA0A; Sun, 7 Jul 2002 16:15:31 -0500 (CDT) Received: by cs.rice.edu (Postfix, from userid 19572) id C5B9C4A9FE; Sun, 7 Jul 2002 16:15:30 -0500 (CDT) Date: Sun, 7 Jul 2002 16:15:30 -0500 From: Alan Cox To: Jake Burkholder Cc: Alan Cox , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/vm vm_zeroidle.c Message-ID: <20020707211530.GZ11391@cs.rice.edu> References: <200207071927.g67JRvwi019399@freefall.freebsd.org> <20020707193742.GX11391@cs.rice.edu> <20020707161105.D14627@locore.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20020707161105.D14627@locore.ca> User-Agent: Mutt/1.3.28i X-Virus-Scanned: by AMaViS snapshot-20010714 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Sun, Jul 07, 2002 at 04:11:06PM -0400, Jake Burkholder wrote: > Apparently, On Sun, Jul 07, 2002 at 02:37:42PM -0500, > Alan Cox said words to the effect of; > > > 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. > > (re sparc64) > > The cache flush call is not MP-safe yet, because the IPI code isn't, > but it won't take much to make it so. To do the zeroing it accesses > physical addresses directly, similar to K0SEG addresses on alpha and > RR7 addresses on ia64, which is MP-safe. We should really try to avoid generating IPI's from vm_page_zero_idle(). Perhaps a thread private KVA ... Alan To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message