Re: [PATCHv2 2.6.18-rc1-mm2 1/3] net: UDP-Lite generic support

2006-07-16 Thread Gerrit Renker
Quoting Herbert Xu: |                case SO_NO_CHECK: | -                       sk-sk_no_check = valbool; | +                       /* UDP-Lite (RFC 3828) mandates checksumming, | +                        * hence user must not enable this option.   */ | +                       if

Re: [PATCH] clear skb cb on IP input

2006-07-16 Thread chas williams - CONTRACTOR
In message [EMAIL PROTECTED],David Miller writes: From: Herbert Xu [EMAIL PROTECTED] At least this lets us get rid of a few other memsets :) Applied, good spotting :-) I remember when we added those things. But I'm beginning to think that the onus of this may in fact fall upon the devices, in

Re: [PATCH] WAN: converting generic HDLC to use netif_dormant*()

2006-07-16 Thread Krzysztof Halasa
David Miller [EMAIL PROTECTED] writes: WARNING: hdlc_set_carrier [drivers/char/synclinkmp.ko] undefined! WARNING: hdlc_set_carrier [drivers/char/synclink_gt.ko] undefined! WARNING: hdlc_set_carrier [drivers/char/synclink.ko] undefined! WARNING: hdlc_set_carrier

[PATCH] WAN: Added missing netif_dormant_off() to generic HDLC

2006-07-16 Thread Krzysztof Halasa
WAN: Fixed a problem with PPP/raw HDLC/X.25 protocols not doing netif_dormant_off() at startup. Signed-off-by: Krzysztof Halasa [EMAIL PROTECTED] diff --git a/drivers/net/wan/hdlc_ppp.c b/drivers/net/wan/hdlc_ppp.c index b81263e..fbaab5b 100644 --- a/drivers/net/wan/hdlc_ppp.c +++

[PATCH] WAN: Cosmetic changes to N2 and C101 drivers

