Re: Usermode per-flow selection of congestion control algorithm
On 3/31/06, Angelo P. Castellani <angelo.castellani+iperf --at-- gmail.com> wrote:
> On 3/30/06, Ian McDonald <imcdnzl --at-- gmail.com> wrote:
> >
> >
> > After having a bit more of a look at the code I would suggest the
> > following:
> > - Don't use #define TCP_CONGESTION as then you have to set/unset. I
> > would suggest you don't need this as person will only use -A when
> > needed. You do read the default congestion control though based on
> > this. I would suggest that this is not needed as then the code will
> > work on any OS directly without #define. If a user wants to know which
> > congestion control they are using they can look at the sysctl.
>
>
> this is done to make sure that the congestion control algorithm requested
> was actually loaded correctly
>
> i've manually defined the TCP_CONGESTION to not mess up my system headers,
> if that isn't defined this patch makes no sense at all (there is no kernel
> interface to get/set cc algorithm)
What I actually meant was in the main body of the code rather than in
the headers. I've removed in my version I'm about to post.
>
> - put some text in the usage help in src/Locale.c to explain the option
>
>
I have also modified code around reading getsockopt as if there is not
an algorithm selected by setsockopt earlier then an empty string is
returned. I've changed this to then print default. WIll also help on
other platforms I suspect.
--
Ian McDonald
Web: http://wand.net.nz/~iam4
Blog: http://imcdnzl.blogspot.com
WAND Network Research Group
Department of Computer Science
University of Waikato
New Zealand