From owner-freebsd-current Thu Mar 7 14:22:13 2002 Delivered-To: freebsd-current@freebsd.org Received: from fledge.watson.org (fledge.watson.org [204.156.12.50]) by hub.freebsd.org (Postfix) with ESMTP id 3D39A37B400; Thu, 7 Mar 2002 14:22:06 -0800 (PST) Received: from fledge.watson.org (fledge.pr.watson.org [192.0.2.3]) by fledge.watson.org (8.11.6/8.11.5) with SMTP id g27MLMD26038; Thu, 7 Mar 2002 17:21:22 -0500 (EST) (envelope-from robert@fledge.watson.org) Date: Thu, 7 Mar 2002 17:21:21 -0500 (EST) From: Robert Watson X-Sender: robert@fledge.watson.org To: Warner Losh Cc: Julian Elischer , "Justin T. Gibbs" , Matthew Dillon , John Baldwin , Bruce Evans , Terry Lambert , Alfred Perlstein , Bosko Milekic , Seigo Tanimura , FreeBSD current users Subject: Re: Patch for critical_enter()/critical_exit() & interrupt assem In-Reply-To: <200203072143.g27LhaL97112@harmony.village.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Thu, 7 Mar 2002, Warner Losh wrote: > In message Julian Elischer writes: > : > : > : On Thu, 7 Mar 2002, Justin T. Gibbs wrote: > : > > : > Then do the right things so it will. > : > : Unfortunatly that has been proven to not work. > : > : after reverting the change and silently waiting for a week > : 1/ no person bothered to review it. > : 2/ people assumed the patch had gone away. > > Ummm, There are reviews in the archives that object to the API as it > relates to optimization and those objections haven't been sanely > answered with anything more constructive than "BS". The primary objections I've seen from Jake, and he posted them as part of the earlier thread prior to the commit, was that the API changes proposed by Matt don't make sense for the sparc64 implementation, uni-processor or multi-processor, and that while these changes might be appropriate for i386, he wanted to see the APIs set up in such a way that the differences in architectures were hidden in the MD code. This suggests working some more on the API before moving on, and my reading of earlier posts in the thread from John was that that was what he had in mind also. I don't pretend to understand all the issues here, but I think it's important to recognize that there have been several coherrent responses to the current patch that do need to be addressed. I think the preference I've seen from a number of developers is that the be addressed before the commit, rather than after. Robert N M Watson FreeBSD Core Team, TrustedBSD Project robert@fledge.watson.org NAI Labs, Safeport Network Services To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message