2006-07-16 Thread Krzysztof Halasa
WAN: Cosmetic changes to N2 and C101 drivers Signed-off-by: Krzysztof Halasa [EMAIL PROTECTED] diff --git a/drivers/net/wan/c101.c b/drivers/net/wan/c101.c index 3796a59..2911be3 100644 --- a/drivers/net/wan/c101.c +++ b/drivers/net/wan/c101.c @@ -197,7 +197,6 @@ static int c101_open(struct

[PATCH] WAN: Converted synclink drivers to use netif_carrier_*()

2006-07-16 Thread Krzysztof Halasa
WAN: Converted synclink drivers to use netif_carrier_*() instead of hdlc_set_carrier(). Signed-off-by: Krzysztof Halasa [EMAIL PROTECTED] diff --git a/drivers/char/pcmcia/synclink_cs.c b/drivers/char/pcmcia/synclink_cs.c index 0721345..bdd09d9 100644 --- a/drivers/char/pcmcia/synclink_cs.c +++

Re: [2.6 patch] drivers/net/wireless/zd1211rw/: possible cleanups

2006-07-16 Thread Daniel Drake
Adrian Bunk wrote: This patch contains the following possible cleanups: - make needlessly global functions static - #if 0 unused functions Please review which of these functions do make sense and which do conflict with pending patches. Thanks Adrian. I have put this in my tree and made an

Re: [PATCH 3/7] NetLabel: CIPSOv4 engine

2006-07-16 Thread Paul Moore
On Friday 14 July 2006 10:03 pm, James Morris wrote: On Fri, 14 Jul 2006, [EMAIL PROTECTED] wrote: +/** + * cipso_v4_bitmap_walk - Walk a bitmap looking for a bit + * cipso_v4_bitmap_setbit - Sets a single bit in a bitmap Can you use lib/bitmap.c instead? Looking again at

Re: [2.6.18-rc2][e1000][swsusp] - Regression - Suspend to disk and resume breaks e1000

2006-07-16 Thread Auke Kok
Shawn Starr wrote: Hardware IBM ThinkPad T42 E1000 card info: 02:01.0 Ethernet controller: Intel Corporation 82540EP Gigabit Ethernet Controller (Mobile) (rev 03) Subsystem: IBM PRO/1000 MT Mobile Connection Flags: bus master, 66MHz, medium devsel, latency 64, IRQ 9

Re: [2.6.18-rc2][e1000][swsusp] - Regression - Suspend to disk and resume breaks e1000

2006-07-16 Thread Shawn Starr
On Sunday 16 July 2006 12:33, Auke Kok wrote: [adding netdev to the cc] unfortunately I didn't. e1000 has a special e1000_pci_save_state/e1000_pci_restore_state set of routines that save and restore the configuration space. the fact that it works for suspend to memory to me suggests that

[PATCH] Fix slab corruption with netem (2nd try)

2006-07-16 Thread Guillaume Chazarain
CONFIG_DEBUG_SLAB found the following bug: netem_enqueue() in sch_netem.c gets a pointer inside a slab object: struct netem_skb_cb *cb = (struct netem_skb_cb *)skb-cb; But then, the slab object may be freed: skb = skb_unshare(skb, GFP_ATOMIC) cb is still pointing inside the freed skb, so here is

Re: [PATCH] clear skb cb on IP input

2006-07-16 Thread Herbert Xu
On Sat, Jul 15, 2006 at 06:12:22PM -0700, David Miller wrote: But I'm beginning to think that the onus of this may in fact fall upon the devices, in fact. Loopback is one of the few devices where the control block might not be cleared out, due to uses in the output path. Devices

Re: [PATCH 3/7] NetLabel: CIPSOv4 engine

2006-07-16 Thread David Miller
From: Paul Moore [EMAIL PROTECTED] Date: Sun, 16 Jul 2006 12:10:44 -0400 On Friday 14 July 2006 10:03 pm, James Morris wrote: On Fri, 14 Jul 2006, [EMAIL PROTECTED] wrote: +/** + * cipso_v4_bitmap_walk - Walk a bitmap looking for a bit + * cipso_v4_bitmap_setbit - Sets a single bit

Re: [PATCH] clear skb cb on IP input

2006-07-16 Thread David Miller
From: Herbert Xu [EMAIL PROTECTED] Date: Mon, 17 Jul 2006 08:03:50 +1000 The thing is qdiscs using cb means that this method of clearing cb before netif_rx doesn't work anymore. In particular, even if loopback clears cb before calling netif_rx, some qdisc could come along between netif_rx

Re: [PATCH] WAN: Added missing netif_dormant_off() to generic HDLC

2006-07-16 Thread David Miller
From: Krzysztof Halasa [EMAIL PROTECTED] Date: Sun, 16 Jul 2006 14:17:20 +0200 WAN: Fixed a problem with PPP/raw HDLC/X.25 protocols not doing netif_dormant_off() at startup. Signed-off-by: Krzysztof Halasa [EMAIL PROTECTED] Applied, thanks. - To unsubscribe from this list: send the line

Re: [PATCH] WAN: Cosmetic changes to N2 and C101 drivers

2006-07-16 Thread David Miller
From: Krzysztof Halasa [EMAIL PROTECTED] Date: Sun, 16 Jul 2006 14:19:07 +0200 WAN: Cosmetic changes to N2 and C101 drivers Signed-off-by: Krzysztof Halasa [EMAIL PROTECTED] Applied, thanks. - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

Re: [PATCH] WAN: Converted synclink drivers to use netif_carrier_*()

2006-07-16 Thread David Miller
From: Krzysztof Halasa [EMAIL PROTECTED] Date: Sun, 16 Jul 2006 14:23:49 +0200 WAN: Converted synclink drivers to use netif_carrier_*() instead of hdlc_set_carrier(). Signed-off-by: Krzysztof Halasa [EMAIL PROTECTED] Also applied, thanks for fixing this up Krzysztof. - To unsubscribe from

Re: [PATCH] Fix slab corruption with netem (2nd try)

2006-07-16 Thread David Miller
From: Guillaume Chazarain [EMAIL PROTECTED] Date: Sun, 16 Jul 2006 23:56:31 +0200 CONFIG_DEBUG_SLAB found the following bug: netem_enqueue() in sch_netem.c gets a pointer inside a slab object: struct netem_skb_cb *cb = (struct netem_skb_cb *)skb-cb; But then, the slab object may be freed:

Re: [PATCH] clear skb cb on IP input

2006-07-16 Thread David Miller
From: chas williams - CONTRACTOR [EMAIL PROTECTED] Date: Sun, 16 Jul 2006 07:20:35 -0400 why does the input side of the ip layer believe the cb contains valid data? it should zero the contents of the cb, or just fill in the cb correctly when the packet arrives at its doorstop. Yes, that's

Re: [PATCH 3/7] NetLabel: CIPSOv4 engine

2006-07-16 Thread Paul Moore
On Sunday 16 July 2006 9:12 pm, David Miller wrote: From: Paul Moore [EMAIL PROTECTED] Date: Sun, 16 Jul 2006 12:10:44 -0400 On Friday 14 July 2006 10:03 pm, James Morris wrote: On Fri, 14 Jul 2006, [EMAIL PROTECTED] wrote: +/** + * cipso_v4_bitmap_walk - Walk a bitmap looking for

Re: [PATCH 3/7] NetLabel: CIPSOv4 engine

2006-07-16 Thread David Miller
From: Paul Moore [EMAIL PROTECTED] Date: Sun, 16 Jul 2006 22:42:07 -0400 Right now I use both the bitmap_walk() and bitmap_setbit() routines to deal with both CIPSO tags straight from the sk_buff as well as the internal bitmap representation. Padding out the internal bitmaps would require

[PATCH] Bug in pskb_trim_rcsum()

2006-07-16 Thread Wei Yongjun
Since network device can auto calculate and verify the checksum of a packet, for example: some e1000 interface. Different device will set different value of skb-ip_summed. a) If device do nothing to checksum, skb-ip_summed would be set to CHECKSUM_NONE. b) If device can only calculate a