RE: raw PF_PACKET protocol selection

2007-10-09 Thread Joakim Tjernlund
-Original Message- From: Herbert Xu [mailto:[EMAIL PROTECTED] Sent: den 9 oktober 2007 05:17 To: [EMAIL PROTECTED] Cc: netdev@vger.kernel.org Subject: Re: raw PF_PACKET protocol selection Joakim Tjernlund [EMAIL PROTECTED] wrote: I trying to open my own raw PF_PACKET

Re: [RFC][PATCH 1/2] TCP: fix lost retransmit detection

2007-10-09 Thread TAKANO Ryousei
From: Ilpo Järvinen [EMAIL PROTECTED] Subject: Re: [RFC][PATCH 1/2] TCP: fix lost retransmit detection Date: Mon, 8 Oct 2007 14:11:55 +0300 (EEST) On Sun, 7 Oct 2007, TAKANO Ryousei wrote: From: Ilpo Järvinen [EMAIL PROTECTED] Subject: Re: [RFC][PATCH 1/2] TCP: fix lost retransmit

[PATCH net-2.6.24][trivial] fix inconsistency of terms

2007-10-09 Thread TAKANO Ryousei
Fix inconsistency of terms: 1) D-SACK 2) F-RTO Signed-off-by: Ryousei Takano [EMAIL PROTECTED] --- Documentation/networking/ip-sysctl.txt |6 +++--- net/ipv4/tcp_input.c | 16 2 files changed, 11 insertions(+), 11 deletions(-) diff --git

Re: [PATCH 4/8][BNX2X] resubmit as attachments: bnx2x_hsi.h

2007-10-09 Thread Chris Wedgwood
On Mon, Oct 08, 2007 at 06:16:17PM +0200, Eliezer Tamir wrote: bnx2x_hsi.h - machine generated hardware and microcode definitions over 7000 lines... how much of this is really needed? - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED]

[PATCH][TCP] tcp_vegas.c: (tcp_vegas_cong_avoid) fix a bug in disabling slow start by gamma parameter

2007-10-09 Thread Xiaoliang (David) Wei
TCP Vegas implementation has a bug in the process of disabling slow-start with gamma parameter. The bug may lead to extreme unfairness in the presence of early packet loss. See details in: http://www.cs.caltech.edu/~weixl/technical/ns2linux/known_linux/index.html#vegas Switch the order of if

Re: raw PF_PACKET protocol selection

2007-10-09 Thread Evgeniy Polyakov
On Tue, Oct 09, 2007 at 08:08:22AM +0200, Joakim Tjernlund ([EMAIL PROTECTED]) wrote: Your program works fine here. You did run it as root, right? Yes and ETH_P_ALL is the only protocol that prints anything I am on 2.6.22 ETH_P_ARP works too. Did you try stracing it? Just did and

[PATCH] [IPV6] Defer IPv6 device initialization until a valid qdisc is specified

2007-10-09 Thread Mitsuru Chinen
To judge the timing for DAD, netif_carrier_ok() is used. However, there is a possibility that dev-qdisc stays noop_qdisc even if netif_carrier_ok() returns true. In that case, DAD NS is not sent out. We need to defer the IPv6 device initialization until a valid qdisc is specified. Signed-off-by:

Re: [ofa-general] Re: [PATCH V6 0/9] net/bonding: ADD IPoIB support for the bonding driver

2007-10-09 Thread Moni Shoua
Jay Vosburgh wrote: Jeff Garzik [EMAIL PROTECTED] wrote: Moni Shoua wrote: Jay Vosburgh wrote: ACK patches 3 - 9. Roland, are you comfortable with the IB changes in patches 1 and 2? Jeff, when Roland acks patches 1 and 2, please apply all 9. -J Hi Jeff, Roland acked the

Re: raw PF_PACKET protocol selection

2007-10-09 Thread Joakim Tjernlund
On Tue, 2007-10-09 at 11:13 +0400, Evgeniy Polyakov wrote: On Tue, Oct 09, 2007 at 08:08:22AM +0200, Joakim Tjernlund ([EMAIL PROTECTED]) wrote: Your program works fine here. You did run it as root, right? Yes and ETH_P_ALL is the only protocol that prints anything I am on 2.6.22

Re: raw PF_PACKET protocol selection

2007-10-09 Thread Evgeniy Polyakov
On Tue, Oct 09, 2007 at 09:27:38AM +0200, Joakim Tjernlund ([EMAIL PROTECTED]) wrote: Did you change eth_type_trans() to catch your proto? Just fond out something: if I redirect my prog like so: ./sniff log and press Ctrl-C after a packet has been sent to it, it does NOT work. I

Re: raw PF_PACKET protocol selection

