[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <56537198.4000809@gmail.com>
Date: Mon, 23 Nov 2015 21:05:44 +0100
From: "Bendik Rønning Opstad" <bro.devel@...il.com>
To: Eric Dumazet <eric.dumazet@...il.com>,
Bendik Rønning Opstad <bro.devel@...il.com>
Cc: "David S. Miller" <davem@...emloft.net>, netdev@...r.kernel.org,
Yuchung Cheng <ycheng@...gle.com>,
Neal Cardwell <ncardwell@...gle.com>,
Andreas Petlund <apetlund@...ula.no>,
Carsten Griwodz <griff@...ula.no>,
Pål Halvorsen <paalh@...ula.no>,
Jonas Markussen <jonassm@....uio.no>,
Kristian Evensen <kristian.evensen@...il.com>,
Kenneth Klette Jonassen <kennetkl@....uio.no>
Subject: Re: [PATCH RFC v2 net-next 2/2] tcp: Add Redundant Data Bundling
(RDB)
On 23/11/15 18:43, Eric Dumazet wrote:
> On Mon, 2015-11-23 at 17:26 +0100, Bendik Rønning Opstad wrote:
>
>> > +
>> > +tcp_rdb_max_skbs - INTEGER
>> > + Enable restriction on how many previous SKBs in the output queue
>> > + RDB may include data from. A value of 1 will restrict bundling to
>> > + only the data from the last packet that was sent.
>> > + Default: 1
>> > +
> skb is an internal thing. I would rather not expose a sysctl with such
> name.
>
> Can be multi segment or not (if GSO/TSO is enabled)
>
> So even '1' skb can have very different content, from 1 byte to ~64 KB
I see your point about not exposing the internal naming. What about
tcp_rdb_max_packets?
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists