Re: LC-trie implementation notes

2005-07-05 Thread David S. Miller
Thanks Robert, I've added this to the tree as Documentation/networking/fib_trie.txt - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] part 1/3 : hashed spinlocks in net/ipv4/route.c

2005-07-05 Thread David S. Miller
From: Eric Dumazet <[EMAIL PROTECTED]> Date: Wed, 06 Jul 2005 00:34:14 +0200 > But please check the memset() is there, or else... panic for sure. It got added by patch 2/3. So if I had actually applied the "fixed" version of patch 1/3 which contained the memset(), then patch 2/3 would not have a

LC-trie implementation notes

2005-07-05 Thread Robert Olsson
Dave, Below is an updated (by my colleagues) version of the notes I sent before. it will help to understand the implementation and this even for us. :) Cheers. --ro LC-trie implementation notes. Node types -- leaf

Re: [PATCH] part 1/3 : hashed spinlocks in net/ipv4/route.c

2005-07-05 Thread Eric Dumazet
David S. Miller a écrit : From: Eric Dumazet <[EMAIL PROTECTED]> Date: Wed, 29 Jun 2005 10:30:23 +0200 [NET] hashed spinlocks in net/ipv4/route.c - Locking abstraction - Spinlocks moved out of rt hash table : Less memory (50%) used by rt hash table. it's a win even on UP. - Sizing of spinl

Re: [RESEND] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread David S. Miller
From: Thomas Graf <[EMAIL PROTECTED]> Date: Wed, 6 Jul 2005 00:13:24 +0200 > Useful in combination with classful qdiscs to drop or > temporary disable certain flows, e.g. one could block > specific ds flows with dsmark. > > Unlike the noop qdisc it can be controlled by the user and > statistic ac

Re: help on tg3 polling extension

2005-07-05 Thread David S. Miller
From: "Qinghua(Kevin) Ye" <[EMAIL PROTECTED]> Date: Tue, 5 Jul 2005 16:19:27 -0600 > Compared to NAPI, click polling will disable Nic interrupts during its > operation, even there is no any packets in the rx buffer. This destroys latency if you only recheck the RX buffer using timer interrupts.

Re: [PATCH] part 1/3 : hashed spinlocks in net/ipv4/route.c

2005-07-05 Thread David S. Miller
From: Eric Dumazet <[EMAIL PROTECTED]> Date: Wed, 29 Jun 2005 10:30:23 +0200 > [NET] hashed spinlocks in net/ipv4/route.c > - Locking abstraction > - Spinlocks moved out of rt hash table : Less memory (50%) used by rt hash > table. it's a win even on UP. > - Sizing of spinlocks table depend

Re: [PATCH] [DECNET]: Fix memset overflow on 64bit archs while dumping decnet routing rules

2005-07-05 Thread David S. Miller
Applied, thanks Thomas. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: help on tg3 polling extension

2005-07-05 Thread Qinghua(Kevin) Ye
Click is a modular software router developed by MIT. Since router is a dedicated network traffice processor, polling can be introduced to reduce the overhead of interrupt. Once click runs, it will continuously polling the Nic interface, get packets and process it, send it out from selected interfac

Re: [patch 2/2] include/net/slhc_vj.h: remove __ARGS

2005-07-05 Thread David S. Miller
From: [EMAIL PROTECTED] Date: Thu, 30 Jun 2005 23:59:40 +0200 > I suspect "#define __ARGS(x) ()" was deprecated before I was born. Indeed :) Patch applied, thanks. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo inf

[RESEND] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread Thomas Graf
Useful in combination with classful qdiscs to drop or temporary disable certain flows, e.g. one could block specific ds flows with dsmark. Unlike the noop qdisc it can be controlled by the user and statistic accounting is done. Signed-off-by: Thomas Graf <[EMAIL PROTECTED]> Index: net-2.6/net/s

Re: [PATCH] part 3/3 : Bug fix in rt_check_expire()

2005-07-05 Thread David S. Miller
Also applied, thanks a lot. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [patch 1/2] [KJ] [PATCH 2.4.28 1/1] networking: improve readability of dev_set_promiscuity() in net/core/dev.c