2007-10-09 Thread Joakim Tjernlund
On Tue, 2007-10-09 at 11:34 +0400, Evgeniy Polyakov wrote: On Tue, Oct 09, 2007 at 09:27:38AM +0200, Joakim Tjernlund ([EMAIL PROTECTED]) wrote: Did you change eth_type_trans() to catch your proto? Just fond out something: if I redirect my prog like so: ./sniff log and press

Re: raw PF_PACKET protocol selection

2007-10-09 Thread Herbert Xu
On Tue, Oct 09, 2007 at 09:27:38AM +0200, Joakim Tjernlund wrote: Just fond out something: if I redirect my prog like so: ./sniff log and press Ctrl-C after a packet has been sent to it, it does NOT work. I don't get ANY output in my log file, not even the printf(-\n) appears.

Re: [PATCHES] TX batching

2007-10-09 Thread Krishna Kumar2
J Hadi Salim [EMAIL PROTECTED] wrote on 10/08/2007 07:35:20 PM: I dont see something from Krishna's approach that i can take and reuse. This maybe because my old approaches have evolved from the same path. There is a long list but as a sample: i used to do a lot more work while holding the

Re: raw PF_PACKET protocol selection

2007-10-09 Thread Evgeniy Polyakov
On Tue, Oct 09, 2007 at 09:51:25AM +0200, Joakim Tjernlund ([EMAIL PROTECTED]) wrote: On Tue, 2007-10-09 at 11:34 +0400, Evgeniy Polyakov wrote: On Tue, Oct 09, 2007 at 09:27:38AM +0200, Joakim Tjernlund ([EMAIL PROTECTED]) wrote: Did you change eth_type_trans() to catch your proto?

Re: raw PF_PACKET protocol selection

2007-10-09 Thread Joakim Tjernlund
On Tue, 2007-10-09 at 12:17 +0400, Evgeniy Polyakov wrote: On Tue, Oct 09, 2007 at 09:51:25AM +0200, Joakim Tjernlund ([EMAIL PROTECTED]) wrote: On Tue, 2007-10-09 at 11:34 +0400, Evgeniy Polyakov wrote: On Tue, Oct 09, 2007 at 09:27:38AM +0200, Joakim Tjernlund ([EMAIL PROTECTED])

net-2.6.24 rebased...

2007-10-09 Thread David Miller
The TCP bug fix that went into Linus's tree today created some conflicts with net-2.6.24, so I rebased: kernel.org:/pub/scm/linux/kernel/git/davem/net-2.6.24.git I took the opportunity to integrate fix patches into the patches which introduced those bugs or build failures. BTW, when

[PATCH][NET-2.6.24] Remove double dev-flags checking when calling dev_close()

2007-10-09 Thread Pavel Emelyanov
The unregister_netdevice() and dev_change_net_namespace() both check for dev-flags to be IFF_UP before calling the dev_close(), but the dev_close() checks for IFF_UP itself, so remove those unneeded checks. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] --- diff --git a/net/core/dev.c

RE: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Krishna Kumar2
Hi Peter, Waskiewicz Jr, Peter P [EMAIL PROTECTED] wrote on 10/09/2007 04:03:42 AM: true, that needs some resolution. Heres a hand-waving thought: Assuming all packets of a specific map end up in the same qdiscn queue, it seems feasible to ask the qdisc scheduler to give us enough

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread David Miller
From: Krishna Kumar2 [EMAIL PROTECTED] Date: Tue, 9 Oct 2007 16:28:27 +0530 Isn't it enough that the multiqueue+batching drivers handle skbs belonging to different queue's themselves, instead of qdisc having to figure that out? This will reduce costs for most skbs that are neither batched nor

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Krishna Kumar2
Hi Dave, David Miller [EMAIL PROTECTED] wrote on 10/09/2007 04:32:55 PM: Isn't it enough that the multiqueue+batching drivers handle skbs belonging to different queue's themselves, instead of qdisc having to figure that out? This will reduce costs for most skbs that are neither batched

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Krishna Kumar2
David Miller [EMAIL PROTECTED] wrote on 10/09/2007 04:32:55 PM: Ignore LLTX, it sucks, it was a big mistake, and we will get rid of it. Great, this will make life easy. Any idea how long that would take? It seems simple enough to do. thanks, - KK - To unsubscribe from this list: send the

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread David Miller
From: Krishna Kumar2 [EMAIL PROTECTED] Date: Tue, 9 Oct 2007 16:51:14 +0530 David Miller [EMAIL PROTECTED] wrote on 10/09/2007 04:32:55 PM: Ignore LLTX, it sucks, it was a big mistake, and we will get rid of it. Great, this will make life easy. Any idea how long that would take? It

Re: [PATCH][NET-2.6.24] Remove double dev-flags checking when calling dev_close()

