netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).