Re: [PATCH (take 2)] netdev: lockdep classes in register_netdevice Re: [patch 04/13] ppp_generic: fix lockdep warning

2007-05-16 Thread Jarek Poplawski
On Tue, May 15, 2007 at 10:47:25PM -0700, David Miller wrote: From: Jarek Poplawski [EMAIL PROTECTED] Date: Wed, 16 May 2007 07:40:00 +0200 After initializing dev-_xmit_lock register_netdevice() sets lockdep class according to dev-type. Idea of this patch - by David Miller.

Re: [PATCH (take 2)] netdev: lockdep classes in register_netdevice Re: [patch 04/13] ppp_generic: fix lockdep warning

2007-05-16 Thread David Miller
From: Jarek Poplawski [EMAIL PROTECTED] Date: Wed, 16 May 2007 08:17:32 +0200 BTW - I think some patch on vlan cannot do any harm (at least like this previous of mine - with only ppp considered), and maybe this all could be forgotten. Let's wait to see if any new messages show up. I think

Re: 2.6.20.7 TCP cubic (and bic) initial slow start way too slow?

2007-05-16 Thread Bill Fink
Hi Sangtae, On Sat, 12 May 2007, Sangtae Ha wrote: Hi Bill, This is the small patch that has been applied to 2.6.22. Also, there is limited slow start, which is an experimental RFC (RFC3742), to surmount this large increase during slow start. But, your kernel might not have this. Please

incorrect defines in mii.h

2007-05-16 Thread Matvejchikov Ilya
Hi all! The following defines are incorrect (include/linux/mii.h): #define BMSR_100FULL2 0x0200 /* Can do 100BASE-T2 HDX */ #define BMSR_100HALF2 0x0400 /* Can do 100BASE-T2 FDX */ it must be #define BMSR_100HALF2 0x0200 /* Can do 100BASE-T2 HDX */ #define

Re: [PATCH (take 2)] netdev: lockdep classes in register_netdevice Re: [patch 04/13] ppp_generic: fix lockdep warning

2007-05-16 Thread Jarek Poplawski
On Tue, May 15, 2007 at 11:17:51PM -0700, David Miller wrote: From: Jarek Poplawski [EMAIL PROTECTED] Date: Wed, 16 May 2007 08:17:32 +0200 BTW - I think some patch on vlan cannot do any harm (at least like this previous of mine - with only ppp considered), and maybe this all could be

Re: [PATCH] Intel IXP4xx network drivers v.2 - Ethernet and HSS

2007-05-16 Thread Christoph Hellwig
On Tue, May 08, 2007 at 10:29:03AM +0200, Tomasz Chmielewski wrote: Michael Jones wrote: +#ifndef __ARMEB__ +#warning Little endian mode not supported +#endif Personally I'm less fussed about WAN / LE support. Anyone with any sense will run ixp4xx boards doing such a specialised

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Fri, 11 May 2007, Satyam Sharma wrote: (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); + local_bh_disable(); +

Re: [PATCH] Intel IXP4xx network drivers v.2 - Ethernet and HSS

2007-05-16 Thread Lennert Buytenhek
On Wed, May 16, 2007 at 08:13:01AM +0100, Christoph Hellwig wrote: +#ifndef __ARMEB__ +#warning Little endian mode not supported +#endif Personally I'm less fussed about WAN / LE support. Anyone with any sense will run ixp4xx boards doing such a specialised network operation as

Re: [PATCH] Intel IXP4xx network drivers v.2 - Ethernet and HSS

2007-05-16 Thread Michael-Luke Jones
On 16 May 2007, at 10:41, Lennert Buytenhek wrote: Making a driver work in both modes of operation is generally not just an issue of adding a couple of be32_to_cpu()s in the right places. While this comment is technically correct, Christian's driver achieves endian agnostic operation with

why we use skb-rt-rt_src as dest address when replying packet?

2007-05-16 Thread lepton
I found in function [ip_send_reply] and [icmp_reply], we use such code to get the destination address of our packet: struct rtable *rt = (struct rtable *)skb-dst; .. daddr = ipc.addr = rt-rt_src; I have a question here: Is there any special reason for using rt-rt_src as destination address?

Re: [PATCH] Intel IXP4xx network drivers v.2 - Ethernet and HSS

2007-05-16 Thread Rod Whitby
Lots of people wrote: Lots of huffing and puffing about endian support by this driver ... For what it's worth, the NSLU2-Linux project (which has over 10,000 known users of our custom ixp4xx firmware, most of which will eventually be users of this new driver) is *endian-neutral*. We support

Re: [PATCH] Intel IXP4xx network drivers v.2 - Ethernet and HSS

2007-05-16 Thread Lennert Buytenhek
On Wed, May 16, 2007 at 08:16:38PM +0930, Rod Whitby wrote: So, if the author of these patches wishes to concentrate on big-endian support first, then we will not say (and have not said) anything which will block inclusion of a big-endian only version of this driver. The NSLU2 people are the