2007-10-09 Thread Jeff Garzik
Pavel Emelyanov wrote: The unregister_netdevice() and dev_change_net_namespace() both check for dev-flags to be IFF_UP before calling the dev_close(), but the dev_close() checks for IFF_UP itself, so remove those unneeded checks. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] --- diff

Re: [PATCH][NET-2.6.24] Remove double dev-flags checking when calling dev_close()

2007-10-09 Thread Pavel Emelyanov
Jeff Garzik wrote: Pavel Emelyanov wrote: The unregister_netdevice() and dev_change_net_namespace() both check for dev-flags to be IFF_UP before calling the dev_close(), but the dev_close() checks for IFF_UP itself, so remove those unneeded checks. Signed-off-by: Pavel Emelyanov [EMAIL

[PATCH][NETNS] Don't memset() netns to zero manually

2007-10-09 Thread Pavel Emelyanov
The newly created net namespace is set to 0 with memset() in setup_net(). The setup_net() is also called for the init_net_ns(), which is zeroed naturally as a global var. So remove this memset and allocate new nets with the kmem_cache_zalloc(). Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED]

Re: [RFC][PATCH 1/2] TCP: fix lost retransmit detection

2007-10-09 Thread Ilpo Järvinen
On Tue, 9 Oct 2007, TAKANO Ryousei wrote: From: Ilpo Järvinen [EMAIL PROTECTED] Subject: Re: [RFC][PATCH 1/2] TCP: fix lost retransmit detection Date: Mon, 8 Oct 2007 14:11:55 +0300 (EEST) On Sun, 7 Oct 2007, TAKANO Ryousei wrote: From: Ilpo Järvinen [EMAIL PROTECTED] Just

[RFC PATCH net-2.6.24 0/3]: Attempt to fix lost_retrans brokeness

2007-10-09 Thread Ilpo Järvinen
Lost_retrans handling of sacktag was found to be flawed, two problems that were found have an intertwined solution. Fastpath problem has existed since hints got added and the other problem has probably been there even longer than that. ...This change may add non-trivial processing cost. Initial

[PATCH] [TCP]: Separate lost_retrans loop into own function

2007-10-09 Thread Ilpo Järvinen
Follows own function for each task principle, this is really somewhat separate task being done in sacktag. Also reduces indentation. In addition, added ack_seq local var to break some long lines fixed coding style things. Signed-off-by: Ilpo Järvinen [EMAIL PROTECTED] --- net/ipv4/tcp_input.c

[RFC PATCH] [TCP]: Fix lost_retrans loop vs fastpath problems

2007-10-09 Thread Ilpo Järvinen
Detection implemented with lost_retrans must work also when fastpath is taken, yet most of the queue is skipped including (very likely) those retransmitted skb's we're interested in. This problem appeared when the hints got added, which removed a need to always walk over the whole write queue

[RFC PATCH] [TCP]: Limit processing lost_retrans loop to work-to-do cases

2007-10-09 Thread Ilpo Järvinen
This addition of lost_retrans_low to tcp_sock might be unnecessary, it's not clear how often lost_retrans worker is executed when there wasn't work to do. Cc: TAKANO Ryousei [EMAIL PROTECTED] Signed-off-by: Ilpo Järvinen [EMAIL PROTECTED] --- include/linux/tcp.h |2 ++ net/ipv4/tcp_input.c

[PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread Pavel Emelyanov
Currently indexes for netdevices come sequentially one by one, and the same stays true even for devices that are created for namespaces. Side effects of this are: * lo device has not 1 index in a namespace. This may break some userspace that relies on it (and AFAIR something really broke

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Jeff Garzik
David Miller wrote: From: Krishna Kumar2 [EMAIL PROTECTED] Date: Tue, 9 Oct 2007 16:51:14 +0530 David Miller [EMAIL PROTECTED] wrote on 10/09/2007 04:32:55 PM: Ignore LLTX, it sucks, it was a big mistake, and we will get rid of it. Great, this will make life easy. Any idea how long that

ax88796: add superh to kconfig depencencies

2007-10-09 Thread Magnus Damm
ax88796: add superh to kconfig depencencies This patch adds sh architecture support to the ax88796 kconfig. Signed-off-by: Magnus Damm [EMAIL PROTECTED] --- This is a broken out version of the larger patch recently posted to netdev:

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Herbert Xu
On Tue, Oct 09, 2007 at 08:44:25AM -0400, Jeff Garzik wrote: David Miller wrote: I can just threaten to do them all and that should get the driver maintainers going :-) What, like this? :) Awsome :) -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Jeff Garzik
Herbert Xu wrote: On Tue, Oct 09, 2007 at 08:44:25AM -0400, Jeff Garzik wrote: David Miller wrote: I can just threaten to do them all and that should get the driver maintainers going :-) What, like this? :) Awsome :) Note my patch is just to get the maintainers going. :) I'm not going

