Re: [PATCH] tcp: Socket option to set congestion window

Andi Kleen
2010-05-27T07:57:23Z

> Then all the app does is say "I'am in peer id foo" right? Is that really
> that much different from making the setsockopt() call for a different cwnd
> value? Particularly if say the limit were not a global sysctl, but based on
> the existing per-route value (perhaps expanded to have a min, max and
> default?)

The worst case with peer id would be app using an own peer id
for each connection. So each connection would have an own cwnd,
just like today. So the worst case is the same as today.

If it shares connections between peer ids the real effective cwnd
of all those connections would be also never be "worse" (that is
larger) than it could be on single connection.

So this limits the cwnds effectively with peer ids, although it also
gives a nice way to reuse an already existing cwnd for a new
connection (this does not make things worse because in theory
the app could have reused the same connection too)

So overall peer ids don't allow to enlarge cwnds over today.

If the cwnd is fully application controlled all these limits
are not there and a bittorrent client could just always set
it to 1 million.

-Andi

--
ak@linux.intel.com -- Speaking for myself only.