[PATCH 35/59] drivers/net/wireless: Add missing "space"

2007-11-19 Thread Joe Perches
Signed-off-by: Joe Perches <[EMAIL PROTECTED]> --- drivers/net/wireless/b43/phy.c |2 +- drivers/net/wireless/b43legacy/phy.c |2 +- drivers/net/wireless/bcm43xx/bcm43xx_phy.c |2 +- drivers/net/wireless/libertas/wext.c |2 +- drivers/net/wireless/netwave_c

[PATCH 32/59] drivers/net/netxen: Add missing "space"

2007-11-19 Thread Joe Perches
Signed-off-by: Joe Perches <[EMAIL PROTECTED]> --- drivers/net/netxen/netxen_nic_hw.c |2 +- drivers/net/netxen/netxen_nic_init.c |4 ++-- drivers/net/netxen/netxen_nic_main.c |6 +++--- drivers/net/netxen/netxen_nic_niu.c |4 ++-- 4 files changed, 8 insertions(+), 8 deletions(

[PATCH 31/59] drivers/net/ixgb: Add missing "space"

2007-11-19 Thread Joe Perches
Signed-off-by: Joe Perches <[EMAIL PROTECTED]> --- drivers/net/ixgbe/ixgbe_common.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ixgbe/ixgbe_common.c b/drivers/net/ixgbe/ixgbe_common.c index 512e3b2..b7e50bc 100644 --- a/drivers/net/ixgbe/ixgbe_common.c ++

[PATCH 30/59] drivers/net/cxgb3: Add missing "space"

2007-11-19 Thread Joe Perches
Signed-off-by: Joe Perches <[EMAIL PROTECTED]> --- drivers/net/cxgb3/cxgb3_offload.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/cxgb3/cxgb3_offload.c b/drivers/net/cxgb3/cxgb3_offload.c index bd25421..7086f76 100644 --- a/drivers/net/cxgb3/cxgb3_offload

[PATCH 29/59] drivers/net/chelsio: Add missing "space"

2007-11-19 Thread Joe Perches
Signed-off-by: Joe Perches <[EMAIL PROTECTED]> --- drivers/net/chelsio/cxgb2.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/chelsio/cxgb2.c b/drivers/net/chelsio/cxgb2.c index 2dbf8dc..3eca4e3 100644 --- a/drivers/net/chelsio/cxgb2.c +++ b/drivers/net/chel

Re: [PATCH 0/5] fixups for mpc8360 rev. 2.1 erratum #2 (RGMII Timing)

2007-11-19 Thread Jeff Garzik
Kim Phillips wrote: On Mon, 5 Nov 2007 12:15:30 -0600 Kim Phillips <[EMAIL PROTECTED]> wrote: Hello all, the following patches fix RGMII timing for rev. 2.1 of the mpc8360, according to erratum #2 (erratum text included below). Basically the most intrusive part is the addition of two new RGMI

Problem with sysctl_tcp_congestion_control function

2007-11-19 Thread Sam Jansen
sysctl_tcp_congestion_control seems to have a bug that prevents it from actually calling the tcp_set_default_congestion_control function. This is not so apparent because it does not return an error and generally the /proc interface is used to configure the default TCP congestion control algorithm.

Re: [PATCH 0/5] fixups for mpc8360 rev. 2.1 erratum #2 (RGMII Timing)

2007-11-19 Thread Kim Phillips
On Mon, 5 Nov 2007 12:15:30 -0600 Kim Phillips <[EMAIL PROTECTED]> wrote: > Hello all, > > the following patches fix RGMII timing for rev. 2.1 of the mpc8360, > according to erratum #2 (erratum text included below). Basically the > most intrusive part is the addition of two new RGMII Internal De

Re: r8169 crash

2007-11-19 Thread Francois Romieu
Denys <[EMAIL PROTECTED]> : > Maybe it is good idea to print some registers/etc in watchdog message? > > I will try to reproduce problem with debug enabled, maybe it will be possible > to catch problem with more debug. Your customers will not necessarily like it but it could be useful to trace

Re: [patch 00/22] DM9000 updates for 2.6.25

2007-11-19 Thread Stephen Hemminger
On Mon, 19 Nov 2007 20:39:10 + Ben Dooks <[EMAIL PROTECTED]> wrote: > This is my current proposed patch series to the DM9000 > driver for both general cleanups, support for ethtool > and to make the code better. > You could also change all instances of dev->priv with netdev_priv(dev). --

Re: [PATCH 1/2] [IPV4] UDP: Always checksum even if without socket filter

2007-11-19 Thread David Miller
From: Andi Kleen <[EMAIL PROTECTED]> Date: Mon, 19 Nov 2007 16:29:33 +0100 > > > > > > > > All of our options suck, we just have to choose the least sucking one > > > > and right now to me that's decrementing the counter as much as I > > > > empathize with the SNMP application overflow detection

Re: 2.6.24-rc3: find complains about /proc/net

2007-11-19 Thread Rafael J. Wysocki
On Monday, 19 of November 2007, Pavel Machek wrote: > Hi! > > I think that this worked before: > > [EMAIL PROTECTED]:/proc# find . -name "timer_info" > find: WARNING: Hard link count is wrong for ./net: this may be a bug > in your filesystem driver. Automatically turning on find's -noleaf > opti

[patch 12/22] NET: DM9000: Remove EEPROM initialisation code.

2007-11-19 Thread Ben Dooks
Remove the old hack to program an initial EEPROM setting into the DM9000 as we now have ethtool support for reading and writing the EEPROM. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c

[patch 07/22] NET: DM9000: Use msleep() instead of udelay()

2007-11-19 Thread Ben Dooks
We can use sleeping functions when reading and writing the PHY registers, so let us sleep instead of busy waiting for the PHY. Note, this also fixes a bug reading the PHY where only 100uS was being used instead of 150uS Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/driv

[patch 14/22] NET: DM9000: Remove unnecessary changelog in header comment

2007-11-19 Thread Ben Dooks
We have a perfectly good version control system, so we do not need to duplicate change comments in the header for this code. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt2/drivers/net/dm9000.c === --- linux-2

[patch 01/22] NET: DM9000 use dev_xxx() instead of printk for output.

2007-11-19 Thread Ben Dooks
Move to using dev_dbg() and friends for the output of information to the user. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.22-quilt4/drivers/net/dm9000.c === --- linux-2.6.22-quilt4.orig/drivers/net/dm9000.c +++ lin

[patch 10/22] NET: DM9000: Add ethtool support for reading and writing EEPROM

2007-11-19 Thread Ben Dooks
Add ethtool support to access the configuration EEPROM connected to the DM9000. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c === --- linux-2.6.23-quilt3.orig/drivers/net/dm9000.c +++ li

[patch 03/22] NET: DM9000: Pass IRQ flags via platform data

2007-11-19 Thread Ben Dooks
Use the platform data to pass modifications to the flags passed to request_irq() to alter the behaviour of the IRQ. This can be used to specify IRQ type for the platform the DM9000 is connected to. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.22-quilt6/drivers/net/dm9000.c =

[patch 16/22] NET: DM9000: Fix delays used by EEPROM read and write

2007-11-19 Thread Ben Dooks
The code was using a delay of 8ms, when it should have been using the EEPROM status flag from the device to indicate the EEPROM transaction had finished. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c ==

[patch 02/22] NET: DM9000 update debugging macros to use debug level

2007-11-19 Thread Ben Dooks
Change the debug macros to use the compiler to elide any unnecessary debug level, and to allow device configurable debug control. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt1/drivers/net/Kconfig === --- lin

[patch 08/22] NET: DM9000: Remove barely used SROM array read.

2007-11-19 Thread Ben Dooks
The srom array in the board data is only being used in the device probe routines. The probe also only uses the first 6 bytes of an array we spend 512ms reading 128 bytes from. Change to reading the MAC area directly to the MAC address structure. As a side product, we rename the read_srom_word to d

[patch 15/22] NET: DM9000: Use netif_msg to enable debugging options

2007-11-19 Thread Ben Dooks
Use the netif_msg_*() macros to enable the debugging based on the board's msg_enable field. The output still goes via the dev_dbg() macros, so will be tagged and output as appropriate. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt4/drivers/net/dm9000.c ==

[patch 05/22] NET: DM9000: Do not sleep with spinlock and IRQs held

2007-11-19 Thread Ben Dooks
The phy read and write routines call udelay() with the board lock held, and with the posibility of IRQs being disabled. Since these delays can be up to 500usec, and are only required as we have to save the chip's address register. To improve the behaviour, hold the lock whilst we are writing and

[patch 00/22] DM9000 updates for 2.6.25

2007-11-19 Thread Ben Dooks
This is my current proposed patch series to the DM9000 driver for both general cleanups, support for ethtool and to make the code better. -- Ben ([EMAIL PROTECTED], http://www.fluff.org/) 'a smiley only costs 4 bytes' - To unsubscribe from this list: send the line "unsubscribe netdev" in the b

[patch 19/22] NET: DM9000: Add platform flag for no attached EEPROM

2007-11-19 Thread Ben Dooks
Allow the platform data to specify to the DM9000 driver that there is no posibility of an attached EEPROM on the device, so default all reads to 0xff and ignore any write operations. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c =

[patch 13/22] NET: DM9000: Ensure spinlock held whilst accessing EEPROM registers

2007-11-19 Thread Ben Dooks
Ensure we hold the spinlock whilst the registers and being modified even though we hold the overall lock. This should protect against an interrupt happening whilst we are using the device. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c ===

[patch 20/22] NET: DM9000: Add support for MII ioctl() calls

2007-11-19 Thread Ben Dooks
Add entry to handle the MII ioctl() calls via the generic_mii_ioctl call. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c === --- linux-2.6.23-quilt3.orig/drivers/net/dm9000.c +++ linux-2.

[patch 09/22] NET: DM9000: Add mutex to protect access

2007-11-19 Thread Ben Dooks
Add a mutex to serialise access to the chip functions from entries such as the ethtool and the MII code. This should reduce the amount of time the spinlock is held to protect the address register. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c ==

[patch 06/22] NET: DM9000: Use kthread to probe MII status when device open

2007-11-19 Thread Ben Dooks
When the device is open, we have to probe the PHY's MII status periodically as there is no status change interrupt. As the phy code is going to change to using calls which might sleep we move to using a kthread instead of a timer. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-

[patch 22/22] NET: DM9000: Show the MAC address source after printing MAC

2007-11-19 Thread Ben Dooks
Show whether the MAC address was read from the EEPROM or the onboard PAR registers. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt5/drivers/net/dm9000.c === --- linux-2.6.23-quilt5.orig/drivers/net/dm9000.c ++

[patch 18/22] NET: DM9000: Remove redudant use of "& 0xff"

2007-11-19 Thread Ben Dooks
The writing of the data should implicitly truncate the data to 8bits, so do not bother with the ands in the code. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c === --- linux-2.6.23-quilt

[patch 11/22] NET: DM9000: Add ethtool control of msg_enable value

2007-11-19 Thread Ben Dooks
Allow the msg_enable value to be read and written by the ethtool interface. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c === --- linux-2.6.23-quilt3.orig/drivers/net/dm9000.c +++ linux-

[patch 04/22] NET: DM9000: Add initial ethtool support

2007-11-19 Thread Ben Dooks
Add support for ethtool operations for the DM9000. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c === --- linux-2.6.23-quilt3.orig/drivers/net/dm9000.c +++ linux-2.6.23-quilt3/drivers/net

[patch 21/22] NET: DM9000: Update retry count whilst identifying chip

2007-11-19 Thread Ben Dooks
Reading the ID register does not always return the correct ID from the device, so we retry several times to see if we get a correct value. These failures seem to be excaserbated by the speed of the access to the chip (possibly time between issuing the address and then the data cycle). Signed-off-

[patch 17/22] NET: DM9000: Remove cal_CRC() and use ether_crc_le instead

2007-11-19 Thread Ben Dooks
Remove the cal_CRC as this is basically wrappering the ether_crc_le function, and is only being used by the multicast hash table functions. Signed-off-by: Ben Dooks <[EMAIL PROTECTED]> Index: linux-2.6.23-quilt3/drivers/net/dm9000.c

[PATCH 2.6.24 1/2]S2io: Strip the vlan tag if the vlan group is not NULL

2007-11-19 Thread Ramkrishna Vepa
- Updated the vlan tag stripping code as per Dave Johnson's patch <[EMAIL PROTECTED]> Below is the driver behaviour for vlan_tag_strip loadable paramter, vlan_tag_strip - 0: Don't strip the vlan tag vlan_tag_strip - 1: Always strip the vlan tag vlan_tag_strip - 2 (defaul

[PATCH 2.6.24 2/2]S2io: Fix to aggregate vlan tagged packets

2007-11-19 Thread Ramkrishna Vepa
- Fix to aggregate vlan packets. IP offset is incremented by 4 bytes if the packet contains vlan header. Signed-off-by: Santoshkumar Rastapur <[EMAIL PROTECTED]> Signed-off-by: Ramkrishna Vepa <[EMAIL PROTECTED]> --- diff -Nurp 2.0.26.8P1/drivers/net/s2io.c 2.0.26.8P2/drivers/net/s2io.c --- 2.0

Re: [PATCH 2.6.25 6/6] net: consolidate net namespace related proc files creation

2007-11-19 Thread Eric W. Biederman
"Denis V. Lunev" <[EMAIL PROTECTED]> writes: > net: consolidate net namespace related proc files creation > > Signed-off-by: Denis V. Lunev <[EMAIL PROTECTED]> > Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> Acked-by: "Eric W. Biederman" <[EMAIL PROTECTED]> Looks good to me, and it looks li

[PATCH] forcedeth boot delay fix

2007-11-19 Thread Ayaz Abdulla
This patch fixes a long boot delay in the forcedeth driver. During initialization, the timeout for the handshake between mgmt unit and driver can be very long. The patch reduces the timeout by eliminating a extra loop around the timeout logic. Signed-off-by: Ayaz Abdulla <[EMAIL PROTECTED]> -

Re: Netfilter: kernel panic with REDIRECT target. (2.6.23 and 2.6.23.8)

2007-11-19 Thread David
Evgeniy Polyakov wrote: > On Mon, Nov 19, 2007 at 10:24:23PM +0300, Evgeniy Polyakov ([EMAIL > PROTECTED]) wrote: > >> On Mon, Nov 19, 2007 at 06:51:38PM +, David ([EMAIL PROTECTED]) wrote: >> >>> Patrick McHardy wrote: >>> >>> iptables -t nat -A PREROUTING -j REDIRECT -

Re: Netfilter: kernel panic with REDIRECT target. (2.6.23 and 2.6.23.8)

2007-11-19 Thread Evgeniy Polyakov
On Mon, Nov 19, 2007 at 10:24:23PM +0300, Evgeniy Polyakov ([EMAIL PROTECTED]) wrote: > On Mon, Nov 19, 2007 at 06:51:38PM +, David ([EMAIL PROTECTED]) wrote: > > Patrick McHardy wrote: > > iptables -t nat -A PREROUTING -j REDIRECT -i eth2 -p udp --dport > > 5061 --to-ports 5060

Re: [PATCH] Compact sk_stream_mem_schedule() code

2007-11-19 Thread Arnaldo Carvalho de Melo
Em Mon, Nov 19, 2007 at 03:13:44PM +0300, Pavel Emelyanov escreveu: > This function references sk->sk_prot->xxx for many times. > It turned out, that there's so many code in it, that gcc > cannot always optimize access to sk->sk_prot's fields. > > After saving the sk->sk_prot on the stack and com

Re: Netfilter: kernel panic with REDIRECT target. (2.6.23 and 2.6.23.8)

2007-11-19 Thread Evgeniy Polyakov
On Mon, Nov 19, 2007 at 06:51:38PM +, David ([EMAIL PROTECTED]) wrote: > Patrick McHardy wrote: > iptables -t nat -A PREROUTING -j REDIRECT -i eth2 -p udp --dport > 5061 --to-ports 5060 > > > >>> Also post the kernel panic log. > >>> > > > > Please try if

[PATCH] dm9601: Consolidate common parts of dm_write_*_async

2007-11-19 Thread Peter Korsgaard
dm_write_async and dm_write_reg_async are almost identical. Move common functionality to dm_write_async_helper (saves ~256b). Signed-off-by: Peter Korsgaard <[EMAIL PROTECTED]> --- drivers/net/usb/dm9601.c | 53 +++-- 1 files changed, 13 insertions(+), 40

[PATCH] dm9601: Fix printk

2007-11-19 Thread Peter Korsgaard
A printk in the error handling code of dm9601.c was missing a newline. Signed-off-by: Peter Korsgaard <[EMAIL PROTECTED]> --- drivers/net/usb/dm9601.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/usb/dm9601.c b/drivers/net/usb/dm9601.c index 2c68573..1ffdd

Re: HTB/HSFC shaping precision

2007-11-19 Thread jamal
Denys, You certainly make a very compelling case. It is always compelling if you can translate a bug/feature into $$;->. So in your measurements, what kind of clock sources did you use? I think the parameters to worry about are: packet size, rate and clock source. I know that based on very old m

Re: [PATCH][PKTGEN] Fix double unlock of xfrm_state->lock

2007-11-19 Thread jamal
On Mon, 2007-19-11 at 12:47 +0300, Pavel Emelyanov wrote: > The pktgen_output_ipsec() function can unlock this lock twice > due to merged error and plain paths. Remove one of the calls > to spin_unlock. Good catch. Acked-by: Jamal Hadi Salim <[EMAIL PROTECTED]> cheers, jamal - To unsubscribe fro

Re: [PATCH 1/2] [IPV4] UDP: Always checksum even if without socket filter

2007-11-19 Thread Andi Kleen
> > > > > > All of our options suck, we just have to choose the least sucking one > > > and right now to me that's decrementing the counter as much as I > > > empathize with the SNMP application overflow detection issue. > > > > If the SNMP monitor detects an false overflow the error it reports

Get the IP address of an interface with a given name

2007-11-19 Thread Giacomo
/* returns in *addr the internet address having the name ifname */ int get_ifaddr_by_name(const char *ifname, __u32 * addr) { struct net_device *pnet_device; struct in_device *pin_device; struct in_ifaddr* inet_ifaddr; read_lock_bh(&dev_base_lock); #if LINUX_VER

[PATCH net-2.6 (2nd try)] [TCP] MTUprobe: fix potential sk_send_head corruption

2007-11-19 Thread Ilpo Järvinen
Please ignore the previous version, it had an rbtree artifact which made it to not apply cleanly, here's the corrected version: -- [PATCH] [TCP] MTUprobe: fix potential sk_send_head corruption When the abstraction functions got added, conversion here was made incorrectly. As a result, the skb ma

[PATCH net-2.6] [TCP] MTUprobe: fix potential sk_send_head corruption

2007-11-19 Thread Ilpo Järvinen
When the abstraction functions got added, conversion was made here incorrectly. As a result, the skb may end up pointing to skb which got included to the probe skb and then was freed. For it to trigger, however, transmit_skb must fail sending as well. Signed-off-by: Ilpo Järvinen <[EMAIL PROTECTE

Re: [PATCH] net/ipv4/arp.c: Fix arp reply when sender ip 0 (was: Strange behavior in arp probe reply, bug or feature?)

2007-11-19 Thread Alexey Kuznetsov
Hello! > Is there a reason that the target hardware address isn't the target > hardware address? It is bound only to the fact that linux uses protocol address of the machine, which responds. It would be highly confusing (more than confusing :-)), if we used our protocol address and hardware addre

[PATCH] Compact sk_stream_mem_schedule() code

2007-11-19 Thread Pavel Emelyanov
This function references sk->sk_prot->xxx for many times. It turned out, that there's so many code in it, that gcc cannot always optimize access to sk->sk_prot's fields. After saving the sk->sk_prot on the stack and comparing disassembled code, it turned out that the function became ~10 bytes sho

Re: [PATCH 1/2] [IPV4] UDP: Always checksum even if without socket filter

2007-11-19 Thread Arnaldo Carvalho de Melo
Em Mon, Nov 19, 2007 at 12:41:17PM +0800, Herbert Xu escreveu: > On Sun, Nov 18, 2007 at 02:40:10PM -0800, David Miller wrote: > > > > This can be fixed, the above cannot. > > That's a good point. Perhaps one way of getting that info to > the user without putting it in UDPInDatagrams is to create

Re: [PATCH][net-2.6.25][NETNS] make netns cleanup to run in a separate workqueue

2007-11-19 Thread Kirill Korotaev
Ah, sorry. Didn't notice it's called only on boot. Acked-By: Kirill Korotaev <[EMAIL PROTECTED]> Kirill Korotaev wrote: > imho panic() is too much. > create_singlethread_workqueue() can fail e.g. due to out of memory... > > Thanks, > Kirill > > > Daniel Lezcano wrote: > >>Subject: make netns

Re: [PATCH][net-2.6.25][NETNS] make netns cleanup to run in a separate workqueue

2007-11-19 Thread Kirill Korotaev
imho panic() is too much. create_singlethread_workqueue() can fail e.g. due to out of memory... Thanks, Kirill Daniel Lezcano wrote: > Subject: make netns cleanup to run in a separate queue > From: Benjamin Thery <[EMAIL PROTECTED]> > > This patch adds a separate workqueue for cleaning up a net

[PATCH][net-2.6.25][NETNS] make netns cleanup to run in a separate workqueue

2007-11-19 Thread Daniel Lezcano
Subject: make netns cleanup to run in a separate queue From: Benjamin Thery <[EMAIL PROTECTED]> This patch adds a separate workqueue for cleaning up a network namespace. If we use the keventd workqueue to execute cleanup_net(), there is a problem to unregister devices in IPv6. Indeed the code

Re: [PATCH 2/2] virtio: free transmit skbs when notified, not on next xmit.

2007-11-19 Thread Christian Borntraeger
Am Montag, 19. November 2007 schrieb Rusty Russell: > This fixes a potential dangling xmit problem. > > We also suppress refill interrupts until we need them. > (Anthony and I have been working on performance recently, so this > is a WIP). > > Signed-off-by: Rusty Russell <[EMAIL PROTECTED]> Tha

[PATCH][IPVS] Relax the module get/put in ip_vs_app.c

2007-11-19 Thread Pavel Emelyanov
Both try_module_get/module_put already handle the module == NULL case, so no need in manual checking. This patch fits both net-2.6 and net-2.6.25. Signed-off-by: Pavel Emelyanov <[EMAIL PROTECTED]> --- diff --git a/net/ipv4/ipvs/ip_vs_app.c b/net/ipv4/ipvs/ip_vs_app.c index 664cb8e..535abe0 100

HTB/HSFC shaping precision

2007-11-19 Thread Denys Fedoryshchenko
Hi 2 all again This is not a bug report this time :-) Just it is very interesting question, about using Linux "shaping" technologies in serious jobs. What i realised few days ago, many ISP's set on their STM-1(15552 bits/s) links (over Cisco) packet buffer/queue 40 packets(for example). It m

Re: [PATCH 1/1]: SAs created although tmpl->optional set

2007-11-19 Thread Herbert Xu
On Mon, Nov 19, 2007 at 02:53:50AM -0600, Joy Latten wrote: > Although tmpl->optional is set, kernel still attempts > to create a set of SAs. Heh I made the same mistake when I first read this piece of code too :) The optional flag isn't saying that it doesn't need to be protected, but rather tha

[PATCH][PKTGEN] Fix double unlock of xfrm_state->lock

2007-11-19 Thread Pavel Emelyanov
The pktgen_output_ipsec() function can unlock this lock twice due to merged error and plain paths. Remove one of the calls to spin_unlock. Other possible solution would be to place "return 0" right after the first unlock, but at this place the err is known to be 0, so these solutions are the sam

Re: r8169 crash

2007-11-19 Thread Denys
Maybe it is good idea to print some registers/etc in watchdog message? I will try to reproduce problem with debug enabled, maybe it will be possible to catch problem with more debug. On Sun, 18 Nov 2007 22:00:38 +0100, Francois Romieu wrote > Denys <[EMAIL PROTECTED]> : > > > > Before it happe

[PATCH 1/1]: SAs created although tmpl->optional set

2007-11-19 Thread Joy Latten
Although tmpl->optional is set, kernel still attempts to create a set of SAs. In xfrm_tmpl_resolve_one(), xfrm_state_find() is called to find an SA. First time, there won't be an SA, so an ACQUIRE will be sent and code then returns to xfrm_tmpl_resolv_one() who then checks tmpl->optional. Since

Re: [PATCH 1/2] [IPV4] UDP: Always checksum even if without socket filter

2007-11-19 Thread David Miller
From: Herbert Xu <[EMAIL PROTECTED]> Date: Mon, 19 Nov 2007 12:41:17 +0800 > On Sun, Nov 18, 2007 at 02:40:10PM -0800, David Miller wrote: > > > > This can be fixed, the above cannot. > > That's a good point. Perhaps one way of getting that info to > the user without putting it in UDPInDatagrams

Re: [PATCH 1/2] [IPV4] UDP: Always checksum even if without socket filter

2007-11-19 Thread David Miller
From: Wang Chen <[EMAIL PROTECTED]> Date: Mon, 19 Nov 2007 10:02:03 +0800 > So if "udp_queue_rcv_skb() doing sucessfully" means "a UDP > datagrams delivered to UDP users", the InDatagrams should be > increased in udp_queue_rcv_skb(). > Otherwise it should be increased until the UDP datagrams is >

Re: [PATCH 1/2] [IPV4] UDP: Always checksum even if without socket filter

2007-11-19 Thread David Miller
From: Herbert Xu <[EMAIL PROTECTED]> Date: Mon, 19 Nov 2007 09:09:29 +0800 > In any case, I just looked up RFC1213 and it says: > > udpInDatagrams OBJECT-TYPE > SYNTAX Counter > ACCESS read-only > STATUS mandatory > DESCRIPTION

<    1   2