Re: BUG: soft lockup detected on CPU#0! (2.6.18.2 plus hacks)

2007-01-02 Thread Jarek Poplawski
On Tue, Jan 02, 2007 at 08:39:09AM +0100, Jarek Poplawski wrote: ... It is hard to say what kind of bug to expect because at the same time other net_rx_action with the same vlan dev could take place on other processor and this inetdev_init could do more. Sorry! inetdev_init couldn't do more

Re: [PATCH] [IPVS] replace if .. goto with while

2007-01-02 Thread David Miller
From: Horms [EMAIL PROTECTED] Date: Mon, 18 Dec 2006 12:11:11 +0900 I guess that this code used to be more complex, but replacing the goto with a while seems to make things a bit more readable. Or in other words, two fairly gratuitous goto are removed. On a related note, I wonder if there

Re: [PATCH] xt_hashlimit.c: fix typo

2007-01-02 Thread David Miller
From: Alexey Dobriyan [EMAIL PROTECTED] Date: Tue, 19 Dec 2006 18:57:33 +0300 Signed-off-by: Alexey Dobriyan [EMAIL PROTECTED] Applied, thanks Alexey. - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at

Re: [2.6 patch] net/irda/: proper prototypes

2007-01-02 Thread David Miller
From: Adrian Bunk [EMAIL PROTECTED] Date: Mon, 18 Dec 2006 04:46:26 +0100 This patch adds proper prototypes for some functions in include/net/irda/irda.h Signed-off-by: Adrian Bunk [EMAIL PROTECTED] ... +struct net_device; +struct packet_type; + +void irda_proc_register(void); +void

Re: BUG: soft lockup detected on CPU#0! (2.6.18.2 plus hacks)

2007-01-02 Thread Jarek Poplawski
On Tue, Jan 02, 2007 at 09:23:02AM +0100, Jarek Poplawski wrote: On Tue, Jan 02, 2007 at 08:39:09AM +0100, Jarek Poplawski wrote: ... The main thing is the possibility of processing skb with not entirely open source dev which isn't expected (and checked) by receive functions. I think the

Re: [PATCH] net: ifb error path loop fix

2007-01-02 Thread Jarek Poplawski
On 02-01-2007 08:51, David Miller wrote: From: Mariusz Kozlowski [EMAIL PROTECTED] Date: Tue, 2 Jan 2007 00:55:51 +0100 On error we should start freeing resources at [i-1] not [i-2]. Signed-off-by: Mariusz Kozlowski [EMAIL PROTECTED] Patch applied, thanks Mariusz. diff -upr

Re: [PATCH] net: ifb error path loop fix

2007-01-02 Thread Mariusz Kozlowski
Hello David, One could argue from a defensive programming perspective that this bug comes from the fact that the ifb_init_one() loop advances state before checking for errors ('i' is advanced before the 'err' check due to the loop construct), and that's why the error recovery code had to be

Re: [2.6 patch] net/irda/: proper prototypes

2007-01-02 Thread Adrian Bunk
On Tue, Jan 02, 2007 at 12:39:53AM -0800, David Miller wrote: From: Adrian Bunk [EMAIL PROTECTED] Date: Mon, 18 Dec 2006 04:46:26 +0100 This patch adds proper prototypes for some functions in include/net/irda/irda.h Signed-off-by: Adrian Bunk [EMAIL PROTECTED] ... +struct

[PATCH] net: af_netlink module_put cleanup

2007-01-02 Thread Mariusz Kozlowski
Hello, This patch removes redundant argument check for module_put(). Signed-off-by: Mariusz Kozlowski [EMAIL PROTECTED] net/netlink/af_netlink.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff -upr linux-2.6.20-rc2-mm1-a/net/netlink/af_netlink.c

Re: [PATCH] d80211: Fix inconsistent sta_lock usage

2007-01-02 Thread Ivo Van Doorn
On 1/1/07, Jan Kiszka [EMAIL PROTECTED] wrote: Hacking a bit on rt2x00 to make it work in master and ad-hoc mode, lockdep popped up on some hostapd ioctls, pointing out remaining inconsistencies related to sta_lock: 1. sta_lock holders must always be protected against softirq 2.

