[RFC/PATCH 3/3] UDP memory usage accounting (take 2): measurement

2007-09-28 Thread Satoshi OSHIMA
This patch introduces memory usage measurement for UDP. These 3 points were updated. - UDP specific codes in IP layer were removed. - atomic_sub() in a loop was removed - accounting during socket destruction signed-off-by: Satoshi Oshima [EMAIL PROTECTED] signed-off-by: Hideo Aoki [EMAIL

Upgradeing 2.6.21.7-2.6.22.9 kill my network (sky2): sky2 eth0: rx error, status 0x402300 length 60

2007-09-28 Thread Krzysztof Oledzki
Hello, After upgrading my kernel from 2.6.21.7 to 2.6.22.9 my 88E8053 no longer works: sky2 :02:00.0: v1.14 addr 0xcfffc000 irq 17 Yukon-EC (0xb6) rev 1 sky2 eth0: addr 00:11:d8:50:f6:28 sky2 eth0: enabling interface sky2 eth0: ram buffer 48K sky2 eth0: Link is up at 100 Mbps, full

[RFC/PATCH 2/3] UDP memory usage accounting: accounting unit and variable

2007-09-28 Thread Satoshi OSHIMA
This patch introduces global variable for UDP memory accounting. The unit is page. signed-off-by: Satoshi Oshima [EMAIL PROTECTED] signed-off-by: Hideo Aoki [EMAIL PROTECTED] Index: 2.6.23-rc3-udp_limit/include/net/sock.h === ---

Re: [RFC/PATCH 0/3] UDP memory usage accounting

2007-09-28 Thread Satoshi OSHIMA
Hi, Thank you for your comment. Evgeniy Polyakov wrote: Hi. On Fri, Sep 21, 2007 at 09:18:07PM +0900, Satoshi OSHIMA ([EMAIL PROTECTED]) wrote: This patch set try to introduce memory usage accounting for UDP(currently ipv4 only). Currently, memory usage of UDP can be observed as the sam

[RFC/PATCH 1/3] UDP memory usage accounting (take 2): fix send buffer check

2007-09-28 Thread Satoshi OSHIMA
This patch introduces sndbuf size check before memory allcation for send buffer. signed-off-by: Satoshi Oshima [EMAIL PROTECTED] signed-off-by: Hideo Aoki [EMAIL PROTECTED] Index: 2.6.23-rc7-udp_limit/net/ipv4/ip_output.c ===

[PATCH] mv643xx_eth: Check ETH_INT_CAUSE_STATE bit

2007-09-28 Thread Dale Farnsworth
Commit 468d09f8946d40228c56de26fe4874b2f98067ed masked the state interrupt (bit 20 of the cause register). This results in Radstone's PPC7D repeatedly re-entering the interrupt routine, locking up the board. The following patch returns the required handling for this interrupt. Signed-off-by:

Re: 2.6.23-rc8-mm2 - tcp_fastretrans_alert() WARNING

2007-09-28 Thread Cedric Le Goater
Hello ! Andrew Morton wrote: ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.23-rc8/2.6.23-rc8-mm2/ I just found that warning in my logs. It seems that it's been happening since rc7-mm1 at least. Thanks ! C. WARNING: at

Re: Fwd: [PATCH#2 3/4] [PPC] Compile fix for 8xx CPM Ehernet driver

2007-09-28 Thread Jeff Garzik
Kumar Gala wrote: Begin forwarded message: From: Jochen Friedrich [EMAIL PROTECTED] Date: September 24, 2007 12:15:35 PM CDT To: [EMAIL PROTECTED] Cc: [EMAIL PROTECTED], Marcelo Tosatti [EMAIL PROTECTED] Subject: [PATCH#2 3/4] [PPC] Compile fix for 8xx CPM Ehernet driver Jeff, Please pick

Re: 2.6.23-rc8: cannot make netconsole work

2007-09-28 Thread Matt Mackall
On Fri, Sep 28, 2007 at 01:27:55PM +0400, Andrey Borzenkov wrote: I finally decided to try netconsole in attempt to get some more information why my system does not resume (but that is different story). But I cannot make it work - it does load but I see no traffic flowing ever. This is

Re: Upgrading 2.6.21.7-2.6.22.9 kills my network (sky2): sky2 eth0: rx error, status 0x402300 length 60

2007-09-28 Thread Krzysztof Oledzki
On Fri, 28 Sep 2007, Krzysztof Oledzki wrote: On Fri, 28 Sep 2007, Krzysztof Oledzki wrote: Hello, After upgrading my kernel from 2.6.21.7 to 2.6.22.9 my 88E8053 no longer works: Small update: 2.6.22.9 with sky2.c/sky2.h from 2.4.22.4 works without any problems. Final update.

Re: [RFC/PATCH 2/3] UDP memory usage accounting: accounting unit and variable

2007-09-28 Thread Satoshi OSHIMA
Hi, Thank you for your comment. Andi Kleen wrote: Satoshi OSHIMA [EMAIL PROTECTED] writes: This patch introduces global variable for UDP memory accounting. The unit is page. The global variable doesn't seem to be very MP scalable, especially if you change it for each packet. This will be

Re: [PKT_SCHED]: Add stateless NAT

2007-09-28 Thread Patrick McHardy
David Miller wrote: From: jamal [EMAIL PROTECTED] Date: Thu, 27 Sep 2007 08:39:45 -0400 +config NET_ACT_NAT +tristate Stateless NAT +depends on NET_CLS_ACT +select NETFILTER I am gonna have to agree with Evgeniy on this Herbert;- The rewards are it will improve

[PATCH 3/3] sky2: fix transmit state on resume

2007-09-28 Thread Stephen Hemminger
After resume, driver has reset the chip so the current state of transmit checksum offload state machine and DMA state machine will be undefined. The fix is to set the state so that first Tx will set MSS and offset values. Signed-off-by: Stephen Hemminger [EMAIL PROTECTED] ---

Re: e1000 tcp checksum incorrect, x86 64b

2007-09-28 Thread Rick Jones
Herbert Xu wrote: Jon Smirl [EMAIL PROTECTED] wrote: App is writing seven bytes to the socket. Socket write timeout expires and the seven bytes are sent. The checksum is not getting inserted into the packet. It is set to a constant 0x8389 instead of the right value. App is gmpc 0.15.4.95,

Re: [PKT_SCHED]: Add stateless NAT

2007-09-28 Thread Patrick McHardy
David Miller wrote: I still think the nf_*() prefixes should all go and the extern prototypes should go into an independant header file. These are not netfilter routines, they are INET helpers. Agreed. Evgeniy, can you send a new patch for this? And we should make similar treatment for

[IPV6] Fix ICMPv6 redirect handling with target multicast address

2007-09-28 Thread Brian Haley
When the ICMPv6 Target address is multicast, Linux processes the redirect instead of dropping it. The problem is in this code in ndisc_redirect_rcv(): if (ipv6_addr_equal(dest, target)) { on_link = 1; } else if (!(ipv6_addr_type(target) IPV6_ADDR_LINKLOCAL)) {

Re: [PATCH] net: Add network namespace clone unshare support.

2007-09-28 Thread Cedric Le Goater
Andrew Morton wrote: On Fri, 28 Sep 2007 11:12:13 +0200 Cedric Le Goater [EMAIL PROTECTED] wrote: Cedric made a good point that we will have conflicts of code being added to the same place in nsproxy.c and the like. So I copied Andrew to give him a heads up. here's a suggestion, we could

Re: e1000 tcp checksum incorrect, x86 64b

2007-09-28 Thread Jon Smirl
On 9/28/07, Rick Jones [EMAIL PROTECTED] wrote: Herbert Xu wrote: Jon Smirl [EMAIL PROTECTED] wrote: App is writing seven bytes to the socket. Socket write timeout expires and the seven bytes are sent. The checksum is not getting inserted into the packet. It is set to a constant 0x8389

Re: [PATCH] sb1250-mac: Driver model phylib update

2007-09-28 Thread Maciej W. Rozycki
On Mon, 24 Sep 2007, Andrew Morton wrote: Well, this is against Jeff's netdev-2.6 tree which hopefully is not as crufty as Linus's old mainline; if it is not possible to queue this change for 2.6.25 or suchlike, then I will try to resubmit later. Most of Jeff's netdev tree got dumped

Re: Upgradeing 2.6.21.7-2.6.22.9 kill my network (sky2): sky2 eth0: rx error, status 0x402300 length 60

2007-09-28 Thread Krzysztof Oledzki
On Fri, 28 Sep 2007, Krzysztof Oledzki wrote: Hello, After upgrading my kernel from 2.6.21.7 to 2.6.22.9 my 88E8053 no longer works: Small update: 2.6.22.9 with sky2.c/sky2.h from 2.4.22.4 works without any problems. Best regards, Krzysztof Olędzki

Re: [PATCH] net: Add network namespace clone unshare support.

2007-09-28 Thread Andrew Morton
On Fri, 28 Sep 2007 11:12:13 +0200 Cedric Le Goater [EMAIL PROTECTED] wrote: Cedric made a good point that we will have conflicts of code being added to the same place in nsproxy.c and the like. So I copied Andrew to give him a heads up. here's a suggestion, we could keep the net

Re: [PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-09-28 Thread Oliver Hartkopp
Eric W. Biederman wrote: Currently IFF_LOOPBACK set in dev-flags means we are dealing with drivers/net/loopback.c. This is a very general view, don't you think? The one is an interface flag and the other one is an interface itself. This looks like a risky mixture, when there is no clean

Re: [RFC] Zero-length write() does not generate a datagram on connected socket

2007-09-28 Thread Michael Kerrisk
On 9/28/07, Stephen Hemminger [EMAIL PROTECTED] wrote: On Thu, 27 Sep 2007 13:53:34 -0700 (PDT) David Miller [EMAIL PROTECTED] wrote: From: Stephen Hemminger [EMAIL PROTECTED] Date: Mon, 24 Sep 2007 15:34:35 -0700 The bug http://bugzilla.kernel.org/show_bug.cgi?id=5731 describes an

Re: [PKT_SCHED]: Add stateless NAT

2007-09-28 Thread Evgeniy Polyakov
On Thu, Sep 27, 2007 at 12:52:39PM -0700, David Miller ([EMAIL PROTECTED]) wrote: From: Patrick McHardy [EMAIL PROTECTED] Date: Thu, 27 Sep 2007 15:39:34 +0200 Evgeniy Polyakov wrote: On Thu, Sep 27, 2007 at 09:20:37PM +0800, Herbert Xu ([EMAIL PROTECTED]) wrote: How about

Re: ax88796: add 93cx6 eeprom support

2007-09-28 Thread Magnus Damm
Hi Andrew, Thanks for picking up the patch. On 9/28/07, Andrew Morton [EMAIL PROTECTED] wrote: On Thu, 27 Sep 2007 19:51:19 +0900 Magnus Damm [EMAIL PROTECTED] wrote: ax88796: add 93cx6 eeprom support This patch hooks up the 93cx6 eeprom code to the ax88796 driver and modifies the

Re: /proc/net/ bad hard links count [Was: 2.6.23-rc8-mm2]

2007-09-28 Thread Eric W. Biederman
Jiri Slaby [EMAIL PROTECTED] writes: On 09/27/2007 11:22 AM, Andrew Morton wrote: ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.23-rc8/2.6.23-rc8-mm2/ Yep. # find /proc /dev/null find: WARNING: Hard link count is wrong for /proc/net: this may be a bug in your

Re: [PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-09-28 Thread Eric W. Biederman
Oliver Hartkopp [EMAIL PROTECTED] writes: Eric W. Biederman wrote: Currently IFF_LOOPBACK set in dev-flags means we are dealing with drivers/net/loopback.c. This is a very general view, don't you think? The one is an interface flag and the other one is an interface itself. This looks

MSI interrupts and disable_irq

2007-09-28 Thread Ayaz Abdulla
I am trying to track down a forcedeth driver issue described by bug 9047 in bugzilla (2.6.23-rc7-git1 forcedeth w/ MCP55 oops under heavy load). I added a patch to synchronize the timer handlers so that one handler doesn't accidently enable the IRQ while another timer handler is running (see

[PATCH 2/3] sky2: fix VLAN receive processing

2007-09-28 Thread Stephen Hemminger
The length check for truncated frames was not correctly handling the case where VLAN acceleration had already read the tag. Also, the Yukon EX has some features that use high bit of status as security tag. Signed-off-by: Pierre-Yves Ritschard [EMAIL PROTECTED] Signed-off-by: Stephen Hemminger

[PATCH 1/3] sky2: reduce impact of watchdog timer

2007-09-28 Thread Stephen Hemminger
This is the 2.6.22 version of a regression fix that is already in 2.6.23. Change the watchdog timer form 10 per second all the time, to 1 per second and only if interface is up. Signed-off-by: Stephen Hemminger [EMAIL PROTECTED] --- a/drivers/net/sky2.c2007-09-17 10:39:47.0

[PATCH 0/3] sky2: patches for 2.6.22.y

2007-09-28 Thread Stephen Hemminger
Fixes for power regression, VLAN and resume problems. These are all in 2.6.23. -- Stephen Hemminger [EMAIL PROTECTED] - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at

Re: [PATCH] net: Add network namespace clone unshare support.

2007-09-28 Thread Cedric Le Goater
Eric W. Biederman wrote: David Miller [EMAIL PROTECTED] writes: Eric, pick an appropriate new non-conflicting number NOW. Done. My apologies for the confusion. I thought the way Cedric and the IBM guys were testing someone would have shouted at me long before now. This adds

Re: [PKT_SCHED]: Add stateless NAT

2007-09-28 Thread Evgeniy Polyakov
On Fri, Sep 28, 2007 at 12:19:19PM +0400, Evgeniy Polyakov ([EMAIL PROTECTED]) wrote: I still think the nf_*() prefixes should all go and the extern prototypes should go into an independant header file. These are not netfilter routines, they are INET helpers. And we should make

Re: [RFC/PATCH 3/3] UDP memory usage accounting (take 2): measurement

2007-09-28 Thread Evgeniy Polyakov
On Fri, Sep 28, 2007 at 10:41:31PM +0900, Satoshi OSHIMA ([EMAIL PROTECTED]) wrote: This patch introduces memory usage measurement for UDP. These 3 points were updated. - UDP specific codes in IP layer were removed. - atomic_sub() in a loop was removed - accounting during socket

Re: 2.6.23-rc8 network problem. Mem leak? ip1000a?

2007-09-28 Thread Andrew Morton
On 27 Sep 2007 22:06:17 -0400 [EMAIL PROTECTED] wrote: Uniprocessor Althlon 64, 64-bit kernel, 2G ECC RAM, 2.6.23-rc8 + linuxpps (5.0.0) + ip1000a driver. (patch from http://marc.info/?l=linux-netdevm=118980588419882) After a few hours of operation, ntp loses the ability to send packets.

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

2007-09-28 Thread Thomas Gleixner
On Tue, 2007-09-25 at 14:09 -0700, David Miller wrote: Then please make all exported symbols marked EXPORT_SYMBOL_GPL to make sure that the other CAN protocol can not reuse your infrastructure. We don't want to force other CAN protocol implementations to be GPL also. AFAIR from

Re: [stable] Upgrading 2.6.21.7-2.6.22.9 kills my network (sky2): sky2 eth0: rx error, status 0x402300 length 60

2007-09-28 Thread Greg KH
On Fri, Sep 28, 2007 at 01:11:27PM +0200, Krzysztof Oledzki wrote: On Fri, 28 Sep 2007, Krzysztof Oledzki wrote: On Fri, 28 Sep 2007, Krzysztof Oledzki wrote: Hello, After upgrading my kernel from 2.6.21.7 to 2.6.22.9 my 88E8053 no longer works: Small update: 2.6.22.9 with

Re: UDP Out 0f Sequence

2007-09-28 Thread Randy Macleod
Rick Jones wrote: Majumder, Rajib wrote: Let's say we have 2 uniprocessor hosts connected back to back. Is there any possibility of an out-of-order scenario on recv? Your application should be written on the assumption that it is possible, regardless of the specifics of the hosts involved,

/proc/net/ bad hard links count [Was: 2.6.23-rc8-mm2]

2007-09-28 Thread Jiri Slaby
On 09/27/2007 11:22 AM, Andrew Morton wrote: ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.23-rc8/2.6.23-rc8-mm2/ # find /proc /dev/null find: WARNING: Hard link count is wrong for /proc/net: this may be a bug in your filesystem driver. Automatically turning on find's

Fwd: [PATCH#2 3/4] [PPC] Compile fix for 8xx CPM Ehernet driver

2007-09-28 Thread Kumar Gala
Begin forwarded message: From: Jochen Friedrich [EMAIL PROTECTED] Date: September 24, 2007 12:15:35 PM CDT To: [EMAIL PROTECTED] Cc: [EMAIL PROTECTED], Marcelo Tosatti [EMAIL PROTECTED] Subject: [PATCH#2 3/4] [PPC] Compile fix for 8xx CPM Ehernet driver Jeff, Please pick up for 2.6.23 if you

Re: [PATCH]: Second preliminary release of Sun Neptune driver

2007-09-28 Thread Matheos Worku
David Miller wrote: Ok, this is rev2, changes: 1) Jumbo MTU support is now present. I got stuck on this for a while because I didn't realize that resetting the RX XMAC would reset the XMAC_MAX register. I thought that was a TX XMAC resource, ho hum... Fix this by re-initializing

Re: 2.6.23-rc8: cannot make netconsole work

2007-09-28 Thread Andrey Borzenkov
On Friday 28 September 2007, Matt Mackall wrote: On Fri, Sep 28, 2007 at 01:27:55PM +0400, Andrey Borzenkov wrote: [...] sudo modprobe netconsole netconsole=@/eth0,@/ [...] What is your console log level set to? If the messages don't come out on the local console, they won't get sent out

[RFC/PATCH 0/3] UDP memory usage accounting(take 2)

2007-09-28 Thread Satoshi OSHIMA
This patch set try to introduce memory usage accounting for UDP(currently ipv4 only). 3 points are improved along with some feedback. (a) to improve scalability, avoiding atomic_*()s as small as possible (b) avoiding UDP specific code in IP layer (c) supporting socket destruction

Re: [stable] Upgrading 2.6.21.7-2.6.22.9 kills my network (sky2): sky2 eth0: rx error, status 0x402300 length 60

2007-09-28 Thread Krzysztof Oledzki
On Fri, 28 Sep 2007, Greg KH wrote: On Fri, Sep 28, 2007 at 01:11:27PM +0200, Krzysztof Oledzki wrote: On Fri, 28 Sep 2007, Krzysztof Oledzki wrote: On Fri, 28 Sep 2007, Krzysztof Oledzki wrote: Hello, After upgrading my kernel from 2.6.21.7 to 2.6.22.9 my 88E8053 no longer works:

Re: [PATCH] sb1250-mac: Driver model phylib update

2007-09-28 Thread Andrew Morton
On Fri, 28 Sep 2007 17:23:00 +0100 (BST) Maciej W. Rozycki [EMAIL PROTECTED] wrote: On Mon, 24 Sep 2007, Andrew Morton wrote: Well, this is against Jeff's netdev-2.6 tree which hopefully is not as crufty as Linus's old mainline; if it is not possible to queue this change for

Re: [PATCH 2/3] sky2: fix VLAN receive processing

2007-09-28 Thread Krzysztof Oledzki
On Fri, 28 Sep 2007, Stephen Hemminger wrote: The length check for truncated frames was not correctly handling the case where VLAN acceleration had already read the tag. Also, the Yukon EX has some features that use high bit of status as security tag. Thank you. Best regards

Re: [PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-09-28 Thread Oliver Hartkopp
Eric W. Biederman wrote: Oliver Hartkopp [EMAIL PROTECTED] writes: The CAN protocol family is some kind of a closed ecosystem with a complete different addressing scheme that uses the bare networking functionality of the Linux Kernel as well as DECNET or ARCNET. You would never been

Re: 2.6.23-rc8-mm2 - tcp_fastretrans_alert() WARNING

2007-09-28 Thread Ilpo Järvinen
On Fri, 28 Sep 2007, Cedric Le Goater wrote: Hello ! Andrew Morton wrote: ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.23-rc8/2.6.23-rc8-mm2/ I just found that warning in my logs. It seems that it's been happening since rc7-mm1 at least. Thanks ! C.

Re: [stable] [PATCH 2/3] sky2: fix VLAN receive processing

2007-09-28 Thread Chris Wright
* Stephen Hemminger ([EMAIL PROTECTED]) wrote: The length check for truncated frames was not correctly handling the case where VLAN acceleration had already read the tag. Also, the Yukon EX has some features that use high bit of status as security tag. Did you leave out the GMR_FS_LEN change

Re: [stable] [PATCH 2/3] sky2: fix VLAN receive processing

2007-09-28 Thread Stephen Hemminger
On Fri, 28 Sep 2007 12:20:44 -0700 Chris Wright [EMAIL PROTECTED] wrote: * Stephen Hemminger ([EMAIL PROTECTED]) wrote: The length check for truncated frames was not correctly handling the case where VLAN acceleration had already read the tag. Also, the Yukon EX has some features that use

Re: [ofa-general] [PATCH v3] iw_cxgb3: Supportiwarp-onlyinterfacesto avoid 4-tuple conflicts.

2007-09-28 Thread Steve Wise
Kanevsky, Arkady wrote: Sean, IB aside, it looks like an ULP which is capable of being both RDMA aware and RDMA not-aware, like iSER and iSCSI, NFS-RDMA and NFS, SDP and sockets, will be treated as two separete ULPs. Each has its own IP address, since there is a different IP address for

2.6.23-rc[68]-mm: network hangs

2007-09-28 Thread Laurent Riffard
Hi, From time to time, I experience some complete network hangs: Suddenly, all network connections become unresponsive. Even ping 127.0.0.1 doesn't work. SysRq-w does not show any blocked processus. When such hang happen, I have to reboot (shutdown does work). This is not easily reproducible:

Re: [stable] [PATCH 2/3] sky2: fix VLAN receive processing

2007-09-28 Thread Chris Wright
* Stephen Hemminger ([EMAIL PROTECTED]) wrote: I left it out on purpose because 2.6.22 doesn't support Yukon EX. OK, thanks. - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at

Re: [RFC/PATCH 1/3] UDP memory usage accounting (take 2): fix send buffer check

2007-09-28 Thread David Miller
From: Satoshi OSHIMA [EMAIL PROTECTED] Date: Fri, 28 Sep 2007 22:37:54 +0900 } else if (i MAX_SKB_FRAGS) { +if (atomic_read(sk-sk_wmem_alloc) + PAGE_SIZE + 2 * sk-sk_sndbuf) { +err = -ENOBUFS; +

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

2007-09-28 Thread David Miller
From: Thomas Gleixner [EMAIL PROTECTED] Date: Fri, 28 Sep 2007 18:27:19 +0200 I'm not inclined either way and we really should not make this a religious question whether that code goes in or not, especially not when we granted the mac80211 to export everything w/o _GPL suffix not too long

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

2007-09-28 Thread Thomas Gleixner
On Fri, 2007-09-28 at 13:20 -0700, David Miller wrote: That's not true with CAN. With this CAN stuff, any driver you write for it is intimately integrated into the design and architecture of the CAN subsystem. Any such driver cannot stand on it's own. Look at how these drivers can get

RE: [ofa-general] [PATCH v3] iw_cxgb3: Supportiwarp-onlyinterfacesto avoid 4-tuple conflicts.

2007-09-28 Thread Kanevsky, Arkady
Exactly, it forces the burden on administrator. And one will be forced to try one mount for iWARP and it does not work issue another one TCP or UDP if it fails. Yack! And server will need to listen on different IP address and simple * will not work since it will need to listen in two different

Re: [ofa-general] [PATCH v3] iw_cxgb3: Supportiwarp-onlyinterfacesto avoid 4-tuple conflicts.

2007-09-28 Thread Steve Wise
Kanevsky, Arkady wrote: Exactly, it forces the burden on administrator. And one will be forced to try one mount for iWARP and it does not work issue another one TCP or UDP if it fails. Yack! I see your point. I have no defense. My hands have been tied on fixing this properly... And

Re: error(s) in 2.6.23-rc5 bonding.txt ?

2007-09-28 Thread Rick Jones
Well, I managed to concoct an updated test, this time with 1G's going into a 10G. A 2.6.23-rc8 kernel on the system with four, dual-port 82546GB's, connected to an HP ProCurve 3500 series switch with a 10G link to a system running 2.6.18-8.el5 (I was having difficulty getting cxgb3 going on my

Re: [ofa-general] [PATCH v3] iw_cxgb3: Supportiwarp-onlyinterfacesto avoid 4-tuple conflicts.

2007-09-28 Thread Sean Hefty
Kanevsky, Arkady wrote: Exactly, it forces the burden on administrator. And one will be forced to try one mount for iWARP and it does not work issue another one TCP or UDP if it fails. Yack! And server will need to listen on different IP address and simple * will not work since it will need to

[PATCH] IPoIB: Convert to netdevice internal stats

2007-09-28 Thread Roland Dreier
Use the stats member of struct netdevice in IPoIB, so we can save memory by deleting the stats member of struct ipoib_dev_priv, and save code by deleting ipoib_get_stats(). Signed-off-by: Roland Dreier [EMAIL PROTECTED] --- Dave, can you queue this in net-2.6.24 please? I would ordinarily merge

Re: [PATCH] IPoIB: Convert to netdevice internal stats

2007-09-28 Thread David Miller
From: Roland Dreier [EMAIL PROTECTED] Date: Fri, 28 Sep 2007 15:18:01 -0700 Use the stats member of struct netdevice in IPoIB, so we can save memory by deleting the stats member of struct ipoib_dev_priv, and save code by deleting ipoib_get_stats(). Signed-off-by: Roland Dreier [EMAIL

Re: [RFC] Make TCP prequeue configurable

2007-09-28 Thread David Miller
From: John Heffner [EMAIL PROTECTED] Date: Thu, 27 Sep 2007 22:26:02 -0400 I think it really does help in case (4) with old NICs that don't do rx checksumming. I'm not sure how many people really care about this anymore, but probably some...? OTOH, it would be nice to get rid of

SFQ: backport some features from ESFQ (try 4)

2007-09-28 Thread Corey Hickey
Patchset try 2 addresses the review by Michael Buesch. Patchset try 3 addresses the review by Patrick McHardy. Patchset try 4 has a few cosmetic improvements. Nobody reviewed my last set of patches, and I wasn't pushy about asking. Since it's been a while, I ported the kernel and userspace

[PATCH 03/10] Move two functions.

2007-09-28 Thread Corey Hickey
Move sfq_q_destroy() to above sfq_q_init() so that it can be used by an error case in a later patch. Move sfq_destroy() as well, for clarity. Signed-off-by: Corey Hickey [EMAIL PROTECTED] --- net/sched/sch_sfq.c | 22 +++--- 1 files changed, 11 insertions(+), 11 deletions(-)

[PATCH 01/10] Preparatory refactoring part 1.

2007-09-28 Thread Corey Hickey
Make a new function sfq_q_enqueue() that operates directly on the queue data. This will be useful for implementing sfq_change() in a later patch. A pleasant side-effect is reducing most of the duplicate code in sfq_enqueue() and sfq_requeue(). Similarly, make a new function sfq_q_dequeue().

[PATCH 02/10] Preparatory refactoring part 2.

2007-09-28 Thread Corey Hickey
Factor code out of sfq_init() and sfq_destroy(), again so that the new functions can be used by sfq_change() later. Actually, as the diff itself shows, most of the sfq_q_init() code comes from the original sfq_change(), but sfq_change() is only called by sfq_init() right now. Thus, it is safe to

[PATCH 04/10] Make depth (number of queues) user-configurable:

2007-09-28 Thread Corey Hickey
* replace #define with a parameter * use old hardcoded value as a default * kcalloc() arrays in sfq_q_init() * free() arrays in sfq_q_destroy() Signed-off-by: Corey Hickey [EMAIL PROTECTED] --- net/sched/sch_sfq.c | 85 +++--- 1 files changed, 59

[PATCH 06/10] Make qdisc changeable.

2007-09-28 Thread Corey Hickey
Re-implement sfq_change() and enable Qdisc_opts.change so tc qdisc change will work. Signed-off-by: Corey Hickey [EMAIL PROTECTED] --- net/sched/sch_sfq.c | 61 ++- 1 files changed, 60 insertions(+), 1 deletions(-) diff --git

[PATCH 08/10] Multiply perturb_period by HZ when used rather than when assigned.

2007-09-28 Thread Corey Hickey
perturb_period is the only parameter that doesn't match 1:1 with the value from userspace. This change makes it easy and clean to use a small macro for setting parameters (in a subsequent patch). Signed-off-by: Corey Hickey [EMAIL PROTECTED] --- net/sched/sch_sfq.c | 10 +- 1 files

[PATCH 07/10] Remove comments about hardcoded values.

2007-09-28 Thread Corey Hickey
None of these are true anymore (hooray!). Signed-off-by: Corey Hickey [EMAIL PROTECTED] --- include/linux/pkt_sched.h |8 net/sched/sch_sfq.c | 17 +++-- 2 files changed, 3 insertions(+), 22 deletions(-) diff --git a/include/linux/pkt_sched.h

[PATCH 09/10] Change perturb_period to unsigned.

2007-09-28 Thread Corey Hickey
perturb_period is currently a signed integer, but I can't see any good reason why this is so--a negative perturbation period will add a timer that expires in the past, causing constant perturbation, which makes hashing useless. if (q-perturb_period) {

[PATCH 05/10] Add divisor.

2007-09-28 Thread Corey Hickey
Make hash divisor user-configurable. Signed-off-by: Corey Hickey [EMAIL PROTECTED] --- net/sched/sch_sfq.c | 18 +- 1 files changed, 13 insertions(+), 5 deletions(-) diff --git a/net/sched/sch_sfq.c b/net/sched/sch_sfq.c index 34a299d..d72ea7c 100644 --- a/net/sched/sch_sfq.c

[PATCH 10/10] Use nested compat attributes to pass parameters.

2007-09-28 Thread Corey Hickey
This fixes the ambiguity between, for example: tc qdisc change ... perturb 0 tc qdisc change ... Without this patch, there is no way for SFQ to differentiate between a parameter specified to be 0 and a parameter that was omitted. Signed-off-by: Corey Hickey [EMAIL PROTECTED] ---

sfq

2007-09-28 Thread Corey Hickey
These patches follow the ESFQ--SFQ kernel patches. See the kernel patch summary for general information. Thanks, Corey include/linux/pkt_sched.h | 23 ++- tc/q_sfq.c| 43 ++- 2 files changed, 52 insertions(+), 14

[PATCH 2/3] Change perturb_period to unsigned.

2007-09-28 Thread Corey Hickey
This corresponds to the kernel patch doing the same. Here, too, this will technically break binary compatibility with older kernels, but that shouldn't be a problem because negative perturb_period values aren't usable anyway. Signed-off-by: Corey Hickey [EMAIL PROTECTED] ---

[PATCH 3/3] Use nested compat attributes for passing parameters to the kernel.

2007-09-28 Thread Corey Hickey
Note that I have left sfq_print_opt() alone. At this point, there can be no difference between the data in the nested rtattrs and the data in the compat rtattr, and I didn't want to add clutter that isn't useful. Let me know if I should do differently. Signed-off-by: Corey Hickey [EMAIL

[PATCH 1/3] SFQ: Support changing depth and divisor.

2007-09-28 Thread Corey Hickey
This can safely be applied either before or after the kernel patches because the tc_sfq_qopt struct is unchanged: - old kernels will ignore the parameters from new iproute2 - new kernels will use the same default parameters Signed-off-by: Corey Hickey [EMAIL PROTECTED] ---

Re: sfq (iproute2 patches)

2007-09-28 Thread Corey Hickey
Corey Hickey wrote: These patches follow the ESFQ--SFQ kernel patches. See the kernel patch summary for general information. Dang, I forgot to set the subject; these are the iproute2 patches. -Corey - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to

Null dereference in socket.c

2007-09-28 Thread Chuck Ebbert
After debugging an oops (https://bugzilla.redhat.com/attachment.cgi?id=209231) I find it happens here in socket.c::sock_ioctl() line 902: default: = err = sock-ops-ioctl(sock, cmd, arg); /* * If this ioctl is

Re: [PATCH] IPoIB: Convert to netdevice internal stats

2007-09-28 Thread Roland Dreier
How is that ibm_emac NAPI conversion coming along? :-) Sorry, trying to reduce my backlog first, but it is still on my list of things to work on :) - R. - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at

Re: Null dereference in socket.c

2007-09-28 Thread David Miller
From: Chuck Ebbert [EMAIL PROTECTED] Date: Fri, 28 Sep 2007 18:58:36 -0400 After debugging an oops (https://bugzilla.redhat.com/attachment.cgi?id=209231) I find it happens here in socket.c::sock_ioctl() line 902: default: = err = sock-ops-ioctl(sock, cmd,

Re: [PATCH] Update get_net_ns_by_pid

2007-09-28 Thread Serge E. Hallyn
Quoting Eric W. Biederman ([EMAIL PROTECTED]): In the -mm tree the rules for access an nsproxy have changed, and in get_net_ns_by_pid we access the nsproxy, so update it to follow the new rules. Signed-off-by: Eric W. Biederman [EMAIL PROTECTED] Yup, looks right. I assume Pavel's

Re: Null dereference in socket.c

2007-09-28 Thread Stephen Hemminger
On Fri, 28 Sep 2007 16:00:57 -0700 (PDT) David Miller [EMAIL PROTECTED] wrote: From: Chuck Ebbert [EMAIL PROTECTED] Date: Fri, 28 Sep 2007 18:58:36 -0400 After debugging an oops (https://bugzilla.redhat.com/attachment.cgi?id=209231) I find it happens here in socket.c::sock_ioctl() line

Re: [IPV6] Fix ICMPv6 redirect handling with target multicast address

2007-09-28 Thread David Stevens
Brian, A multicast address should never be the target of a neighbor discovery request; the sender should use the mapping function for all multicasts. So, I'm not sure that your example can ever happen, and it certainly is ok to send ICMPv6 errors to multicast addresses in general. But I

Re: [PKT_SCHED]: Add stateless NAT

2007-09-28 Thread Herbert Xu
On Fri, Sep 28, 2007 at 06:55:32PM +0200, Patrick McHardy wrote: Looking at ip_input.o as example (everything without forced inlining): textdata bss dec hex filename 2076 8 02084 824 net/ipv4/ip_input.o 3483 8 03491 da3

delayed acks question

2007-09-28 Thread Andrea Arcangeli
Hello, I've a few questions about ICSK_ACK_PUSHED2. PUSHED2 is only meant to force the ack out immediately when pingpong is set to 1, but then if pingpong is 1 the delayed acks shouldn't be deferred anyway. However I think the trouble is that there's a race condition in reading pingpong,

[PATCH] rtnl: Simplify ASSERT_RTNL

2007-09-28 Thread Eric W. Biederman
Currently we have the call path: macvlan_open - dev_unicast_add - __dev_set_rx_mode - __dev_set_promiscuity - ASSERT_RTNL - mutex_trylock When mutex debugging is on taking a mutex complains if we are not allowed to sleep. At that point we have called netif_tx_lock_bh so we are clearly

[PATCH 1/5] net: Modify all rtnetlink methods to only work in the initial namespace

2007-09-28 Thread Eric W. Biederman
Before I can enable rtnetlink to work in all network namespaces I need to be certain that something won't break. So this patch deliberately disables all of the rtnletlink methods in everything except the initial network namespace. After the methods have been audited this extra check can be

[PATCH 2/5] net: Make rtnetlink infrastructure network namespace aware

2007-09-28 Thread Eric W. Biederman
After this patch none of the netlink callback support anything except the initial network namespace but the rtnetlink infrastructure now handles multiple network namespaces. Signed-off-by: Eric W. Biederman [EMAIL PROTECTED] --- include/linux/rtnetlink.h |8 ++--

Re: [IPV6] Fix ICMPv6 redirect handling with target multicast address

2007-09-28 Thread YOSHIFUJI Hideaki / 吉藤英明
Dave, Brian, Let me double check this patch. Regards, --yoshfuji In article [EMAIL PROTECTED] (at Fri, 28 Sep 2007 17:50:38 -0700), David Stevens [EMAIL PROTECTED] says: Brian, A multicast address should never be the target of a neighbor discovery request; the sender should use

[PATCH 3/5] net: Make the netlink methods in rtnetlink handle multiple network namespaces

2007-09-28 Thread Eric W. Biederman
After the previous prep work this just consists of removing checks limiting the code to work in the initial network namespace, and updating rtmsg_ifinfo so we can generate events for devices in something other then the initial network namespace. Referring to network other network devices like

[PATCH 4/5] net: Make AF_PACKET handle multiple network namespaces

2007-09-28 Thread Eric W. Biederman
This is done by making packet_sklist_lock and packet_sklist per network namespace and adding an additional filter condition on received packets to ensure they came from the proper network namespace. Signed-off-by: Eric W. Biederman [EMAIL PROTECTED] --- include/net/net_namespace.h |4 +

Re: MSI interrupts and disable_irq

2007-09-28 Thread Jeff Garzik
Ayaz Abdulla wrote: I am trying to track down a forcedeth driver issue described by bug 9047 in bugzilla (2.6.23-rc7-git1 forcedeth w/ MCP55 oops under heavy load). I added a patch to synchronize the timer handlers so that one handler doesn't accidently enable the IRQ while another timer

Re: MSI interrupts and disable_irq

2007-09-28 Thread Stephen Hemminger
On Fri, 28 Sep 2007 22:47:16 -0400 Jeff Garzik [EMAIL PROTECTED] wrote: Ayaz Abdulla wrote: I am trying to track down a forcedeth driver issue described by bug 9047 in bugzilla (2.6.23-rc7-git1 forcedeth w/ MCP55 oops under heavy load). I added a patch to synchronize the timer handlers

Re: [RFC/PATCH 0/3] UDP memory usage accounting

2007-09-28 Thread Herbert Xu
Satoshi OSHIMA [EMAIL PROTECTED] wrote: In such case, from 300 to 500MB memory consumption will be fatal. Users can easily open 1000 sockets per process under default ulimit. If such sockets hold messages but user processes don't receive it. Almost all slab will be occupied by sk_buff. Well

Re: [PKT_SCHED]: Add stateless NAT

2007-09-28 Thread Herbert Xu
Herbert Xu [EMAIL PROTECTED] wrote: On Fri, Sep 28, 2007 at 06:55:32PM +0200, Patrick McHardy wrote: Looking at ip_input.o as example (everything without forced inlining): textdata bss dec hex filename 2076 8 02084 824 net/ipv4/ip_input.o 3483

Re: [PATCH] rtnl: Simplify ASSERT_RTNL

2007-09-28 Thread Herbert Xu
Eric W. Biederman [EMAIL PROTECTED] wrote: Currently we have the call path: macvlan_open - dev_unicast_add - __dev_set_rx_mode - __dev_set_promiscuity - ASSERT_RTNL - mutex_trylock When mutex debugging is on taking a mutex complains if we are not allowed to sleep. At that point we

Re: [PATCH] [3/6] pasemi_mac: set interface speed correctly on XAUI ports

2007-09-28 Thread Jeff Garzik
Olof Johansson wrote: pasemi_mac: set interface speed correctly on XAUI ports Set interface speed for XAUI to 10G per default, not 1G. Signed-off-by: Olof Johansson [EMAIL PROTECTED] applied 3-6 (davem already got 1-2) - To unsubscribe from this list: send the line unsubscribe netdev in

Re: [PATCH] mv643xx_eth: Check ETH_INT_CAUSE_STATE bit

2007-09-28 Thread Jeff Garzik
Dale Farnsworth wrote: Commit 468d09f8946d40228c56de26fe4874b2f98067ed masked the state interrupt (bit 20 of the cause register). This results in Radstone's PPC7D repeatedly re-entering the interrupt routine, locking up the board. The following patch returns the required handling for this

  1   2   >