Re: [RFC PATCH net-2.6.24 0/3]: Attempt to fix lost_retrans brokeness

2007-10-09 Thread Ilpo Järvinen
On Tue, 9 Oct 2007, Ilpo Järvinen wrote: Lost_retrans handling of sacktag was found to be flawed, two problems that were found have an intertwined solution. Fastpath problem has existed since hints got added and the other problem has probably been there even longer than that. ...This change

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread jamal
On Tue, 2007-09-10 at 08:39 +0530, Krishna Kumar2 wrote: Driver might ask for 10 and we send 10, but LLTX driver might fail to get lock and return TX_LOCKED. I haven't seen your code in greater detail, but don't you requeue in that case too? For others drivers that are non-batching and LLTX,

Re: [PATCHES] TX batching

2007-10-09 Thread jamal
On Tue, 2007-09-10 at 13:44 +0530, Krishna Kumar2 wrote: My feeling is that since the approaches are very different, My concern is the approaches are different only for short periods of time. For example, I do requeueing, have xmit_win, have -end_xmit, do batching from core etc; if you see

Re: [PATCH][E1000E] some cleanups

2007-10-09 Thread jamal
On Mon, 2007-08-10 at 15:40 -0700, Kok, Auke wrote: My biggest problem with the patch as you sent it that it's a tonload of changes and no implicit benefit immediately as I can see. The patch looks scary but is pretty tame when you apply it and stare at it. I would really have to see

Re: [RFC][NET_SCHED] explict hold dev tx lock

2007-10-09 Thread jamal
On Tue, 2007-09-10 at 12:00 +0800, Herbert Xu wrote: OK, after waking up a bit more me too;- What I'm worried about is would we see worse behaviour with drivers that do all their TX clean-up with the TX lock held Good point Herbert. When i looked around i only found one driver that

Re: [ofa-general] Re: [PATCH RFC] RDMA/CMA: Allocate PS_TCP ports from the host TCP port space.

2007-10-09 Thread James Lentini
On Mon, 8 Oct 2007, Steve Wise wrote: The correct solution, IMO, is to enhance the core low level 4-tuple allocation services to be more generic (eg: not be tied to a struct sock). Then the host tcp stack and the host rdma stack can allocate TCP/iWARP ports/4tuples from this common

Re: ax88796: add superh to kconfig depencencies

2007-10-09 Thread Paul Mundt
On Tue, Oct 09, 2007 at 09:51:38PM +0900, Magnus Damm wrote: ax88796: add superh to kconfig depencencies This patch adds sh architecture support to the ax88796 kconfig. Signed-off-by: Magnus Damm [EMAIL PROTECTED] Acked-by: Paul Mundt [EMAIL PROTECTED] - To unsubscribe from this list: send

[PATCH] ipv4: kernel panic when only one unsecured port available

2007-10-09 Thread Anton Arapov
Steps to reproduce: Server: [EMAIL PROTECTED] ~]# cat /etc/exports /export *(ro,insecure) // there is insecure ... I am using ports like 1024 to 61000 [EMAIL PROTECTED] ~] service nfs restart Client: [EMAIL PROTECTED] ~]# echo 32768 32768 /proc/sys/net/ipv4/ip_local_port_range 32768

[0/7] IPsec: More work on async crypto on output

2007-10-09 Thread Herbert Xu
Hi Dave: Here's another round of patches which ends with the moving down of the state lock into x-type-output. I need to stop getting distracted by fixing bugs and concentrate on creating them :) Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL

[PATCH 1/7] [IPSEC]: Remove bogus ref count in xfrm_secpath_reject

2007-10-09 Thread Herbert Xu
[IPSEC]: Remove bogus ref count in xfrm_secpath_reject Constructs of the form xfrm_state_hold(x); foo(x); xfrm_state_put(x); tend to be broken because foo is either synchronous where this is totally unnecessary or if foo is asynchronous then the reference count is in the

[PATCH 2/7] [IPSEC]: Store IPv6 nh pointer in mac_header on output

2007-10-09 Thread Herbert Xu
[IPSEC]: Store IPv6 nh pointer in mac_header on output Current the x-mode-output functions store the IPv6 nh pointer in the skb network header. This is inconvenient because the network header then has to be fixed up before the packet can leave the IPsec stack. The mac header field is unused on

[PATCH 3/7] [IPSEC]: Remove gratuitous km wake-up events on ACQUIRE

2007-10-09 Thread Herbert Xu
[IPSEC]: Remove gratuitous km wake-up events on ACQUIRE There is no point in waking people up when creating/updating larval states because they'll just go back to sleep again as larval states by definition cannot be found by xfrm_state_find. We should only wake them up when the larvals mature or