2005-07-05 Thread David S. Miller
From: [EMAIL PROTECTED] Date: Thu, 30 Jun 2005 23:59:39 +0200 > A trivial patch to improve the readability of dev_set_promiscuity() > in net/core/dev.c. New code does exactly the same thing as original > code. Applied, thanks a lot. - To unsubscribe from this list: send the line "unsubscribe netd

Re: [PATCH] shaper: switch to spinlocks

2005-07-05 Thread David S. Miller
From: Christoph Hellwig <[EMAIL PROTECTED]> Date: Thu, 30 Jun 2005 22:12:51 +0200 > you were right and the sleeping locks in shaper were broken. Markus > Kanet noticed this and also tested the patch below that switches > locking to spinlocks. > > Signed-off-by: Christoph Hellwig <[EMAIL PROTECTED

Re: Broken memory allocation in fib_trie

2005-07-05 Thread David S. Miller
From: Robert Olsson <[EMAIL PROTECTED]> Date: Thu, 30 Jun 2005 11:23:12 +0200 > Below a patch to preallocate memory when doing resize of trie (inflate halve) > If preallocations fails it just skips the resize of this tnode for this time. > > The oops we got when killing bgpd (with full routing) i

[PATCH] skge: fix deadlock on nway_reset

2005-07-05 Thread Stephen Hemminger
Cleanup the phy_lock deadlock because of relocking in the nway_reset path. Also, don't need to do irqsave/restore for blink, just excluding bh is good enough. Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> Index: sky2/drivers/net/skge.c

Re: [PATCH] part 2/3 : uses the fancy alloc_large_system_hash() function to allocate the route hash table

2005-07-05 Thread David S. Miller
From: Eric Dumazet <[EMAIL PROTECTED]> Date: Wed, 29 Jun 2005 10:50:28 +0200 >- rt hash table allocated using alloc_large_system_hash() function. Applied. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at

Re: help on tg3 polling extension

2005-07-05 Thread David S. Miller
From: "Qinghua(Kevin) Ye" <[EMAIL PROTECTED]> Date: Tue, 5 Jul 2005 15:38:57 -0600 > I am thinking about extending the tg3 driver to support Click Polling. What is Click Polling, and why would we want it? - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message

Re: [IPV4]: Handle large allocations in fib_trie

2005-07-05 Thread David S. Miller
From: Robert Olsson <[EMAIL PROTECTED]> Date: Wed, 29 Jun 2005 19:19:08 +0200 > Thanks Patrick! > > It looks straight forward and I don't see any problem in my test setup. > I'm using a script to load full Internet routing. I've applied Patrick's patch, thanks a lot everyone. - To unsubscribe fr

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread Thomas Graf
* David S. Miller <[EMAIL PROTECTED]> 2005-07-05 14:35 > From: Thomas Graf <[EMAIL PROTECTED]> > Date: Tue, 5 Jul 2005 23:28:10 +0200 > > > Can we agree on NET_XMIT_CN? The dev_xmit callers handle it the same > > way as NET_XMIT_SUCCESS but we still have a chance to check for this > > behaviour wi

Re: [PATCH] tg3: support for ethtool -C

2005-07-05 Thread David S. Miller
From: "Michael Chan" <[EMAIL PROTECTED]> Date: Wed, 29 Jun 2005 14:48:27 -0700 > [TG3]: support for ethtool -C > > Add support for ethtool -C with verification of user parameters. I've added this patch to my tree, thanks Michael. - To unsubscribe from this list: send the line "unsubscribe netdev

Re: Crash in ip_rcv while booting related to netconsole

2005-07-05 Thread David S. Miller
From: Herbert Xu <[EMAIL PROTECTED]> Date: Thu, 30 Jun 2005 12:38:28 +1000 > This patch makes IPv6 rcv registration happen last during initialisation. Also applied, thanks Herbert. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] Mo

Re: Crash in ip_rcv while booting related to netconsole

