[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 che

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 requi

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 bi

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, tha

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 free

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

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

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

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 neti

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 - S

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 predom

[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: [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

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 M

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 inc

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 add

[PATCH] Add zd1211rw MAINTAINERS entry

2006-07-16 Thread Daniel Drake
Hopefully this will help people like Adrian Bunk send patches where the maintainers will see them :) Signed-off-by: Daniel Drake <[EMAIL PROTECTED]> diff --git a/MAINTAINERS b/MAINTAINERS index e99028c..48a5c02 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3298,6 +3298,15 @@ W: http://www.

[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 +++

[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 net

[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 +++ b/drivers/net

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" [drivers/char/

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 >th

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 (