[PATCH] ibm_emac: Correctly detect old link speed

2007-05-16 Thread Stefan Roese
This patch fixes a bug where the link speed change was not detected correctly. This occured on a 440SPe (EMAC4) system where the old link speed was 100Mbps and the new link speed is 1000Mbps. Signed-off-by: Stefan Roese [EMAIL PROTECTED] --- commit 1e2a6085bbb6bd384e0812b6e9c62d18d14e1c0f tree

[PATCH][R8169] Add endianess annotations to [RT]xDesc

2007-05-16 Thread Rolf Eike Beer
[R8169] Add endianess annotations to [RT]xDesc Signed-off-by: Rolf Eike Beer [EMAIL PROTECTED] --- commit 8b36037f072047e61557506ee917dcc25680bd69 tree faafe5e664affabef85ff4dcab280338fd528fae parent b26c4a9f50ccfade25e3699f616ce97590dc2cb7 author Rolf Eike Beer [EMAIL PROTECTED] Wed, 16 May

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Satyam Sharma
Hi Jiri, On 5/16/07, Jiri Kosina [EMAIL PROTECTED] wrote: On Fri, 11 May 2007, Satyam Sharma wrote: (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): -

Re: [PATCH] Intel IXP4xx network drivers v.2 - Ethernet and HSS

2007-05-16 Thread Rod Whitby
Lennert Buytenhek wrote: On Wed, May 16, 2007 at 08:16:38PM +0930, Rod Whitby wrote: So, if the author of these patches wishes to concentrate on big-endian support first, then we will not say (and have not said) anything which will block inclusion of a big-endian only version of this driver.

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Marcel Holtmann
Hi Satayam, (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); + local_bh_disable(); + bh_lock_sock_nested(sock-sk);

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Satyam Sharma
Hi Marcel, On 5/16/07, Marcel Holtmann [EMAIL PROTECTED] wrote: Hi Satayam, (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); +

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Satyam Sharma
On 5/16/07, Satyam Sharma [EMAIL PROTECTED] wrote: Hi Marcel, [...] (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); +

Re: [PATCH] Intel IXP4xx network drivers v.2 - Ethernet and HSS

2007-05-16 Thread Lennert Buytenhek
On Wed, May 16, 2007 at 09:05:18PM +0930, Rod Whitby wrote: So, if the author of these patches wishes to concentrate on big-endian support first, then we will not say (and have not said) anything which will block inclusion of a big-endian only version of this driver. The NSLU2 people

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Marcel Holtmann
Hi Satyam, (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); + local_bh_disable(); +

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Wed, 16 May 2007, Marcel Holtmann wrote: since Jiri has a good test case for it, I leave it to him for testing. If he confirms that this fixes the locking issues, then this is Signed-off-by: Marcel Holtmann [EMAIL PROTECTED] I will verify later this evening and will let you know. I am

[PATCH] small netdevices.txt fix

2007-05-16 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig [EMAIL PROTECTED] Index: linux-2.6/Documentation/networking/netdevices.txt === --- linux-2.6.orig/Documentation/networking/netdevices.txt 2007-01-04 19:38:05.0 +0100 +++

[PATCH] spidernet: node-aware skbuff allocation

2007-05-16 Thread Christoph Hellwig
Spidernet was the driver I original did all the node-aware netdevice allocation for, but after a year it still hasn't hit mainline. So here's the patch again: Signed-off-by: Christoph Hellwig [EMAIL PROTECTED] Index: linux-2.6/drivers/net/spider_net.c

Purpose of bit shifts in tcp_get_info

2007-05-16 Thread Daniel Schaffrath
Dear Community, I was wondering what the purpose of the bit shifts in tcp_get_info right after the jiffies conversion might be. What's the time unit after that shift? info-tcpi_rtt = jiffies_to_usecs(tp-srtt)3; info-tcpi_rttvar = jiffies_to_usecs(tp-mdev)2; [...]

Re: [PATCH] Intel IXP4xx network drivers v.2 - Ethernet and HSS

2007-05-16 Thread Krzysztof Halasa
Christoph Hellwig [EMAIL PROTECTED] writes: Not even trying to support LE is a clear merge blocker. Maybe Krzysztof can't actually test it himself, which is fine - but not even pretending to be endian clean is not what proper Linux drivers do. The drivers can only work with IXP4xx CPU and

Re: [PATCH] Intel IXP4xx network drivers v.2 - Ethernet and HSS

2007-05-16 Thread Krzysztof Halasa
Rod Whitby [EMAIL PROTECTED] writes: Please let's just stop arguing about it. If a patch appears before it gets merged, then great. If it doesn't then it will appear at a later date. Sure. The swapping patch is trivial and I can add it if needed at some point but I hope we can do that data

