From: Andi Kleen <ak@suse.de>
To: Stephen Hemminger <shemminger@osdl.org>
Cc: linux-net <linux-net@vger.kernel.org>, netdev@oss.sgi.com
Subject: Re: [RFC] TCP Vegas for 2.6
Date: Mon, 8 Mar 2004 22:21:56 +0100 [thread overview]
Message-ID: <20040308212156.GE26401@wotan.suse.de> (raw)
In-Reply-To: <20040308130454.0442c04d@dell_ss3.pdx.osdl.net>
> +/* Vegas variables */
> + struct {
> + __u32 beg_snd_nxt; /* right edge during last RTT */
> + __u32 beg_snd_una; /* left edge during last RTT */
> + __u32 beg_snd_cwnd; /* saves the size of the cwnd */
> + __u8 do_vegas; /* do vegas for this connection */
> + __u8 doing_vegas_now;/* if true, do vegas for this RTT */
> + __u16 cntRTT; /* # of RTTs measured within last RTT */
> + __u32 minRTT; /* min of RTTs measured within last RTT (in usec) */
> + __u32 baseRTT; /* the min of all Vegas RTT measurements seen (in usec) */
> + } vegas;
How about making this and the experimental westwood stuff CONFIG_*
options? At least for the data structures? Or maybe alternatively allocating
it separately when needed only (this would avoid CONFIG_* options)
I remember when we made jokes about the size of TCBs of other stacks
compared to Linux, but we must have exceeded them all by far. The additional
code is not really a problem, but adding all that bloat to dynamic data
structures adds up quickly when you have a few thousands of them, limiting
scalability.
-Andi
next prev parent reply other threads:[~2004-03-08 21:21 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-03-08 21:04 [RFC] TCP Vegas for 2.6 Stephen Hemminger
2004-03-08 21:21 ` Andi Kleen [this message]
2004-03-08 21:30 ` Stephen Hemminger
2004-03-08 21:36 ` Andi Kleen
2004-03-08 21:45 ` Stephen Hemminger
2004-03-08 23:37 ` David S. Miller
2004-03-09 17:52 ` John Heffner
2004-03-09 18:03 ` Andi Kleen
2004-03-09 18:11 ` John Heffner
2004-03-09 18:22 ` Stephen Hemminger
2004-03-09 18:56 ` Nivedita Singhvi
2004-03-09 19:03 ` Stephen Hemminger
2004-03-09 19:36 ` Nivedita Singhvi
2004-03-09 18:58 ` jamal
2004-03-08 21:51 ` Nivedita Singhvi
2004-03-08 23:31 ` David S. Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20040308212156.GE26401@wotan.suse.de \
--to=ak@suse.de \
--cc=linux-net@vger.kernel.org \
--cc=netdev@oss.sgi.com \
--cc=shemminger@osdl.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).