Re: [PATCH] d80211: Reinit keys on mode change

2007-01-02 Thread Ivo Van Doorn
On 1/1/07, Jan Kiszka [EMAIL PROTECTED] wrote: Switching the interface mode with some encryption keys set and then later touching any key, triggers an oops because ieee80211_if_reinit fails to NULL'ify the related pointers after free'ing the key on mode change. Long explanation, simple fix

Re: 2.6.19 and up to 2.6.20-rc2 Ethernet problems x86_64

2007-01-02 Thread Sid Boyce
Same problem with 2.6.19-rc3. Apologies for the long spiel, if memory serves me correct, gzip'd attachments are verboten. openSUSE 10.2 Network does not get configured with acpi=noirq or acpi=off. There may be something in dmesg that allows further analysis of the problem. 00:0a:e4:4e:a1:42

Re: Network card IRQ balancing with Intel 5000 series chipsets

2007-01-02 Thread Rick Jones
The best way to achieve such balancing is to have the network card help and essentially be able to select the CPU to notify while at the same time considering: a) avoiding any packet reordering - which restricts a flow to be processed to a single CPU at least within a timeframe b) be

Re: Network card IRQ balancing with Intel 5000 series chipsets

2007-01-02 Thread Rick Jones
With NAPI, if i have a few interupts it likely implies i have a huge network load (and therefore CPU use) and would be much more happier if you didnt start moving more interupt load to that already loaded CPU current irqbalance accounts for napi by using the number of packets as indicator

Re: [patch 0/6] sky2 driver update (v1.11)

2007-01-02 Thread Stephen Hemminger
On Tue, 26 Dec 2006 16:44:24 -0500 Jeff Garzik [EMAIL PROTECTED] wrote: Stephen Hemminger wrote: Patches are in order of severity. 1-3 are bug fixes, 4 is a cleanup of the power state code, and 5 adds wake on lan support. IMHO, it is bad security policy to allow wake on lan to enabled

Re: [patch 0/6] sky2 driver update (v1.11)

2007-01-02 Thread Tino Keitel
On Mon, Jan 01, 2007 at 10:36:44 -0800, Stephen Hemminger wrote: On Tue, 26 Dec 2006 16:44:24 -0500 Jeff Garzik [EMAIL PROTECTED] wrote: Stephen Hemminger wrote: Patches are in order of severity. 1-3 are bug fixes, 4 is a cleanup of the power state code, and 5 adds wake on lan support.

Re: 2.6.19 and up to 2.6.20-rc2 Ethernet problems x86_64

2007-01-02 Thread Len Brown
On Tuesday 02 January 2007 10:41, Sid Boyce wrote: Same problem with 2.6.19-rc3. Do you mean 2.6.20-rc3 still does not work? What was the last kernel that worked properly with no cmdline parameters? Apologies for the long spiel, if memory serves me correct, gzip'd attachments are verboten.

[patch] selinux: fix selinux_netlbl_inode_permission() locking

2007-01-02 Thread Ingo Molnar
* Andrew Morton [EMAIL PROTECTED] wrote: There's a glaring bug in selinux_netlbl_inode_permission() - taking lock_sock() inside rcu_read_lock(). Note that the bug is still in -rc3, and is easily triggerable via a default FC6 bootup. It's fixed by the (slightly modified) patch from Parag

Re: [patch 0/6] sky2 driver update (v1.11)

2007-01-02 Thread Gerd v. Egidy
IMHO, it is bad security policy to allow wake on lan to enabled by default. The sky2 driver doesn't do WOL until enabled with ethtool. While in general I agree with you on the security principle, this seems like it might break working setups. WOL is a partnership between the

Re: [PATCH] xfrm_user: avoid pointless void ** casts

2007-01-02 Thread David Miller
From: Christoph Hellwig [EMAIL PROTECTED] Date: Tue, 2 Jan 2007 15:39:06 +0100 All -doit handlers want a struct rtattr **, so pass down the right type. Signed-off-by: Christoph Hellwig [EMAIL PROTECTED] This code was just trying to mirror the typing used by struct rtnetlink_link's -doit()

Re: BUG: soft lockup detected on CPU#0! (2.6.18.2 plus hacks)

