From owner-cvs-all Sun Jun 30 2:56:43 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 7BDF337B406; Sun, 30 Jun 2002 02:56:34 -0700 (PDT) Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.86.163]) by mx1.FreeBSD.org (Postfix) with ESMTP id 10D3B43E0A; Sun, 30 Jun 2002 02:56:33 -0700 (PDT) (envelope-from phk@critter.freebsd.dk) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.12.3/8.12.2) with ESMTP id g5U9sK6J062779; Sun, 30 Jun 2002 11:54:21 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Luigi Rizzo Cc: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/i386/isa clock.c src/sys/i386/include param.h src/sys/conf options.i386 In-Reply-To: Your message of "Sun, 30 Jun 2002 02:05:10 PDT." <20020630020510.B79625@iguana.icir.org> Date: Sun, 30 Jun 2002 11:54:20 +0200 Message-ID: <62778.1025430860@critter.freebsd.dk> From: Poul-Henning Kamp 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 In message <20020630020510.B79625@iguana.icir.org>, Luigi Rizzo writes: >On Sun, Jun 30, 2002 at 10:25:25AM +0200, Poul-Henning Kamp wrote: >... >> >> > If someone feels like porting this code to -current, it might be >> >> > a good idea to keep separate per-cpu buffers to avoid the cost >> >> > of protecting against concurrent accesses. >> >> The please explain the "If someone feels like..." to me. It certainly >> reads as if you don't plan to... > >Not in the next 35 seconds, but just because i want first to hear >feedback from our friends in the US. If you want feedback on the idea, you should have posted the patch to arch@ and similar places, not committed it to the tree, and _never_ committed it directly to -stable. >But otherwise, you read too much in that sentence. Why should I >even spend time thinking on how to do it in -current if I had no >plans to commit it ? Well, It could be because you are doing your work in -stable instead of -current ? It could also be because you tried to do the same thing with your polling code ? :-) Quite frankly, I think the stuff you committed may not belong in either of current or stable, it seems of far too limited scope (only i386, no smp etc etc) to me to be generally useful. But no matter its actual qualities or in this case lack thereof, it certainly does not belong in -stable until it has proven its worth in -current. Please back it out. -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message