[PATCH 4/7] [IPSEC]: Move common code into xfrm_alloc_spi

2007-10-09 Thread Herbert Xu
[IPSEC]: Move common code into xfrm_alloc_spi This patch moves some common code that conceptually belongs to the xfrm core from af_key/xfrm_user into xfrm_alloc_spi. In particular, the spin lock on the state is now taken inside xfrm_alloc_spi. Previously it also protected the construction of the

[PATCH 6/7] [IPSEC]: Lock state when copying non-atomic fields to user-space

2007-10-09 Thread Herbert Xu
[IPSEC]: Lock state when copying non-atomic fields to user-space This patch adds locking so that when we're copying non-atomic fields such as life-time or coaddr to user-space we don't get a partial result. For af_key I've changed every instance of pfkey_xfrm_state2msg apart from expiration

[PATCH 5/7] [XFRM] user: Move attribute copying code into copy_to_user_state_extra

2007-10-09 Thread Herbert Xu
[XFRM] user: Move attribute copying code into copy_to_user_state_extra Here's a good example of code duplication leading to code rot. The notification patch did its own netlink message creation for xfrm states. It duplicated code that was already in dump_one_state. Guess what, the next time

[PATCH 7/7] [IPSEC]: Move state lock into x-type-output

2007-10-09 Thread Herbert Xu
[IPSEC]: Move state lock into x-type-output This patch releases the lock on the state before calling x-type-output. It also adds the lock to the spots where they're currently needed. Most of those places (all except mip6) are expected to disappear with async crypto. Signed-off-by: Herbert Xu

Re: [PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread Daniel Lezcano
Pavel Emelyanov wrote: Currently indexes for netdevices come sequentially one by one, and the same stays true even for devices that are created for namespaces. Side effects of this are: * lo device has not 1 index in a namespace. This may break some userspace that relies on it (and AFAIR

Re: [RESEND]: RNDIS host: adjust MTU instead of refusing to talk to devices with MTU 1.5k

2007-10-09 Thread Thomas Sailer
On Tue, 2007-10-09 at 08:18 -0700, David Brownell wrote: This patch makes the host RNDIS driver talk to RNDIS devices with an MTU less than 1.5k, instead of refusing to talk to such a device. Please apply. Signed-Off-by: Thomas Sailer [EMAIL PROTECTED] Acked-by: David Brownell [EMAIL

[PATCH 0/6][NET-2.6.24] Consolidate private allocations in seq files

2007-10-09 Thread Pavel Emelyanov
Many (very many) seq files in net/ allocate some private data to use it later (mostly for iteration state). All this code was obviously get using copy-paste method, so move it into one place. Almost all of these places either set this private to 0, or keep uninitialized. Some places, however,

[PATCH 1/6][NET-2.6.24] Introduce the seq_open_private()

2007-10-09 Thread Pavel Emelyanov
This function allocates the zeroed chunk of memory and call seq_open(). The __seq_open_private() helper returns the allocated memory to make it possible for the caller to initialize it. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] --- diff --git a/fs/seq_file.c b/fs/seq_file.c index

[PATCH 2/6][NET-2.6.24] Make core networking code use seq_open_private

2007-10-09 Thread Pavel Emelyanov
This concerns the ipv4 and ipv6 code mostly, but also the netlink and unix sockets. The netlink code is an example of how to use the __seq_open_private() call - it saves the net namespace on this private. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] --- diff --git a/net/ipv4/arp.c

[PATCH 3/6][NET-2.6.24] Make netfilter code use the seq_open_private

2007-10-09 Thread Pavel Emelyanov
Just switch to the consolidated calls. ipt_recent() has to initialize the private, so use the __seq_open_private() helper. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] Cc: Patrick McHardy [EMAIL PROTECTED] --- diff --git a/net/ipv4/netfilter/ipt_recent.c b/net/ipv4/netfilter/ipt_recent.c

Re: [PATCH][E1000E] some cleanups

2007-10-09 Thread Kok, Auke
jamal wrote: On Mon, 2007-08-10 at 15:40 -0700, Kok, Auke wrote: My biggest problem with the patch as you sent it that it's a tonload of changes and no implicit benefit immediately as I can see. The patch looks scary but is pretty tame when you apply it and stare at it. I would

[PATCH 4/6][NET-2.6.24] Make decnet code use the seq_open_private()

2007-10-09 Thread Pavel Emelyanov
Just switch to the consolidated code. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] Cc: Patrick Caulfield [EMAIL PROTECTED] --- diff --git a/net/decnet/dn_neigh.c b/net/decnet/dn_neigh.c index b66e3be..e851b14 100644 --- a/net/decnet/dn_neigh.c +++ b/net/decnet/dn_neigh.c @@ -580,24 +580,8

