netdev
[Top] [All Lists]

Re: [BK PATCH] 2.6 SCTP updates

To: Sridhar Samudrala <sri@xxxxxxxxxx>
Subject: Re: [BK PATCH] 2.6 SCTP updates
From: "David S. Miller" <davem@xxxxxxxxxx>
Date: Fri, 13 Feb 2004 21:52:27 -0800
Cc: netdev@xxxxxxxxxxx, lksctp-developers@xxxxxxxxxxxxxxxxxxxxx
In-reply-to: <Pine.LNX.4.58.0402131443350.2190@xxxxxxxxxxxxxxxxxxxxx>
References: <Pine.LNX.4.58.0402131443350.2190@xxxxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
On Fri, 13 Feb 2004 16:07:05 -0800 (PST)
Sridhar Samudrala <sri@xxxxxxxxxx> wrote:

> Please do a
>       bk pull http://linux-lksctp.bkbits.net/lksctp-2.5.work
> to get the following fixes to SCTP on top of linux 2.6.2.

I am pulling this, however I strongly disagree with this change:

> # 04/02/12      sri@xxxxxxxxxx  1.1552
> # [SCTP] Use __get_free_pages() to allocate ssnmap.
> #
> # This is needed to avoid kmalloc()'s 128K limit when an association is
> # initialized with a large no. of streams(more than 65000 inbound +
> # outbound streams).

This is madness because it means that every assosciation created eats
at least PAGE_SIZE bytes of memory, even if the ssnmap is tiny.

Maybe you should just brance between two allocation schemes based upon
what sctp_ssnmap_size() returns.

<Prev in Thread] Current Thread [Next in Thread>