From owner-freebsd-current Fri Dec 14 3: 0: 2 2001 Delivered-To: freebsd-current@freebsd.org Received: from albatross.prod.itd.earthlink.net (albatross.mail.pas.earthlink.net [207.217.120.120]) by hub.freebsd.org (Postfix) with ESMTP id C97DF37B405 for ; Fri, 14 Dec 2001 02:59:55 -0800 (PST) Received: from pool0073.cvx40-bradley.dialup.earthlink.net ([216.244.42.73] helo=mindspring.com) by albatross.prod.itd.earthlink.net with esmtp (Exim 3.33 #1) id 16Eq4H-0001tA-00; Fri, 14 Dec 2001 02:59:53 -0800 Message-ID: <3C19DBAE.D962856A@mindspring.com> Date: Fri, 14 Dec 2001 02:59:58 -0800 From: Terry Lambert X-Mailer: Mozilla 4.7 [en]C-CCK-MCD {Sony} (Win98; U) X-Accept-Language: en MIME-Version: 1.0 To: Peter Wemm Cc: Luigi Rizzo , current@FreeBSD.ORG Subject: Re: Solved (Re: -current vs. -stable network performance) References: <20011214103436.7931438CC@overcee.netplex.com.au> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit 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 Peter Wemm wrote: > > Luigi Rizzo wrote: > [..] > > The change below has been committed to STABLE 7 weeks ago, but did > > not go into CURRENT because there was some disagreement on the > > semantics of M_LEADINGSPACE. However I would strongly vote for > > committing this change to CURRENT as well, given the huge performance > > implications (even if the 21143 were not buggy, not being able to > > write into clusters hurts a lot of pieces of the networking stack). > > Incidently, this is a poster-child example of why fixes are not to go to > -stable first. It leads to exactly this sort of lossage. If we waited for all disagreement about semantics in -current to be resolved, we would all be tripping over our beards before some things would be committed. Is the semantic complain going to be overridden by the performance advantages of not caring about the semantics, only the speed? > rev 1.44.2.11: > ... > This does not go in CURRENT as is: as discussed in -net, > M_LEADINGSPACE should not check for writability, just report > available space, leaving the check to some other piece of code. > Unfortunately, some code in the tree depends on M_LEADINGSPACE > checking for writability, and so implementing M_LEADINGSPACE > in the correct way also requires to fix the incorrect usage. > This is what will be done in CURRENT, but for STABLE, this is > probably more than we want, and so we are happy (more or less) with > this simple fix. > ... > > How about fixing it for real as described in the commit message? Uh... "patches welcome from the people who complained about the writability check"? 8^) 8^)... -- Terry To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message