[PATCH 5/6][NET-2.6.24] Make the IRDA use the seq_open_private()

2007-10-09 Thread Pavel Emelyanov
Just switch to the consolidated code Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] Cc: Samuel Ortiz [EMAIL PROTECTED] --- diff --git a/net/irda/irlap.c b/net/irda/irlap.c index 3d76aaf..f3236ac 100644 --- a/net/irda/irlap.c +++ b/net/irda/irlap.c @@ -1219,29 +1219,11 @@ static const struct

[PATCH 6/6][NET-2.6.24] Make the sunrpc use the seq_open_private()

2007-10-09 Thread Pavel Emelyanov
Just switch to the consolidated code. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] Cc: Neil Brown [EMAIL PROTECTED] --- diff --git a/net/sunrpc/cache.c b/net/sunrpc/cache.c index ebe344f..8e05557 100644 --- a/net/sunrpc/cache.c +++ b/net/sunrpc/cache.c @@ -1218,23 +1218,15 @@ static const

Re: [PATCH 1/8][BNX2X] resubmit as attachments: add bnx2x to Kconfig and Makefile

2007-10-09 Thread Eliezer Tamir
On Mon, 2007-10-08 at 21:29 -0700, David Miller wrote: From: Eliezer Tamir [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 06:13:23 +0200 Due to the size of the patch I can not post it to the list. Understood Here is an FTP link. ftp://[EMAIL

Re: [PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread David Stevens
Sorry if this is a dumb question, but what is the model you intend for SNMP? Do you want each namespace to be its own virtual machine with its own, separate MIB? Ifindex's have to uniquely identify the interface (virtual or otherwise) to remote queriers (not just local applications), so unless

Re: [PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread Eric W. Biederman
Pavel Emelyanov [EMAIL PROTECTED] writes: Currently indexes for netdevices come sequentially one by one, and the same stays true even for devices that are created for namespaces. Side effects of this are: * lo device has not 1 index in a namespace. This may break some userspace that

Re: [PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread Eric W. Biederman
David Stevens [EMAIL PROTECTED] writes: Sorry if this is a dumb question, but what is the model you intend for SNMP? Do you want each namespace to be its own virtual machine with its own, separate MIB? Each network namespace appears to user space as a completely separate network stack. So

Re: [ofa-general] Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Stephen Hemminger
On 09 Oct 2007 18:51:51 +0200 Andi Kleen [EMAIL PROTECTED] wrote: David Miller [EMAIL PROTECTED] writes: 2) Switch the default qdisc away from pfifo_fast to a new DRR fifo with load balancing using the code in #1. I think this is kind of in the territory of what Peter said he is

Re: [ofa-general] Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Andi Kleen
I wonder about the whole idea of queueing in general at such high speeds. Given the normal bi-modal distribution of packets, and the predominance of 1500 byte MTU; does it make sense to even have any queueing in software at all? Yes that is my point -- it should just pass it through directly

[PATCH] xen-netfront: rearrange netfront_info structure to separate tx and rx

2007-10-09 Thread Jeremy Fitzhardinge
Keep tx and rx elements separate on different cachelines to prevent bouncing. Signed-off-by: Jeremy Fitzhardinge [EMAIL PROTECTED] Cc: Stephen Hemminger [EMAIL PROTECTED] Cc: Christoph Hellwig [EMAIL PROTECTED] --- drivers/net/xen-netfront.c | 37 ++--- 1 file

RE: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Waskiewicz Jr, Peter P
IMO the net driver really should provide a hint as to what it wants. 8139cp and tg3 would probably prefer multiple TX queue behavior to match silicon behavior -- strict prio. If I understand what you just said, I disagree. If your hardware is running strict prio, you don't want to enforce

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Jeff Garzik
Waskiewicz Jr, Peter P wrote: IMO the net driver really should provide a hint as to what it wants. 8139cp and tg3 would probably prefer multiple TX queue behavior to match silicon behavior -- strict prio. If I understand what you just said, I disagree. If your hardware is running strict

Re: 2.6.23-rc8-mm2 BUG: register_netdevice() issue as (ab)used by ISDN

2007-10-09 Thread Andreas Mohr
Hi, On Mon, Oct 08, 2007 at 05:58:36PM +0200, Karsten Keil wrote: You could try following patch with 2.6.23-rc8-mm2, it change I4L to use alloc_netdev(). I just did the horribly unthinkable: I rebooted the production internet gateway *remotely*, and: [EMAIL PROTECTED]:/home/andi# uname -a

RE: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Waskiewicz Jr, Peter P
A misunderstanding, I think. To my brain, DaveM's item #2 seemed to assume/require the NIC hardware to balance fairly across hw TX rings, which seemed to preclude the 8139cp/tg3 style of strict-prio hardware. That's what I was responding to. As long as there is some modular way to

Re: [PATCH][NETNS] Don't memset() netns to zero manually

2007-10-09 Thread David Miller
From: Pavel Emelyanov [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 15:48:19 +0400 The newly created net namespace is set to 0 with memset() in setup_net(). The setup_net() is also called for the init_net_ns(), which is zeroed naturally as a global var. So remove this memset and allocate new

Re: [PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Tue, 09 Oct 2007 11:43:58 -0600 David Stevens [EMAIL PROTECTED] writes: Sorry if this is a dumb question, but what is the model you intend for SNMP? Do you want each namespace to be its own virtual machine with its own, separate MIB?

Re: [PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread David Miller
From: David Stevens [EMAIL PROTECTED] Date: Tue, 9 Oct 2007 09:18:25 -0700 Ifindex's have to uniquely identify the interface (virtual or otherwise) to remote queriers (not just local applications), so unless you pay the price of separating all the SNMP MIBs per namespace too, it seems you'll

Re: [PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread David Miller
From: Pavel Emelyanov [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 16:19:25 +0400 Currently indexes for netdevices come sequentially one by one, and the same stays true even for devices that are created for namespaces. Side effects of this are: * lo device has not 1 index in a namespace.

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread David Miller
From: Jeff Garzik [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 08:44:25 -0400 David Miller wrote: From: Krishna Kumar2 [EMAIL PROTECTED] Date: Tue, 9 Oct 2007 16:51:14 +0530 David Miller [EMAIL PROTECTED] wrote on 10/09/2007 04:32:55 PM: Ignore LLTX, it sucks, it was a big mistake, and

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Jeff Garzik
David Miller wrote: From: Jeff Garzik [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 08:44:25 -0400 David Miller wrote: From: Krishna Kumar2 [EMAIL PROTECTED] Date: Tue, 9 Oct 2007 16:51:14 +0530 David Miller [EMAIL PROTECTED] wrote on 10/09/2007 04:32:55 PM: Ignore LLTX, it sucks, it was a big

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Roland Dreier
I'd say we can probably try to get rid of it in 2.6.25, this is assuming we get driver authors to cooperate and do the conversions or alternatively some other motivated person. I can just threaten to do them all and that should get the driver maintainers going :-) I can definitely

Re: [PATCH 1/7] [IPSEC]: Remove bogus ref count in xfrm_secpath_reject

2007-10-09 Thread David Miller
From: Herbert Xu [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 22:36:31 +0800 [IPSEC]: Remove bogus ref count in xfrm_secpath_reject Constructs of the form xfrm_state_hold(x); foo(x); xfrm_state_put(x); tend to be broken because foo is either synchronous where this is

Re: [PATCH 2/7] [IPSEC]: Store IPv6 nh pointer in mac_header on output

2007-10-09 Thread David Miller
From: Herbert Xu [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 22:36:32 +0800 [IPSEC]: Store IPv6 nh pointer in mac_header on output Current the x-mode-output functions store the IPv6 nh pointer in the skb network header. This is inconvenient because the network header then has to be fixed up

Re: [PATCH 3/7] [IPSEC]: Remove gratuitous km wake-up events on ACQUIRE

2007-10-09 Thread David Miller
From: Herbert Xu [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 22:36:33 +0800 [IPSEC]: Remove gratuitous km wake-up events on ACQUIRE There is no point in waking people up when creating/updating larval states because they'll just go back to sleep again as larval states by definition cannot be

Re: [PATCH 4/7] [IPSEC]: Move common code into xfrm_alloc_spi

2007-10-09 Thread David Miller
From: Herbert Xu [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 22:36:34 +0800 [IPSEC]: Move common code into xfrm_alloc_spi This patch moves some common code that conceptually belongs to the xfrm core from af_key/xfrm_user into xfrm_alloc_spi. In particular, the spin lock on the state is now

Re: [PATCH 5/7] [XFRM] user: Move attribute copying code into copy_to_user_state_extra

2007-10-09 Thread David Miller
From: Herbert Xu [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 22:36:35 +0800 [XFRM] user: Move attribute copying code into copy_to_user_state_extra Here's a good example of code duplication leading to code rot. The notification patch did its own netlink message creation for xfrm states. It

Re: [PATCH 6/7] [IPSEC]: Lock state when copying non-atomic fields to user-space

2007-10-09 Thread David Miller
From: Herbert Xu [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 22:36:36 +0800 [IPSEC]: Lock state when copying non-atomic fields to user-space This patch adds locking so that when we're copying non-atomic fields such as life-time or coaddr to user-space we don't get a partial result. For

Re: [PATCH 7/7] [IPSEC]: Move state lock into x-type-output

2007-10-09 Thread David Miller
From: Herbert Xu [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 22:36:37 +0800 [IPSEC]: Move state lock into x-type-output This patch releases the lock on the state before calling x-type-output. It also adds the lock to the spots where they're currently needed. Most of those places (all except

Re: [ofa-general] Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread David Miller
From: Andi Kleen [EMAIL PROTECTED] Date: 09 Oct 2007 18:51:51 +0200 Hopefully that new qdisc will just use the TX rings of the hardware directly. They are typically large enough these days. That might avoid some locking in this critical path. Indeed, I also realized last night that for the

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread David Miller
From: Roland Dreier [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 13:22:44 -0700 I can definitely kill LLTX for IPoIB by 2.6.25 and I just added it to my TODO list so I don't forget. In fact if 2.6.23 drags on long enough I may do it for 2.6.24 Before you add new entries to your list, how is

Re: [ofa-general] Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Stephen Hemminger
On Tue, 09 Oct 2007 13:43:31 -0700 (PDT) David Miller [EMAIL PROTECTED] wrote: From: Andi Kleen [EMAIL PROTECTED] Date: 09 Oct 2007 18:51:51 +0200 Hopefully that new qdisc will just use the TX rings of the hardware directly. They are typically large enough these days. That might avoid

Re: [PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread Eric W. Biederman
David Miller [EMAIL PROTECTED] writes: From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Tue, 09 Oct 2007 11:43:58 -0600 David Stevens [EMAIL PROTECTED] writes: Sorry if this is a dumb question, but what is the model you intend for SNMP? Do you want each namespace to be its own virtual

Re: net-2.6.24 rebased...

2007-10-09 Thread Urs Thuermann
Hallo Dave, 9.1MB, 739 changesets, keep those patches flowing! Last week I have sent another version of our patch series for PF_CAN. The changes after the last review feedback were only cosmetics. Do you have any plans with that code for this or the next release? Regards, urs - To unsubscribe

Re: [PATCH][NETNS] Make ifindex generation per-namespace

2007-10-09 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Tue, 09 Oct 2007 15:00:10 -0600 Regardless it is early yet and there is plenty of time to revisit this after we solved the easier and less controversial problems. Ok. I would encourage you to learn how the SNMP mibs work, and whether they

Re: net-2.6.24 rebased...

2007-10-09 Thread David Miller
From: Urs Thuermann [EMAIL PROTECTED] Date: 09 Oct 2007 23:13:42 +0200 Last week I have sent another version of our patch series for PF_CAN. The changes after the last review feedback were only cosmetics. Do you have any plans with that code for this or the next release? I think PF_CAN will

Re: [ofa-general] Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread David Miller
From: Stephen Hemminger [EMAIL PROTECTED] Date: Tue, 9 Oct 2007 13:53:40 -0700 I was thinking why not have a default transmit queue len of 0 like the virtual devices. I'm not so sure. Even if the device has huge queues I still think we need a software queue for when the hardware one backs up.

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread David Miller
From: Jeff Garzik [EMAIL PROTECTED] Date: Tue, 09 Oct 2007 16:20:14 -0400 David Miller wrote: If you unconditionally take those locks in the transmit function, there is probably an ABBA deadlock elsewhere in the driver now, most likely in the TX reclaim processing, and you therefore need

Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread Roland Dreier
Before you add new entries to your list, how is that ibm driver NAPI conversion coming along? :-) I still haven't done much. OK, I will try to get my board booting again this week. - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED]

Re: [ofa-general] Re: [PATCH 2/3][NET_BATCH] net core use batching

2007-10-09 Thread jamal
On Tue, 2007-09-10 at 14:22 -0700, David Miller wrote: Even if the device has huge queues I still think we need a software queue for when the hardware one backs up. It should be fine to just pretend the qdisc exists despite it sitting in the driver and not have s/ware queues at all to avoid

[PATCHES] TX batching rev2.5

2007-10-09 Thread jamal
Please provide feedback on the code and/or architecture. They are now updated to work with the latest rebased net-2.6.24 from a few hours ago. I am on travel mode so wont have time to do more testing for the next few days - i do consider this to be stable at this point based on what i have been

[PATCH 1/3] [NET_BATCH] Introduce batching interface Rev2.5

2007-10-09 Thread jamal
This patch introduces the netdevice interface for batching. cheers, jamal [NET_BATCH] Introduce batching interface This patch introduces the netdevice interface for batching. BACKGROUND - A driver dev-hard_start_xmit() has 4 typical parts: a) packet formating (example vlan, mss,

[PATCH 2/3][NET_BATCH] Rev2.5 net core use batching

2007-10-09 Thread jamal
This patch adds the usage of batching within the core. cheers, jamal [NET_BATCH] net core use batching This patch adds the usage of batching within the core. Performance results demonstrating improvement are provided separately. I have #if-0ed some of the old functions so the patch is more

  1   2   >