2007-01-02 Thread David Stevens
I've looked at this a little too -- it'd be nice to know who holds the write lock. I see ip_mc_destroy_dev() is bouncing through the lock for each multicast address, though it starts at the beginning of the list each time. I don't see a problem with it, but it'd be simpler if it acquired the

Re: BUG: soft lockup detected on CPU#0! (2.6.18.2 plus hacks)

2007-01-02 Thread Ben Greear
David Stevens wrote: I've looked at this a little too -- it'd be nice to know who holds the write lock. I see ip_mc_destroy_dev() is bouncing through the lock for each multicast address, though it starts at the beginning of the list each time. I don't see a problem with it, but it'd be simpler

Re: [2.6 patch] net/irda/: proper prototypes

2007-01-02 Thread David Miller
From: Adrian Bunk [EMAIL PROTECTED] Date: Tue, 2 Jan 2007 13:45:57 +0100 On Tue, Jan 02, 2007 at 12:39:53AM -0800, David Miller wrote: I don't like it, even if it's correct, because it is inconsistent with what we do in the entire rest of the networking code. Good point. Is it OK to

Re: selinux networking: sleeping functin called from invalid context in 2.6.20-rc[12]

2007-01-02 Thread Paul Moore
On Sunday, December 24 2006 7:25 pm, Andrew Morton wrote: On Mon, 25 Dec 2006 05:21:24 +0800 Adam J. Richter [EMAIL PROTECTED] wrote: Under 2.6.20-rc1 and 2.6.20-rc2, I get the following complaint for several network programs running on my system: [ 156.381868] BUG: sleeping

Re: selinux networking: sleeping functin called from invalid context in 2.6.20-rc[12]

2007-01-02 Thread Paul Moore
On Tuesday, January 2 2007 2:58 am, Adam J. Richter wrote: I have not yet performed the 21 steps of linux-2.6.20-rc3/Documentation/SubmitChecklist, which I think is a great objectives list for future automation or some kind of community web site. I hope to find time to make progress

Re: [PATCH] net: ifb error path loop fix