[patch 0/7] CAN: Add new PF_CAN protocol family

2007-05-16 Thread Urs Thuermann
This patch series applies against linux-2.6.22-rc1-git4. It adds a new protocol family to Linux for communication on the CAN (Controller Area Network) using the socket API. The current implementation supports two protocols in the family, a raw protocol for sending and receiving raw CAN frames,

[patch 6/7] CAN: Add maintainer entries

2007-05-16 Thread Urs Thuermann
This patch adds entries in the CREDITS and MAINTAINERS file for CAN. Signed-Off-By: Oliver Hartkopp [EMAIL PROTECTED] Signed-Off-By: Urs Thuermann [EMAIL PROTECTED] --- CREDITS | 16 MAINTAINERS |9 + 2 files changed, 25 insertions(+) Index:

[patch 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-05-16 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-Off-By: Oliver Hartkopp [EMAIL PROTECTED] Signed-Off-By: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if_arp.h |1 +

[patch 5/7] CAN: Add virtual CAN netdevice driver

2007-05-16 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-Off-By: Oliver Hartkopp [EMAIL PROTECTED] Signed-Off-By: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1

Re: [PATCH] [e1000] Lower the MSI unavailable message to INFO priority

2007-05-16 Thread Kok, Auke
Jeff Garzik wrote: H. Peter Anvin wrote: diff --git a/drivers/net/e1000/e1000_main.c b/drivers/net/e1000/e1000_main.c index 637ae8f..089ae3f 100644 --- a/drivers/net/e1000/e1000_main.c +++ b/drivers/net/e1000/e1000_main.c @@ -307,7 +307,7 @@ static int e1000_request_irq(struct e1000_adapter

[patch 3/7] CAN: Add raw protocol

2007-05-16 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-Off-By: Oliver Hartkopp [EMAIL PROTECTED] Signed-Off-By: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 ++ net/can/Kconfig | 26 + net/can/Makefile|3 net/can/raw.c | 703

[patch 7/7] CAN: Add documentation

2007-05-16 Thread Urs Thuermann
This patch adds documentation for the PF_CAN protocol family. Signed-Off-By: Oliver Hartkopp [EMAIL PROTECTED] Signed-Off-By: Urs Thuermann [EMAIL PROTECTED] --- Documentation/networking/can.txt | 635 +++ 1 file changed, 635 insertions(+) Index:

[PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread Auke Kok
pci_enable_msi failure is a normal event so we should not print any error. Going over the code I spotted a missing pci_disable_msi() leak when irq allocation fails. The whole code also needed a cleanup, so I combined the two different calls to pci_request_irq into a single call making this look a

Re: [PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread Jeff Garzik
Auke Kok wrote: pci_enable_msi failure is a normal event so we should not print any error. Going over the code I spotted a missing pci_disable_msi() leak when irq allocation fails. The whole code also needed a cleanup, so I combined the two different calls to pci_request_irq into a single call

Re: [PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread Kok, Auke
Jeff Garzik wrote: Auke Kok wrote: pci_enable_msi failure is a normal event so we should not print any error. Going over the code I spotted a missing pci_disable_msi() leak when irq allocation fails. The whole code also needed a cleanup, so I combined the two different calls to pci_request_irq

Re: select(0, ..) is valid ?

2007-05-16 Thread Anton Blanchard
Hi Hugh, It's interesting that compat_core_sys_select() shows this kmalloc(0) failure but core_sys_select() does not. That's because core_sys_select() avoids kmalloc by using a buffer on the stack for small allocations (and 0 sure is small). Shouldn't compat_core_sys_select() do just the

[PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread Auke Kok
pci_enable_msi failure is a normal event so we should not print any error. Going over the code I spotted a missing pci_disable_msi() leak when irq allocation fails. The whole code also needed a cleanup, so I combined the two different calls to pci_request_irq into a single call making this look a

[PATCH] e1000: Work around 82571 completion timout on Pseries HW

2007-05-16 Thread Auke Kok
Our 82571 (first PCI-E hardware) causes P-Series hardware to throw issues. Disabling PCI-E completion timeouts in our NIC resolves the issue. Signed-off-by: Auke Kok [EMAIL PROTECTED] Cc: Wen Xiong [EMAIL PROTECTED] --- drivers/net/e1000/e1000_main.c | 10 ++ 1 files changed, 10

Re: [patch 2/7] CAN: Add PF_CAN core module

2007-05-16 Thread Arnaldo Carvalho de Melo
On 5/16/07, Urs Thuermann [EMAIL PROTECTED] wrote: This patch adds the CAN core functionality but no protocols or drivers. No protocol implementations are included here. They come as separate patches. Protocol numbers are already in include/linux/can.h. Signed-Off-By: Oliver Hartkopp [EMAIL

Re: [PATCH] ibm_emac: Correctly detect old link speed

2007-05-16 Thread Eugene Surovegin
On Wed, May 16, 2007 at 01:00:08PM +0200, Stefan Roese wrote: This patch fixes a bug where the link speed change was not detected correctly. This occured on a 440SPe (EMAC4) system where the old link speed was 100Mbps and the new link speed is 1000Mbps. Good catch, Stefan. Unfortunately, I

Re: Re: [patch 2/7] CAN: Add PF_CAN core module

2007-05-16 Thread Oliver Hartkopp
Arnaldo Carvalho de Melo wrote: SNIP + +/** + * struct sockaddr_can - the sockaddr structure for CAN sockets + * @can_family: address family number AF_CAN. + * @can_ifindex: CAN network interface index. + * @can_addr:transport protocol specific address, mostly CAN IDs. + */

Re: [PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread Rick Jones
Some more of my paranoid questions :) So, if a driver tries to enable MSI and that is unsuccessful (I'll try to avoid using the possibly loaded term fails) shouldn't that show-up _somewhere_? Just how normal is an attempt to enable MSI not succeding going to remain over time and aren't there

Re: [PATCH] ibm_emac: Correctly detect old link speed

2007-05-16 Thread Stefan Roese
On Wednesday 16 May 2007, Eugene Surovegin wrote: On Wed, May 16, 2007 at 01:00:08PM +0200, Stefan Roese wrote: This patch fixes a bug where the link speed change was not detected correctly. This occured on a 440SPe (EMAC4) system where the old link speed was 100Mbps and the new link speed

[PATCH 1/3] ibm_emac: fix section mismatch warnings

2007-05-16 Thread Eugene Surovegin
Fix Section mismatch warnings Signed-off-by: Eugene Surovegin [EMAIL PROTECTED] --- drivers/net/ibm_emac/ibm_emac_mal.c |3 +-- drivers/net/ibm_emac/ibm_emac_mal.h |3 +-- drivers/net/ibm_emac/ibm_emac_rgmii.c |2 +- drivers/net/ibm_emac/ibm_emac_rgmii.h |2 +-

[PATCH 2/3] ibm_emac: improved PHY support

2007-05-16 Thread Eugene Surovegin
Original patch is from Jeff Haran [EMAIL PROTECTED] with my minor style fixes. His comments follow: The first problem was in the function that configures the PHY for autonegotiation, genmii_setup_aneg(). The original code does a read/modify/write of the autonegotiation advertizement register

[PATCH 3/3] ibm_emac: fix link speed detection change

2007-05-16 Thread Eugene Surovegin
Fix link speed detection change. Thanks to Stefan Roese [EMAIL PROTECTED] for finding this bug. CC: Stefan Roese [EMAIL PROTECTED] Signed-off-by: Eugene Surovegin [EMAIL PROTECTED] --- drivers/net/ibm_emac/ibm_emac_core.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

Re: [PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread H. Peter Anvin
Rick Jones wrote: Some more of my paranoid questions :) So, if a driver tries to enable MSI and that is unsuccessful (I'll try to avoid using the possibly loaded term fails) shouldn't that show-up _somewhere_? It already does -- in /proc/interrupts. Just how normal is an attempt to enable

Re: [PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread Jeff Garzik
Rick Jones wrote: So, if a driver tries to enable MSI and that is unsuccessful (I'll try to avoid using the possibly loaded term fails) shouldn't that show-up _somewhere_? Just how normal is an attempt to enable MSI not succeding going to remain over time and aren't there times when it does

Re: [PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread David Miller
From: Rick Jones [EMAIL PROTECTED] Date: Wed, 16 May 2007 11:41:15 -0700 Some more of my paranoid questions :) So, if a driver tries to enable MSI and that is unsuccessful (I'll try to avoid using the possibly loaded term fails) shouldn't that show-up _somewhere_? Just how normal is an

[PATCH 1/2] ibmveth: Fix h_free_logical_lan error on pool resize

2007-05-16 Thread Brian King
When attempting to activate additional rx buffer pools on an ibmveth interface that was not yet up, the error below was seen. The patch fixes this by only closing and opening the interface to activate the resize if the interface is already opened. (drivers/net/ibmveth.c:597 ua:3004) ERROR:

[PATCH 2/2] ibmveth: Automatically enable larger rx buffer pools for larger mtu

2007-05-16 Thread Brian King
Currently, ibmveth maintains several rx buffer pools, which can be modified through sysfs. By default, pools are not allocated by default such that jumbo frames cannot be supported without first activating larger rx buffer pools. This results in failures when attempting to change the mtu. This

Re: [PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread Rick Jones
Jeff Garzik wrote: Rick Jones wrote: But that is rather incidental isn't it? Would some sort of system health monitor be likely to be checking that for interrupt flavors? And Well, that's where the information is exported in a standard way. I hope you're not suggesting that a system

[PATCH] gianfar: Add I/O barriers when touching buffer descriptor ownership.

2007-05-16 Thread Scott Wood
The hardware must not see that is given ownership of a buffer until it is completely written, and when the driver receives ownership of a buffer, it must ensure that any other reads to the buffer reflect its final state. Thus, I/O barriers are added where required. Without this patch, I have

Re: [3/3] 2.6.22-rc1: known regressions v2

2007-05-16 Thread Michal Piotrowski
Hi all, Here is a list of some known regressions in 2.6.22-rc1. Feel free to add new regressions/remove fixed etc. http://kernelnewbies.org/known_regressions Timers/NOHZ Subject: 2.6.21-git4 BUG: soft lockup detected on CPU#1! References : http://lkml.org/lkml/2007/5/2/511 Submitter :

Oops in netxen driver, kernel 2.6.20.10

2007-05-16 Thread Chuck Ebbert
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=240190 drivers/net/netxen/netxen_nic_init.c: if (ADDR_IN_WINDOW1(off)) { writel(buf[i].data, NETXEN_CRB_NORMALIZE(adapter, off));

Re: [patch 2/7] CAN: Add PF_CAN core module

2007-05-16 Thread Arnaldo Carvalho de Melo
On 16 May 2007 21:14:20 +0200, Urs Thuermann [EMAIL PROTECTED] wrote: Arnaldo Carvalho de Melo [EMAIL PROTECTED] writes: Can can_ifindex be turned into a unsigned short? That way we would have it nicely packed, avoiding this hole: Since dev-ifindex is int and we have many assignments between

[PATCH] libertas: skb dereferenced after netif_rx

2007-05-16 Thread Florin Malita
In libertas_process_rxed_packet() and process_rxed_802_11_packet() the skb is dereferenced after being passed to netif_rx (called from libertas_upload_rx_packet). Spotted by Coverity (1658, 1659). Also, libertas_upload_rx_packet() unconditionally returns 0 so the error check is dead code -

PROBLEM: SIS900 Driver change in Linux Kernel 2.6.21 causes kernel panic.

2007-05-16 Thread -
Kernel version 2.6.20.4 works. What I'm experiencing is a kernel panic as soon as the first received packet comes in via the sis900 ethernet interface. The machine is locked up and part of the kernel panic message is lost as it has scrolled off the screen and the virtual terminal has crashed as

[PATCH 0/10] spidernet assorted fixes

2007-05-16 Thread Linas Vepstas
Jeff, Please apply forward upstream the following patch series. There's a variety of fixes cleanups in here; the key fix is a rather hard-to-reproduce bug where the hardware runs out of ram, resulting in corruption of some sequencer. The only fix is to reset the card. v-v Unfortunately,

[PATCH 1/10] spidernet: node-aware skbuff allocation

2007-05-16 Thread Linas Vepstas
From: Christoph Hellwig [EMAIL PROTECTED] Spidernet was the driver I original did all the node-aware netdevice allocation for, but after a year it still hasn't hit mainline. Signed-off-by: Christoph Hellwig [EMAIL PROTECTED] Signed-off-by: Linas Vepstas [EMAIL PROTECTED]

[PATCH] TCP slow start: comments and cleanup

2007-05-16 Thread Stephen Hemminger
Add more comments to describe our version of tcp_slow_start(). Signed-off-by: Stephen Hemminger [EMAIL PROTECTED] --- net/ipv4/tcp_cong.c | 40 ++-- 1 files changed, 22 insertions(+), 18 deletions(-) diff --git a/net/ipv4/tcp_cong.c b/net/ipv4/tcp_cong.c

[PATCH 2/10] spidernet: beautify error messages

2007-05-16 Thread Linas Vepstas
Make error messages print which interface they apply to. Signed-off-by: Linas Vepstas [EMAIL PROTECTED] drivers/net/spider_net.c | 10 ++ drivers/net/spider_net.h |2 +- 2 files changed, 7 insertions(+), 5 deletions(-) Index: linux-2.6.22-rc1/drivers/net/spider_net.c

Re: [PATCH] e1000: Fix msi enable leak on error, don't print error message, cleanup

2007-05-16 Thread Jeff Garzik
Rick Jones wrote: Agreed. But is the PCI (?) subsystem doing something in that regard or is this a hole? Depends on your platform, your BIOS, your kernel command line, ... :) Jeff - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

[PATCH 3/10] spidernet: move a block of code around

2007-05-16 Thread Linas Vepstas
Put the enable and disable routines next to one-another, as this makes verifying thier symmetry that much easier. Signed-off-by: Linas Vepstas [EMAIL PROTECTED] drivers/net/spider_net.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) Index:

[PATCH 4/10] spidernet: zero out a pointer.

2007-05-16 Thread Linas Vepstas
Invalidate a pointer as its pci_unmap'ed; this is a bit of paranoia to make sure hardware doesn't continue trying to DMA to it. Signed-off-by: Linas Vepstas [EMAIL PROTECTED] drivers/net/spider_net.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) Index:

Re: [WIP] [PATCH] WAS Re: [RFC] New driver API to speed up small packets xmits

2007-05-16 Thread Sridhar Samudrala
Jamal, Here are some comments i have on your patch. See them inline. Thanks Sridhar +static int try_get_tx_pkts(struct net_device *dev, struct Qdisc *q, int count) +{ + struct sk_buff *skb; + struct sk_buff_head *skbs = dev-blist; + int tdq = count; + + /* +*

[PATCH 5/10] spidernet: null out skb pointer after its been used.

2007-05-16 Thread Linas Vepstas
If the ethernet interface is brought down while there is still RX traffic in flight, the device shutdown routine can end up trying to double-free an skb, leading to a crash in mm/slab.c Avoid the double-free by nulling out the skb pointer. Signed-off-by: Linas Vepstas [EMAIL PROTECTED]

[PATCH 6/10] spidernet: Don't terminate the RX ring

2007-05-16 Thread Linas Vepstas
There is no real reason to terminate the RX ring; it doesn't make the operation any smooother, and it does require an extra sync. So don't do it. Signed-off-by: Linas Vepstas [EMAIL PROTECTED] drivers/net/spider_net.c | 18 +- 1 file changed, 9 insertions(+), 9

[PATCH 7/10] spidernet: enhance the dump routine

2007-05-16 Thread Linas Vepstas
Crazy device problems are hard to debug, when one does not have good trace info. This patch makes a major enhancement to the device dump routine. Signed-off-by: Linas Vepstas [EMAIL PROTECTED] drivers/net/spider_net.c | 62 --- 1 file changed,

Re: PROBLEM: SIS900 Driver change in Linux Kernel 2.6.21 causes kernel panic.

2007-05-16 Thread Dave Jones
On Wed, May 16, 2007 at 02:33:18PM -0700, - wrote: Kernel version 2.6.20.4 works. What I'm experiencing is a kernel panic as soon as the first received packet comes in via the sis900 ethernet interface. The machine is locked up and part of the kernel panic message is lost as it has

[PATCH 8/10] spidernet: reset the card when an rxramfull is seen

2007-05-16 Thread Linas Vepstas
Some versions of the spider have a firmware bug, where the RX ring sequencer goes crazy when the RX RAM on the device fills up. Appearently the only viable wrkaround is a soft reset of the card. Signed-off-by: Linas Vepstas [EMAIL PROTECTED] drivers/net/spider_net.c | 14 +++---

[PATCH 9/10] spidernet: service TX later.

2007-05-16 Thread Linas Vepstas
When entering the netdev poll routine, empty out the RX chain first, before cleaning up the TX chain. This should help avoid RX buffer overflows. Signed-off-by: Linas Vepstas [EMAIL PROTECTED] drivers/net/spider_net.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Index:

[PATCH 10/10] spidernet: increase the NAPI weight

2007-05-16 Thread Linas Vepstas
Another way of minimizing the likelyhood of RX ram from overflowing is to empty out the entire rx ring every chance we get. Change the crazy watchdog timeout from 50 seconds to 3 seconds, while we're here. Signed-off-by: Linas Vepstas [EMAIL PROTECTED] drivers/net/spider_net.h |9

Re: [WIP] [PATCH] WAS Re: [RFC] New driver API to speed up small packets xmits

2007-05-16 Thread jamal
On Wed, 2007-16-05 at 15:12 -0700, Sridhar Samudrala wrote: Jamal, Here are some comments i have on your patch. See them inline. Thanks for taking the time Sridhar. try_tx_pkts() is directly calling the device's batch xmit routine. Don't we need to call dev_hard_start_xmit() to handle

Re: [PATCH] [NET_SCHED]make qdisc_restart more readable

2007-05-16 Thread David Miller
From: jamal [EMAIL PROTECTED] Date: Wed, 16 May 2007 18:55:04 -0400 On Mon, 2007-14-05 at 09:30 -0400, jamal wrote: Incoporating last comments from Patrick. Dave, I think this is ready to apply - against net-2.6 from this morning. Dave - hold onto this patch - Sridhar Samudrala [EMAIL

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Wed, 16 May 2007, Jiri Kosina wrote: since Jiri has a good test case for it, I leave it to him for testing. If he confirms that this fixes the locking issues, then this is Signed-off-by: Marcel Holtmann [EMAIL PROTECTED] I will verify later this evening and will let you know. I am

Re: [stable] PROBLEM: SIS900 Driver change in Linux Kernel 2.6.21 causes kernel panic.

2007-05-16 Thread Greg KH
On Wed, May 16, 2007 at 06:17:04PM -0400, Dave Jones wrote: On Wed, May 16, 2007 at 02:33:18PM -0700, - wrote: Kernel version 2.6.20.4 works. What I'm experiencing is a kernel panic as soon as the first received packet comes in via the sis900 ethernet interface. The machine is locked

Re: [PATCH] [NET_SCHED]make qdisc_restart more readable

2007-05-16 Thread Sridhar Samudrala
On Wed, 2007-05-16 at 18:55 -0400, jamal wrote: On Mon, 2007-14-05 at 09:30 -0400, jamal wrote: Incoporating last comments from Patrick. Dave, I think this is ready to apply - against net-2.6 from this morning. Dave - hold onto this patch - Sridhar Samudrala [EMAIL PROTECTED] caught a

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread David Miller
From: Jiri Kosina [EMAIL PROTECTED] Date: Thu, 17 May 2007 01:03:55 +0200 (CEST) On Wed, 16 May 2007, Jiri Kosina wrote: since Jiri has a good test case for it, I leave it to him for testing. If he confirms that this fixes the locking issues, then this is Signed-off-by: Marcel

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Wed, 16 May 2007, David Miller wrote: I have just verified that this locking scheme is indeed correct. So you can add Signed-off-by: Jiri Kosina [EMAIL PROTECTED] if you wish to, and submit the patch to Andrew. I guess I don't get sent networking patches any more? :-)

Re: [Cbe-oss-dev] [PATCH 4/10] spidernet: zero out a pointer.

2007-05-16 Thread Michael Ellerman
On Wed, 2007-05-16 at 17:09 -0500, Linas Vepstas wrote: Invalidate a pointer as its pci_unmap'ed; this is a bit of paranoia to make sure hardware doesn't continue trying to DMA to it. Signed-off-by: Linas Vepstas [EMAIL PROTECTED] drivers/net/spider_net.c |7 +-- 1 file

Re: [Cbe-oss-dev] [PATCH 10/10] spidernet: increase the NAPI weight

2007-05-16 Thread Michael Ellerman
On Wed, 2007-05-16 at 17:23 -0500, Linas Vepstas wrote: Another way of minimizing the likelyhood of RX ram from overflowing is to empty out the entire rx ring every chance we get. Change the crazy watchdog timeout from 50 seconds to 3 seconds, while we're here. Signed-off-by: Linas Vepstas

RT patches expose netdev race [was Re: [RFC] [patch 2/2] powerpc 2.6.21-rt1: fix kernel hang and/or panic

2007-05-16 Thread Linas Vepstas
Hi, On Tue, May 15, 2007 at 08:09:02PM +1000, Benjamin Herrenschmidt wrote: On Tue, 2007-05-15 at 17:47 +0900, Tsutomu OWA wrote: I encountered the following error when doing netperf from other machine to Celleb running RT kernel. PREEPT_NONE kernel works just fine as well. Hrm...

Resending: RT patches expose netdev race [was Re: [RFC] [patch 2/2] powerpc 2.6.21-rt1: fix kernel hang and/or panic

2007-05-16 Thread Linas Vepstas
(resending , Owa-san was cut from cc list!??) Hi, On Tue, May 15, 2007 at 08:09:02PM +1000, Benjamin Herrenschmidt wrote: On Tue, 2007-05-15 at 17:47 +0900, Tsutomu OWA wrote: I encountered the following error when doing netperf from other machine to Celleb running RT kernel.

Re: RT patches expose netdev race [was Re: [RFC] [patch 2/2] powerpc 2.6.21-rt1: fix kernel hang and/or panic

2007-05-16 Thread David Miller
From: [EMAIL PROTECTED] (Linas Vepstas) Date: Wed, 16 May 2007 19:18:02 -0500 Hi, On Tue, May 15, 2007 at 08:09:02PM +1000, Benjamin Herrenschmidt wrote: On Tue, 2007-05-15 at 17:47 +0900, Tsutomu OWA wrote: I encountered the following error when doing netperf from other machine to

Re: [PATCH] e1000: Work around 82571 completion timout on Pseries HW

2007-05-16 Thread Kumar Gala
On May 16, 2007, at 3:53 AM, Auke Kok wrote: Our 82571 (first PCI-E hardware) causes P-Series hardware to throw issues. Disabling PCI-E completion timeouts in our NIC resolves the issue. Signed-off-by: Auke Kok [EMAIL PROTECTED] Cc: Wen Xiong [EMAIL PROTECTED] ---

Re: Resending: RT patches expose netdev race [was Re: [RFC] [patch 2/2] powerpc 2.6.21-rt1: fix kernel hang and/or panic

2007-05-16 Thread Benjamin Herrenschmidt
I do not know why sk_buff-head would be null, or would be set in a racy kind of way, or why the rt patches would cause this. But the evidence implicates that. Would it be possible that a locking bug in spidernet would cause it under some circumstances to get a stale skb pointer ? Ben. - To

Re: select(0, ..) is valid ?

2007-05-16 Thread Badari Pulavarty
On Wed, 2007-05-16 at 10:37 -0500, Anton Blanchard wrote: Hi Hugh, It's interesting that compat_core_sys_select() shows this kmalloc(0) failure but core_sys_select() does not. That's because core_sys_select() avoids kmalloc by using a buffer on the stack for small allocations (and 0

Re: [stable] [PATCH 3/6] sky2: allow 88E8056

2007-05-16 Thread Chris Wright
* Stephen Hemminger ([EMAIL PROTECTED]) wrote: It looks like the problems of Gigabyte 88E8056 are unique to that chip motherboard and maybe fixable by EEPROM update. So, drop the Gigabyte hunks in the original patch...ok, thanks. -chris - To unsubscribe from this list: send the line unsubscribe

Re: [stable] [PATCH 4/6] sky2: 88e8071 support not ready

2007-05-16 Thread Chris Wright
* Stephen Hemminger ([EMAIL PROTECTED]) wrote: - { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x436B) }, /* 88E8071 */ +// { PCI_DEVICE(PCI_VENDOR_ID_MARVELL, 0x436B) }, /* 88E8071 */ Where-o-where are the CodingStyle police? ;-) - To unsubscribe from this list: send the line unsubscribe netdev in

[PATCH] spidernet: skb used after netif_receive_skb

2007-05-16 Thread Florin Malita
The stats update code in spider_net_pass_skb_up() is touching the skb after it's been passed up to the stack. To avoid that, just update the stats first. Signed-off-by: Florin Malita [EMAIL PROTECTED] --- spider_net.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [stable] PROBLEM: SIS900 Driver change in Linux Kernel 2.6.21 causes kernel panic.

2007-05-16 Thread Chris Wright
* Dave Jones ([EMAIL PROTECTED]) wrote: You need this.. http://git.kernel.org/?p=linux%2Fkernel%2Fgit%2Ftorvalds%2Flinux-2.6.git;a=commitdiff_plain;h=dc5a144991ba803bc8afded105c9db1dea0e57ab Which is queued for -stable afaik, but no sign of 2.6.21.2 yet. Greg/Chris? Yes, it is queued,

Re: [stable] [PATCH 2.6.21-stable] [IPV6]: Restore semantics of Routing Header processing.

2007-05-16 Thread Chris Wright
* YOSHIFUJI Hideaki / 吉藤英明 ([EMAIL PROTECTED]) wrote: In article [EMAIL PROTECTED] (at Fri, 11 May 2007 09:22:43 -0700), Chris Wright [EMAIL PROTECTED] says: * YOSHIFUJI Hideaki / 吉藤英明 ([EMAIL PROTECTED]) wrote: The fix for emerging security threats was overkill and it broke basic

Re: [PATCH] sky2: re-enable 88E8056 for most motherboards

2007-05-16 Thread Daniel Drake
Stephen Hemminger wrote: It looks like these users aren't using MSI? Possible. Want me to find out? I just ask as I note yesterday's -stable patch where you remove the #ifdef completely. Is that going to go upstream into 2.6.22 or do you want to continue pursuing the DMI matching path?

Re: [PATCH] [NET_SCHED]make qdisc_restart more readable

2007-05-16 Thread jamal
On Wed, 2007-16-05 at 16:15 -0700, Sridhar Samudrala wrote: I think your qdisc_restart() cleanup patch is OK. There you are still calling dev_hard_start_xmit() and GSO should work fine. Sorry, you are right - I thought i cutnpasted from it. So Dave, false alarm ;- That patch is clean. But

Re: [stable] [PATCH 2.6.21-stable] [IPV6]: Restore semantics of Routing Header processing.

2007-05-16 Thread Chris Wright
* David Miller ([EMAIL PROTECTED]) wrote: We're not pushing this in, even the ipv6 working group is unsure how this should be handled and one of the possibilities they might choose matches how things currently are. Alright, I'll drop this one from the -stable radar, thanks. -chris - To

Re: [PATCH] sky2: re-enable 88E8056 for most motherboards

2007-05-16 Thread Jeff Garzik
Daniel Drake wrote: Stephen Hemminger wrote: It looks like these users aren't using MSI? Possible. Want me to find out? I just ask as I note yesterday's -stable patch where you remove the #ifdef completely. Is that going to go upstream into 2.6.22 or do you want to continue pursuing the

Re: [PATCH] sky2: re-enable 88E8056 for most motherboards

2007-05-16 Thread Daniel Drake
Jeff Garzik wrote: Stephen has some sky2 patches I need to push. I accidentally filed them until 'Pending: SATA', and they got missed in my most recent net driver push. Thanks, that clears it up, forgot to check netdev before asking. Looks like the 2.6.22 plans are to drop the machine

  1   2   >