2005-07-05 Thread David S. Miller
From: Herbert Xu <[EMAIL PROTECTED]> Date: Thu, 30 Jun 2005 12:37:42 +1000 > Actually you never applied my patch :) That's right. I was hoping someone would fix the Acenic driver, oh well. > Here they are again. This one makes IPv4 ip_rcv registration happen > last in af_inet. Applied, thanks

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread David S. Miller
From: Thomas Graf <[EMAIL PROTECTED]> Date: Tue, 5 Jul 2005 23:28:10 +0200 > Can we agree on NET_XMIT_CN? The dev_xmit callers handle it the same > way as NET_XMIT_SUCCESS but we still have a chance to check for this > behaviour within the parent qdisc. It's not congestion, the packet went in fin

help on tg3 polling extension

2005-07-05 Thread Qinghua(Kevin) Ye
Hi, all, I am thinking about extending the tg3 driver to support Click Polling. Click has its patch on linux kernel, and adds some callback functions and polling variable to the net_device struct(I did this work on 2.6.11 kernel, and on AMD opteron): netdev->polling = 0; // Check if click p

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread Thomas Graf
* David S. Miller <[EMAIL PROTECTED]> 2005-07-05 14:19 > From: jamal <[EMAIL PROTECTED]> > Date: Tue, 05 Jul 2005 11:13:04 -0400 > > > On Tue, 2005-05-07 at 17:03 +0200, Patrick McHardy wrote: > > > > > Sounds reasonable. As you said, blackhole implies NET_XMIT_SUCCESS, > > > so I propose to use

Re: [PATCH 3/4] [PKT_SCHED]: Report rate estimator configuration errors during qdisc allocation

2005-07-05 Thread David S. Miller
From: Thomas Graf <[EMAIL PROTECTED]> Date: Mon, 04 Jul 2005 13:48:11 +0200 > Current behaviour is to not report an error if a rate > estimator is created together with a qdisc and the > configuration of the rate estimator is bogus. This leads > to unexpected behaviour because the user is not noti

Re: [PATCH 1/4] [PKT_SCHED]: Move sch_generic.c prototypes to correct header file

2005-07-05 Thread David S. Miller
Applied. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread David S. Miller
From: jamal <[EMAIL PROTECTED]> Date: Tue, 05 Jul 2005 11:13:04 -0400 > On Tue, 2005-05-07 at 17:03 +0200, Patrick McHardy wrote: > > > Sounds reasonable. As you said, blackhole implies NET_XMIT_SUCCESS, > > so I propose to use this as default. > > > > Actually i am not sure it makes sense to h

Re: [PATCH 2/4] [PKT_SCHED]: Cleanup qdisc creation and alignment macros

2005-07-05 Thread David S. Miller
From: Thomas Graf <[EMAIL PROTECTED]> Date: Mon, 04 Jul 2005 13:48:10 +0200 > Adds qdisc_alloc() to share code between qdisc_create() > and qdisc_create_dflt(). Hides the qdisc alignment behind > macros and makes use of them. Applied. - To unsubscribe from this list: send the line "unsubscribe ne

Re: [PATCH 2/2] [NET]: Reduce size of sk_buff by 4 bytes

2005-07-05 Thread David S. Miller
From: Thomas Graf <[EMAIL PROTECTED]> Date: Mon, 04 Jul 2005 13:29:25 +0200 > Reduce local_df to a bit field and ip_summed to a 2 bits > field thus saving 13 bits. Move bit fields, packet type, > and protocol into the spare area between the priority > and the destructor. Saves 4 bytes on both, 32b

Re: [PATCH 1/2] [NET]: Remove unused security member in sk_buff

2005-07-05 Thread David S. Miller
Applied. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: e1000 82541 tx unit hang

2005-07-05 Thread Ganesh Venkatesan
Emanuil: Could you send me 'ethtool -S' and 'ethtool -k' output after the failure? ganesh. On 7/5/05, Emanuil Christov <[EMAIL PROTECTED]> wrote: > Hi > i have got troubles with Intel 82541 NIC with kernel 2.6.11.2, 2.6.12.2, > 2.6.13-rc1-mm1, > 2.4.27, 2.4.30, 2.4.31 both with acpi enabled or

Msi-X on Opterons?

2005-07-05 Thread Leonid Grossman
Did anyone had any luck getting MSI-X working on Opteron platforms? The newer 8132-based systems support MSI-X, but we could not get it working with our card there. It works fine on Xeon systems, I wonder if the implementation is Xeon-centric... - To unsubscribe from this list: send the line "unsu

