From owner-freebsd-doc@FreeBSD.ORG Fri Jun 26 03:03:11 2009 Return-Path: Delivered-To: freebsd-doc@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 85B5E1065673; Fri, 26 Jun 2009 03:03:11 +0000 (UTC) (envelope-from kaduk@MIT.EDU) Received: from biscayne-one-station.mit.edu (BISCAYNE-ONE-STATION.MIT.EDU [18.7.7.80]) by mx1.freebsd.org (Postfix) with ESMTP id 3AA3A8FC14; Fri, 26 Jun 2009 03:03:11 +0000 (UTC) (envelope-from kaduk@MIT.EDU) Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103]) by biscayne-one-station.mit.edu (8.13.6/8.9.2) with ESMTP id n5Q2osTc009797; Thu, 25 Jun 2009 22:50:54 -0400 (EDT) Received: from multics.mit.edu (MULTICS.MIT.EDU [18.187.1.73]) (authenticated bits=56) (User authenticated as kaduk@ATHENA.MIT.EDU) by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id n5Q2oqjX008725 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Thu, 25 Jun 2009 22:50:53 -0400 (EDT) Received: (from kaduk@localhost) by multics.mit.edu (8.12.9.20060308) id n5Q2oo5M014076; Thu, 25 Jun 2009 22:50:50 -0400 (EDT) Date: Thu, 25 Jun 2009 22:50:50 -0400 (EDT) From: Benjamin Kaduk To: Oleg Bulyzhin In-Reply-To: <20090625104451.GA273@lath.rinet.ru> Message-ID: References: <200906242257.n5OMv71d032996@svn.freebsd.org> <47d0403c0906241644v70f35ba5r7c6440a45c95c369@mail.gmail.com> <20090625104451.GA273@lath.rinet.ru> User-Agent: Alpine 1.10 (GSO 962 2008-03-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Scanned-By: MIMEDefang 2.42 X-Spam-Flag: NO X-Spam-Score: 0.00 Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, freebsd-doc@FreeBSD.org Subject: Re: svn commit: r194930 - in head: sbin/ipfw sys/netinet sys/netinet/ipfw X-BeenThere: freebsd-doc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Documentation project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 26 Jun 2009 03:03:11 -0000 On Thu, 25 Jun 2009, Oleg Bulyzhin wrote: > On Wed, Jun 24, 2009 at 07:44:04PM -0400, Ben Kaduk wrote: > >> There's a grammar error and a style error, here. I'm actually not >> entirely sure what >> the intended meaning is, so it's a bit hard to fix the grammar error. > > It's no wonder - my english writing skill is poor. > >> Looking at the code, it seems that in burst mode, extra data will be >> allowed to be sent, though >> it is capped to not exceed the pipe bandwidth. >> Perhaps "... bytes of data is allowed to bypass the dummynet scheduler >> (...), though >> the transmission rate will still be capped so as to not exceed the >> pipe's bandwidth."? > > Let me explain. For example, we have pipe with bw 1Mbit/s and burst size is > 5GByte. If we try to download 10Gbyte through this pipe following will happen: > 1) 1st 5Gbyte of data will go with 'wire speed'. > 2) last 5Gbyte will be shaped to 1Mbit/s > > Could you please mail me whole 'burst' part (as it should be), and i will fix > it. I think the following will suffice: .It Cm burst Ar size If the data to be sent exceeds the pipe's bandwidth limit (and the pipe was previously idle), up to .Ar size bytes of data are allowed to bypass the .Nm dummynet scheduler, and will be sent as fast as the physical link allows. Any additional data will be transmitted at the rate specified by the .Nm pipe bandwidth. The burst size depends on how long the pipe has been idle; the effective burst size is calculated as follows: MAX( .Ar size , .Nm bw * pipe_idle_time). .Pp The mdoc police may need to correct my markup. > >> >> The style error is that the new sentence ("Effective burst size ...") >> should start >> on a new line. >> >> I would also prefer to see the new sentence be an actual complete sentence >> (i.e., "The effective burst size is calculated as follows"), though >> there appears >> to be at least one other bug of this form in the file already (see, >> for example, the >> quoted text at the beginning of this hunk: "Default value is no delay.", which >> would benefit from a "the".) >> >> >> Unrelated to this commit, there is a grammar error early in the file: >> >> 312 Once >> 313 .Fl p >> 314 has been specified, any additional arguments as passed on to the preproc >> essor >> 315 for interpretation. >> >> The 'as' in line 314 should be 'are'. >> (This is from CVS r1.220, so the line numbers may not be current.) I will submit a doc PR about the other issues. Thanks for taking care of this! -Ben Kaduk