2007-01-02 Thread David Miller
From: Mariusz Kozlowski [EMAIL PROTECTED] Date: Tue, 2 Jan 2007 11:49:42 +0100 Hello David, One could argue from a defensive programming perspective that this bug comes from the fact that the ifb_init_one() loop advances state before checking for errors ('i' is advanced before the

Re: [PATCH] net: af_netlink module_put cleanup

2007-01-02 Thread David Miller
From: Mariusz Kozlowski [EMAIL PROTECTED] Date: Tue, 2 Jan 2007 13:47:37 +0100 Hello, This patch removes redundant argument check for module_put(). Signed-off-by: Mariusz Kozlowski [EMAIL PROTECTED] Applied, thanks. - To unsubscribe from this list: send the line unsubscribe netdev

Re: selinux networking: sleeping functin called from invalid context in 2.6.20-rc[12]

2007-01-02 Thread David Miller
From: Paul Moore [EMAIL PROTECTED] Date: Tue, 2 Jan 2007 16:25:24 -0500 I'm sorry I just saw this mail (mail not sent directly to me get shuffled off to a folder). I agree with your patch, I think dropping and then re-taking the RCU lock is the best way to go, although I'm curious to see

Re: [2.6 patch] the scheduled eepro100 removal

2007-01-02 Thread Eric Piel
02.01.2007 22:57, Adrian Bunk wrote/a écrit: This patch contains the scheduled removal of the eepro100 driver. Hi, I've been using e100 for years with no problem, however more by curiosity than necessity I'd like to know how will be handled the devices which are (supposedly) supported by

Re: Please pull 'upstream' branch of wireless-2.6

2007-01-02 Thread John W. Linville
On Wed, Dec 27, 2006 at 07:10:57PM -0500, John W. Linville wrote: On Tue, Dec 26, 2006 at 04:39:38PM -0500, Jeff Garzik wrote: John W. Linville wrote: The following changes since commit 0c234ae655a45ac3ee53a25b2e56e9bb6c27d71d: Ulrich Kunitz (1): ieee80211softmac: Fix

Please pull 'upstream-fixes' branch of wireless-2.6

2007-01-02 Thread John W. Linville
The following changes since commit 669df1b478803f49a356528d290af7bf442eb3be: Linus Torvalds (1): Linux 2.6.20-rc3 are found in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-2.6.git upstream-fixes Zhu Yi (2): ieee80211: WLAN_GET_SEQ_SEQ

Please pull 'upstream' branch of wireless-2.6

2007-01-02 Thread John W. Linville
The following changes since commit fe5f8e2a1c5c040209c598a28e19c55f30e1040d: Zhu Yi (1): ipw2100: Fix dropping fragmented small packet problem are found in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-2.6.git upstream Daniel Drake (5):

2.6.20-rc3: known unfixed regressions

2007-01-02 Thread Adrian Bunk
This email lists some known regressions in 2.6.20-rc3 compared to 2.6.19 that are not yet fixed in Linus' tree. If you find your name in the Cc header, you are either submitter of one of the bugs, maintainer of an affectected subsystem or driver, a patch of you caused a breakage or I'm

[PATCH] sungem: PHY updates pause fixes

2007-01-02 Thread Benjamin Herrenschmidt
This patch adds support for a few more PHYs used by Apple and fixes advertising and detecting of Pause (we were missing setting the bit in MII_ADVERTISE and weren't testing in LPA for all PHYs). I only do it for gigabit capable PHYs for now. Note that I currently only advertise pause, not

Re: [PATCH] sungem: PHY updates pause fixes

2007-01-02 Thread David Miller
From: Benjamin Herrenschmidt [EMAIL PROTECTED] Date: Wed, 03 Jan 2007 15:58:05 +1100 This patch adds support for a few more PHYs used by Apple and fixes advertising and detecting of Pause (we were missing setting the bit in MII_ADVERTISE and weren't testing in LPA for all PHYs). I only do it

Re: [PATCH] sungem: PHY updates pause fixes

2007-01-02 Thread Benjamin Herrenschmidt
Thanks for finding these bugs, although that's really strange pause behavior you are seeing on your switches. By default, we advertise PAUSE but not ASYM PAUSE in the tg3 driver, and I get flow control on every switch I have here. Yeah, that's strange. I still have the debug values at

Re: [PATCH] sungem: PHY updates pause fixes

2007-01-02 Thread David Miller
From: Benjamin Herrenschmidt [EMAIL PROTECTED] Date: Wed, 03 Jan 2007 16:20:14 +1100 Now, I cross-over with a TG3 and I get: - I advertise 0x5e1 (hopefully same value) - I read in LPA 0xc5e1 from the TG3 (and that other register tells me Rx and Tx pause can be enabled). Does flow control

[PATCH] sungem: PHY updates pause fixes (#2)

2007-01-02 Thread Benjamin Herrenschmidt
This patch adds support for a few more PHYs used by Apple and fixes advertising and detecting of Pause (we were missing setting the bit in MII_ADVERTISE and weren't testing in LPA for all PHYs). Note that I currently only advertise pause, not asymetric pause. I don't know for sure the details

[no subject]

2007-01-02 Thread haitao wang
subscribe netdev - 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] sungem: PHY updates pause fixes

2007-01-02 Thread Benjamin Herrenschmidt
The one with only asym. support is a big Cisco Catalyst 3350 (well.. big but not that many ports :-) Ok, I got in the config of the switch with somebody who knows how to speak ciscong, and it seems that it defaults to flow control desired for send and off for receive on all ports, which means

Re: 2.6.19 and up to 2.6.20-rc2 Ethernet problems x86_64

2007-01-02 Thread Len Brown
..same problem with 2.6.20-rc3. Last worked with 2.6.19-rc6-git12, so it was 2.6.19 where it failed. Attaching both case1 normal, case2 acpi=noirq. With acpi=noirq ethernet doesn't get configured, route -n says it's an Unsupported operation, ifconfig only shows for localhost, ifconfig

2.6.20-rc3: known regressions with patches available (part 2)

2007-01-02 Thread Adrian Bunk
This email lists some known regressions in 2.6.20-rc3 compared to 2.6.19 with patches available If you find your name in the Cc header, you are either submitter of one of the bugs, maintainer of an affectected subsystem or driver, a patch of you caused a breakage or I'm considering you in any