| To: | "David S. Miller" <davem@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] Increase snd/rcv buffers in pppoe |
| From: | Andi Kleen <ak@xxxxxxx> |
| Date: | Wed, 25 Feb 2004 21:15:26 +0100 |
| Cc: | yoshfuji@xxxxxxxxxxxxxx, ak@xxxxxx, netdev@xxxxxxxxxxx, mostrows@xxxxxxxxxxxxxxxxx |
| In-reply-to: | <20040223102613.33838132.davem@redhat.com> |
| References: | <20040223105359.GA91938@colin2.muc.de> <20040223.200101.39143636.yoshfuji@linux-ipv6.org> <20040223111659.GB10681@colin2.muc.de> <20040223.203843.04073965.yoshfuji@linux-ipv6.org> <20040223102613.33838132.davem@redhat.com> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Mon, 23 Feb 2004 10:26:13 -0800
"David S. Miller" <davem@xxxxxxxxxx> wrote:
>
> The proposal I make intends to avoid this endless tweaking.
[...] Sounds good to me.
> Two more observations while grepping for SK_{R,W}MEM_MAX.
>
> 1) IPV4 icmp sents sk_sndbuf of it's sockets to "2 * SK_WMEM_MAX", that's not
> what it really wants. What it really wants is enough space to hold
> ~2 full sized IPV4 packets, roughly 2 * 64K + struct sk_buff overhead
> and thus that is what it should be using there.
Just sk_buff overhead for what MTU? 576? (would be a bit extreme)
And in theory it could be one byte packets too.
> 2) IPV6 icmp does the same as ipv4, except this value is even more wrong there
> especially considering jumbograms. With current code, sending a jumbogram
> ipv6 icmp packet would simply fail, and I wonder if anyone has even tried
> this.
Isn't even ICMPv6 limited to the minimum guaranteed MTU (1000 something) like
ICMPv4 is to
576 bytes?
-Andi
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH 2.4.25] pcnet32.c another diff error fix., Don Fry |
|---|---|
| Next by Date: | Re: [PATCH] Increase snd/rcv buffers in pppoe, David S. Miller |
| Previous by Thread: | Re: [PATCH] Increase snd/rcv buffers in pppoe, David S. Miller |
| Next by Thread: | Re: [PATCH] Increase snd/rcv buffers in pppoe, David S. Miller |
| Indexes: | [Date] [Thread] [Top] [All Lists] |