Re: [PATCH v6 net-next 2/2] tcp: Add Redundant Data Bundling (RDB)

Bill Fink
2016-03-15T00:46:45Z

On Mon, 14 Mar 2016, Yuchung Cheng wrote:

> On Thu, Mar 3, 2016 at 10:06 AM, Bendik Rønning Opstad
> <bro.devel@gmail.com> wrote:
> >
> > Redundant Data Bundling (RDB) is a mechanism for TCP aimed at reducing
> > the latency for applications sending time-dependent data.
...
> > diff --git a/Documentation/networking/ip-sysctl.txt b/Documentation/networking/ip-sysctl.txt
> > index 6a92b15..8f3f3bf 100644
> > --- a/Documentation/networking/ip-sysctl.txt
> > +++ b/Documentation/networking/ip-sysctl.txt
> > @@ -716,6 +716,21 @@ tcp_thin_dpifl_itt_lower_bound - INTEGER
> > calculated, which is used to classify whether a stream is thin.
> > Default: 10000
> >
> > +tcp_rdb - BOOLEAN
> > + Enable RDB for all new TCP connections.
> Please describe RDB briefly, perhaps with a pointer to your paper.
> I suggest have three level of controls:
> 0: disable RDB completely
> 1: enable indiv. thin-stream conn. to use RDB via TCP_RDB socket
> options
> 2: enable RDB on all thin-stream conn. by default
>
> currently it only provides mode 1 and 2. but there may be cases where
> the administrator wants to disallow it (e.g., broken middle-boxes).
>
> > + Default: 0

A per route setting to enable or disable tcp_rdb, overriding
the global setting, could also be useful to the administrator.
Just a suggestion for potential followup work.

-Bill