From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Ricardo Leitner Subject: Re: [patch iproute2 v4 3/3] man: Add -bs option to tc manpage Date: Tue, 2 Jan 2018 18:07:18 -0200 Message-ID: <20180102200718.GB725@localhost.localdomain> References: <20180102142804.27145-1-chrism@mellanox.com> <20180102142804.27145-4-chrism@mellanox.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, gerlitz.or@gmail.com, stephen@networkplumber.org, dsahern@gmail.com To: Chris Mi Return-path: Received: from mx1.redhat.com ([209.132.183.28]:40752 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751143AbeABUHV (ORCPT ); Tue, 2 Jan 2018 15:07:21 -0500 Content-Disposition: inline In-Reply-To: <20180102142804.27145-4-chrism@mellanox.com> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, Jan 02, 2018 at 11:28:04PM +0900, Chris Mi wrote: > Signed-off-by: Chris Mi > --- > man/man8/tc.8 | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/man/man8/tc.8 b/man/man8/tc.8 > index ff071b33..de137e16 100644 > --- a/man/man8/tc.8 > +++ b/man/man8/tc.8 > @@ -601,6 +601,11 @@ must exist already. > read commands from provided file or standard input and invoke them. > First failure will cause termination of tc. > > +.TP > +.BR "\-bs", " \-bs size", " \-batchsize", " \-batchsize size" > +How many commands are accumulated before sending to kernel. > +By default, it is 1. It only takes effect in batch mode. > + You should also describe the limitations it has. Like, it only works for action and filter and that it shouldn't be mixed with other commands. And maybe even do such check in the code: refuse to do other commands if batch_size > 1. > .TP > .BR "\-force" > don't terminate tc on errors in batch mode. > -- > 2.14.3 >