From owner-cvs-all Wed Jun 10 22:51:15 1998 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id WAA26953 for cvs-all-outgoing; Wed, 10 Jun 1998 22:51:15 -0700 (PDT) (envelope-from owner-cvs-all@FreeBSD.ORG) Received: from spinner.netplex.com.au (spinner.netplex.com.au [202.12.86.3]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id WAA26939; Wed, 10 Jun 1998 22:51:06 -0700 (PDT) (envelope-from peter@netplex.com.au) Received: from spinner.netplex.com.au (localhost [127.0.0.1]) by spinner.netplex.com.au (8.8.8/8.8.8/Spinner) with ESMTP id NAA10253; Thu, 11 Jun 1998 13:50:41 +0800 (WST) (envelope-from peter@spinner.netplex.com.au) Message-Id: <199806110550.NAA10253@spinner.netplex.com.au> X-Mailer: exmh version 2.0.2 2/24/98 To: Julian Elischer cc: Bruce Evans , cvs-committers@FreeBSD.ORG, julian@FreeBSD.ORG Subject: Re: cvs commit: src/sys/ufs/ffs ffs_vnops.c In-reply-to: Your message of "Wed, 10 Jun 1998 22:27:37 MST." Date: Thu, 11 Jun 1998 13:50:40 +0800 From: Peter Wemm Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk Julian Elischer wrote: > And your point is? While I'm slightly shocked to be agreeing with Bruce on a pedantic issue for a change :-), this really should have been done as two commits. The first should have been the backout of rev 1.46, the second should have been the other changes you made, including some sort of nutshell description of what was done. It's a heck of a lot easier to figure out what's going on that way. > On Thu, 11 Jun 1998, Bruce Evans wrote: > > > > Modified files: > > > sys/ufs/ffs ffs_vnops.c > > > Log: > > > Back out John's changes 1.45 -> 1.46 > > > Kirk confirms that the original semantic was what he wanted... > > > (well, a very slight difference) > > > May fix "dangling deps" panic with soft updates. > > > > > > Revision Changes Path > > > 1.50 +17 -21 src/sys/ufs/ffs/ffs_vnops.c > > > > A back out would have been +12 -17. > > > > It also seems to change the semantic to "wait for v_numoutput even in > > the soft updates case", and fix some style bugs, and break the > > optimization of using vfs_bio_awrite() instead of bawrite() for async > > writes of indirect blocks. > > > > Bruce > > > Cheers, -Peter -- Peter Wemm Netplex Consulting To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message