[PATCH] skge: doesn't support sk-9exx

2005-07-05 Thread Stephen Hemminger
One of the devices listed was a Yukon2 based board, so the skge driver wouldn't work for that type. Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> Index: sky2/drivers/net/skge.c === --- sky2.orig/drivers/net/skge.c +++ sky2/dr

[PATCH] skge: yukon_stop typo

2005-07-05 Thread Stephen Hemminger
Here is a fix for a typo, thanks Eliot. Since transmitter not active when device is down, it wasn't really noticed. Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> Index: sky2/drivers/net/skge.c === --- sky2.orig/drivers/net/sk

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread jamal
On Tue, 2005-05-07 at 17:03 +0200, Patrick McHardy wrote: > Sounds reasonable. As you said, blackhole implies NET_XMIT_SUCCESS, > so I propose to use this as default. > Actually i am not sure it makes sense to have anything but NET_XMIT_SUCCESS. Its a damn blackhole ;-> What else could it be doi

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread Patrick McHardy
Thomas Graf wrote: * Patrick McHardy <[EMAIL PROTECTED]> 2005-07-05 16:06 It has side-effects on locally originated TCP connections. You should probably return NET_XMIT_SUCCESS. I don't see the notification as a negative side-effect, it is intented. It sure depends on the actual use of this b

e1000 82541 tx unit hang

2005-07-05 Thread Emanuil Christov
Hi i have got troubles with Intel 82541 NIC with kernel 2.6.11.2, 2.6.12.2, 2.6.13-rc1-mm1, 2.4.27, 2.4.30, 2.4.31 both with acpi enabled or disabled when running rsync with 400-450mbit/s i've got an errors and server become unaccesible from network. lspci -v -s 07:04.0 :07:04.0 Ethernet co

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread Thomas Graf
* Patrick McHardy <[EMAIL PROTECTED]> 2005-07-05 16:06 > Thomas Graf wrote: > >* jamal <[EMAIL PROTECTED]> 2005-07-05 09:04 > > > >>BTW, enqueue returning the code by calling the qdisc::drop() - is that > >>gonna cause some issues? > > > >It returns NET_XMIT_DROP which is not really black hole beha

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread Patrick McHardy
Thomas Graf wrote: * jamal <[EMAIL PROTECTED]> 2005-07-05 09:04 BTW, enqueue returning the code by calling the qdisc::drop() - is that gonna cause some issues? It returns NET_XMIT_DROP which is not really black hole behaviour but should never produce side effects. It has side-effects on loc

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread Thomas Graf
* jamal <[EMAIL PROTECTED]> 2005-07-05 09:04 > On Tue, 2005-05-07 at 14:30 +0200, Thomas Graf wrote: > > > > > It is really argueable, my point is that it adds quite some flexibility > > at no cost and I don't want to miss it anymore. ;-> Having a way to > > temporary or permantely "disable" a le

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread jamal
On Tue, 2005-05-07 at 14:30 +0200, Thomas Graf wrote: > > It is really argueable, my point is that it adds quite some flexibility > at no cost and I don't want to miss it anymore. ;-> Having a way to > temporary or permantely "disable" a leaf qdisc seems right to me. Ok, but not for the example

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread Thomas Graf
* jamal <[EMAIL PROTECTED]> 2005-07-05 08:09 > On Mon, 2005-04-07 at 15:49 +0200, Thomas Graf wrote: > > Assume you have a classful setup with per client classes and you > > want to disable one such class for a moment (maybe the customer > > didn't pay the bills ;->). A simple qdisc replace to a bl

Re: [PATCH 4/4] [PKT_SCHED]: Blackhole queueing discipline

2005-07-05 Thread jamal
On Mon, 2005-04-07 at 15:49 +0200, Thomas Graf wrote: > * jamal <[EMAIL PROTECTED]> 2005-07-04 09:16 > > > > Why not just "fix" noop qdisc to have those features? Note attaching a > > filter/drop action should give you the same functionality. > > I like noop as-is, i.e. one copy shared by all use