From owner-cvs-all Sat Jan 9 07:12:05 1999 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id HAA07801 for cvs-all-outgoing; Sat, 9 Jan 1999 07:12:05 -0800 (PST) (envelope-from owner-cvs-all@FreeBSD.ORG) Received: from ren.dtir.qld.gov.au (ns.detir.qld.gov.au [203.46.81.66]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id HAA07792 for ; Sat, 9 Jan 1999 07:12:03 -0800 (PST) (envelope-from syssgm@dtir.qld.gov.au) Received: by ren.dtir.qld.gov.au; id BAA10181; Sun, 10 Jan 1999 01:10:45 +1000 (EST) Received: from ogre.dtir.qld.gov.au(167.123.8.3) by ren.dtir.qld.gov.au via smap (3.2) id xma010179; Sun, 10 Jan 99 01:10:16 +1000 Received: from atlas.dtir.qld.gov.au (atlas.dtir.qld.gov.au [167.123.8.9]) by ogre.dtir.qld.gov.au (8.8.8/8.8.7) with ESMTP id BAA24270; Sun, 10 Jan 1999 01:10:15 +1000 (EST) Received: from nymph.dtir.qld.gov.au (nymph.dtir.qld.gov.au [167.123.10.10]) by atlas.dtir.qld.gov.au (8.8.5/8.8.5) with ESMTP id BAA29996; Sun, 10 Jan 1999 01:10:15 +1000 (EST) Received: from nymph.dtir.qld.gov.au (localhost.dtir.qld.gov.au [127.0.0.1]) by nymph.dtir.qld.gov.au (8.8.8/8.8.7) with ESMTP id BAA24350; Sun, 10 Jan 1999 01:10:13 +1000 (EST) (envelope-from syssgm@nymph.dtir.qld.gov.au) Message-Id: <199901091510.BAA24350@nymph.dtir.qld.gov.au> To: Dmitrij Tejblum cc: cvs-committers@FreeBSD.ORG, syssgm@dtir.qld.gov.au Subject: Re: cvs commit: src/sys/i386/i386 pmap.c References: <199901091430.RAA00524@tejblum.dnttm.rssi.ru> In-Reply-To: <199901091430.RAA00524@tejblum.dnttm.rssi.ru> from Dmitrij Tejblum at "Sat, 09 Jan 1999 17:30:03 +0300" Date: Sun, 10 Jan 1999 01:10:13 +1000 From: Stephen McKay Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk On Saturday, 9th January 1999, Dmitrij Tejblum wrote: >Stephen McKay wrote: >Ahh! That is because I made the patch month or two ago, and "simplified" >it before commit, when already forgot it ;-/ My original patch appears >to be more correct: That's why I never fix anything at the last minute any more. Either it goes in as tested, or I abort the mission. >(btw, I run it on 4 production servers all the >time, and didn't notice degradation...) You must have lots of ram. I think it should cause really suboptimal swapping. >> There's no need for all that removing and inserting anyway. Zap it and >> the bugs will depart for other dark parts of the code. > >Well, I think since the code was added in March 1998 or so as an optimization, >it should make some sense, right? Having pmap_ts_referenced() is an optimization. I think all the inserting and deleting are unnecessary, and just slow it down. For most cases, the list will be back the way it was. In cases where > 4 references are found, the list will be cycled somewhat and that *might* make the next call *slightly* faster, but at the cost of much pissing about. I say toast the inserts and deletes, and the bugs will magically vanish. Stephen. PS My test box has just hung in "thrd_sleep" while swapping like crazy (on a several day old kernel) doing a parallel make world. And I thought I'd have an easy time night... To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message