Re: [RFT] proxy arp deadlock possible

2007-04-05 Thread Herbert Xu
] neigh_proxy_process+0x66/0xc2 [c042f053] run_timer_softirq+0x104/0x168 [c042c1f6] __do_softirq+0x6f/0xe2 [c04063c6] do_softirq+0x61/0xd0 === Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert

[SC92031]: Fix priv-lock context (was: 2.6.21-rc5-mm4)

2007-04-05 Thread Herbert Xu
with BH disabled. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list

Re: tg3 reset_task question

2007-04-05 Thread Herbert Xu
have to figure out whether you're in an RTNL context or not (e.g., tg3_suspend would also need to grab it). Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert

Re: IPsec PMTUD problem

2007-04-05 Thread Herbert Xu
On Thu, Apr 05, 2007 at 02:16:53PM +0200, Patrick McHardy wrote: That sounds easier. I'm currently working in that area anyway, I'll give it a try. Thanks Patrick! -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: IPsec PMTUD problem

2007-04-05 Thread Herbert Xu
that. One such case is delivery of errors to sockets. We'd need to make sure the errors are delivered some other way. Alternatively we can still send the ICMP error but avoid a PMTU update if we received it from ourselves. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert

Re: IPsec PMTUD problem

2007-04-05 Thread Herbert Xu
possible solution is to not send MTU errors to ourselves since we it wouldn't give us any new information. We'd need to audit the users of icmp_send to make sure that there isn't a legitimate case where we'd want to do that. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu

Re: [SC92031]: Fix priv-lock context

2007-04-05 Thread Herbert Xu
and dev-close must disable BH since open/close are made in process context. Conversely, the call in dev-hard_start_xmit does not need to disable BH since it is already executing with BH disabled. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org

Re: [PATCH 2/2] NET: Multiqueue network device support implementation.

2007-04-10 Thread Herbert Xu
for consistency in this particular area. If you'd like me to remove it, I can do that. The other cast is there for the subtraction, not the kfree... Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key

[NET]: Get rid of NETIF_F_INTERNAL_STATS

2007-04-11 Thread Herbert Xu
-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- diff --git a/include/linux/netdevice.h b/include/linux

Re: [PATCH 2/2] NET: Multiqueue network device support.

2007-04-11 Thread Herbert Xu
; Shall I remove my cast? This is totally different as it's casting a long, not a void *. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

Re: [NET]: Get rid of NETIF_F_INTERNAL_STATS

2007-04-11 Thread Herbert Xu
at the wrong tree. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body

Re: TCP connection stops after high load.

2007-04-13 Thread Herbert Xu
/sniffers can 'see' packet transmited several times. This should only happen with LLTX drivers. In fact, LLTX drivers are really more trouble than they're worth. They should all be rewritten to follow the model used in tg3. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu

Re: kernel BUG at net/core/skbuff.c in linux-2.6.21-rc6

2007-04-14 Thread Herbert Xu
) and it does match the crash dump. In fact, there is only a single skb_push (the only thing that can generate an under panic) in ppp_async.c and this is the one that Patrick has fixed. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page

[NET]: Get rid of alloc_skb code duplication

2007-04-17 Thread Herbert Xu
as they've started diverging. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- diff --git a/net

Re: [NET]: Get rid of alloc_skb code duplication

2007-04-17 Thread Herbert Xu
Since this was added originally for Xen, and Xen has recently (~2.6.18) stopped using this function, we can safely get rid of it. Good timing too since this function has started to bit rot. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email

Re: kernel BUG at net/core/skbuff.c in linux-2.6.21-rc6

2007-04-18 Thread Herbert Xu
still have a problem there. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev

Re: [NETLINK] Don't attach callback to a going-away netlink socket

2007-04-18 Thread Herbert Xu
receive packets until all references to the socket are gone, we don't need to kill the CB until that happens. This also aligns ourselves with the receive queue purging which happens at that point. Original patch by Pavel Emelianov who noticed this race condition. Signed-off-by: Herbert Xu [EMAIL

Re: kernel BUG at net/core/skbuff.c in linux-2.6.21-rc6

2007-04-19 Thread Herbert Xu
Herbert Xu [EMAIL PROTECTED] wrote: Paul Mackerras [EMAIL PROTECTED] wrote: So this doesn't change process_input_packet(), which treats the case where the first byte is 0xff (PPP_ALLSTATIONS) but the second byte is 0x03 (PPP_UI) as indicating a packet with a PPP protocol number of 0xff

Re: Getting the new RxRPC patches upstream

2007-04-19 Thread Herbert Xu
David Miller [EMAIL PROTECTED] wrote: I applied already the patches I thought were appropriate, you had some crypto layer changes that you need to work out with Herbert Xu before the rest can be applied. He has already fixed it by using the scatterlist interface for now. So the last set

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-23 Thread Herbert Xu
this into a workqueue. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body

[NET]: Warn about GSO/checksum abuse

2007-04-23 Thread Herbert Xu
and remove the gso_send_check cruft. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- diff

Re: [Bugme-new] [Bug 8057] New: slab corruption running ip6sic

2007-04-24 Thread Herbert Xu
, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

Re: [PATCH] netdev: get rid of casts

2007-04-24 Thread Herbert Xu
. Certainly it is more for human consumption than compiler consumption, but nonetheless... How about just getting rid of the casts? 'arg' is already void * to begin with. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: netlink locking warnings in 2.6.21-rc7-mm1

2007-04-24 Thread Herbert Xu
for a mutex anyway this would seem to be strictly worse than just using our own mutex. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

Re: [net-2.6.22] [IPV4]: Fix build without procfs.

2007-04-24 Thread Herbert Xu
to do with proc so this patch moves it out of proc.c and to the only place that uses it. Right now there is only one user so I'ved made it static too. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED

Re: [net-2.6.22] [IPV4]: Fix build without procfs.

2007-04-24 Thread Herbert Xu
On Wed, Apr 25, 2007 at 12:35:49PM +1000, Herbert Xu wrote: [IPV4]: Move snmp_mib_init out of proc.c The function snmp_mib_init has nothing to do with proc so this patch moves it out of proc.c and to the only place that uses it. Right now there is only one user so I'ved made it static too

[1/2] [IPV4]: Consolidate common SNMP code

2007-04-24 Thread Herbert Xu
Hi: [IPV4]: Consolidate common SNMP code This patch moves the SNMP code shared between IPv4/IPv6 from proc.c into net/ipv4/af_inet.c. This makes sense because these functions aren't specific to /proc. As a result we can again skip proc.o if /proc is disabled. Signed-off-by: Herbert Xu [EMAIL

[2/2] [IPV6]: Consolidate common SNMP code

2007-04-24 Thread Herbert Xu
. If they ever get used elsewhere we can always remove the static. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au

Re: [net-2.6.22] [IPV4]: Fix build without procfs.

2007-04-24 Thread Herbert Xu
:) Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

Re: [2/2] [IPV6]: Consolidate common SNMP code

2007-04-24 Thread Herbert Xu
turned off :) Thanks for fixing it up! Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe

Re: 2.6.20.7 mss negotiation and path mtu discovery mostly broken?

2007-04-25 Thread Herbert Xu
was this dump taken, on 10.2.10.254? If so could youd either take the dump further down the route or show the full contents (with tcpdump -x) of the ICMP error here so that we can see what the actual packet size was? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL

Re: [NET]: Get rid of NETIF_F_INTERNAL_STATS

2007-04-27 Thread Herbert Xu
just had a look at br_if.c and it seems that it does set get_stats through br_dev_setup to br_dev_get_stats. == Remove NETIF_F_INTERNAL_STATS: default to internal stats. Herbert Xu conviced me that a new flag was overkill; every driver currently overrides get_stats, so we might as well make

Re: netfront for review

2007-05-01 Thread Herbert Xu
it you'll just get an ICMP error back or fragmentation. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send

[NETFILTER] sip: Fix RTP address NAT

2007-05-01 Thread Herbert Xu
was unnecessary but OK. However, in doing so it did not DNAT the destination address on the RTP traffic to the Cisco back to the original RTP address. This patch corrects this by noting down the RTP address and using it when the expectation fires. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers

Re: [NETLINK] Don't attach callback to a going-away netlink socket

2007-05-01 Thread Herbert Xu
are gone, we don't need to kill the CB until that happens. This also aligns ourselves with the receive queue purging which happens at that point. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page

[NETLINK]: Remove references to process ID

2007-05-04 Thread Herbert Xu
to it as the port ID which hopefully will make it clear what the purpose of the fields really is. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http

Re: [patch 25/29] xen: Add the Xen virtual network device driver.

2007-05-05 Thread Herbert Xu
, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

[1/2] [NET] link_watch: Move link watch list into net_device

2007-05-08 Thread Herbert Xu
-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- a6c194d06da9aed2a8f5a4ea07e3cbf9266db4ef diff --git

Re: [1/2] [NET] link_watch: Move link watch list into net_device

2007-05-08 Thread Herbert Xu
transition, and so the timing concern has been solved. Signed-off-by: Jeremy Fitzhardinge [EMAIL PROTECTED] Cc: Herbert Xu [EMAIL PROTECTED] Cc: Keir Fraser [EMAIL PROTECTED] Looks good to me. Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home

Re: linkwatch bustage in git-net

2007-05-08 Thread Herbert Xu
, I don't see it here (probably because we use different NICs). But does this help? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- diff

Re: linkwatch bustage in git-net

2007-05-08 Thread Herbert Xu
On Wed, May 09, 2007 at 03:31:55PM +1000, Herbert Xu wrote: Hmm, I don't see it here (probably because we use different NICs). But does this help? Thinking about it again I don't think it will help you because if your carrier started out as off then it would've been considered an urgent event

Re: linkwatch bustage in git-net

2007-05-08 Thread Herbert Xu
? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message

Re: linkwatch bustage in git-net

2007-05-09 Thread Herbert Xu
exclusion for the setting linkwatch_nextevent which is available in the actual work function. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http

Re: [patch 7/9] lguest: the net driver

2007-05-09 Thread Herbert Xu
://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info

Re: [patch 7/9] lguest: the net driver

2007-05-09 Thread Herbert Xu
/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http

Re: SG_IO with 4k buffer size to iscsi sg device causes Bad page panic

2007-05-09 Thread Herbert Xu
resSz; m += PAGE_SIZE ) + { + tmppage++; + SetPageReserved(tmppage); + } + Why not just increase the page use count? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: [PATCH] sched: Optimize return value of qdisc_restart

2007-05-10 Thread Herbert Xu
by Krishna Kumar. Signed-off-by: [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- diff --git a/net/sched/sch_generic.c b/net

Re: [NET]: Fix dev-qdisc race for NETDEV_TX_LOCKED case

2007-05-10 Thread Herbert Xu
at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More

Re: [NET]: Fix dev-qdisc race for NETDEV_TX_LOCKED case

2007-05-10 Thread Herbert Xu
. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message

Re: [PATCH] sched: Optimize return value of qdisc_restart

2007-05-10 Thread Herbert Xu
Now that we return the queue length after NETDEV_TX_OK we better make sure that we have the right queue. Otherwise we can cause a stall after a really quick dev_deactive/dev_activate. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Chers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu

[NET_SCHED]: Avoid requeue warning on dev_deactivate

2007-05-10 Thread Herbert Xu
after we wait for __LINK_STATE_QDISC_RUNNING to clear (but it won't do anything as long as the queue and gso_skb is already clear). Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: [PATCH] sched: Optimize return value of qdisc_restart

2007-05-10 Thread Herbert Xu
words it sits there for an indeterminate period. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line

Re: [PATCH] sched: Optimize return value of qdisc_restart

2007-05-10 Thread Herbert Xu
of contention we should stay in the function anyway. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send

Re: [PATCH] sched: Optimize return value of qdisc_restart

2007-05-10 Thread Herbert Xu
, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED

Re: [PATCH] ppp: don't use 0 in pointer context

2006-02-08 Thread Herbert Xu
up. -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

Re: [PATCH] ppp: don't use 0 in pointer context

2006-02-08 Thread Herbert Xu
); or func(foo, (int *)0); Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev

Re: [PATCH] ppp: don't use 0 in pointer context

2006-02-08 Thread Herbert Xu
the last two should be doable there as well. The difference between gcc -pedantic and sparse is that it doesn't warn about obviously correct cases like p != 0 or p = 0. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: [PATCH] ppp: don't use 0 in pointer context

2006-02-08 Thread Herbert Xu
On Wed, Feb 08, 2006 at 03:58:59PM -0800, David S. Miller wrote: From: Herbert Xu [EMAIL PROTECTED] Date: Thu, 09 Feb 2006 10:49:37 +1100 The difference between gcc -pedantic and sparse is that it doesn't warn about obviously correct cases like p != 0 or p = 0. So obviously correct

Re: [PATCH] check connect(2) status for IPv6 UDP socket

2006-02-08 Thread Herbert Xu
in ip6_dst_lookup is not enough for IPsec because it only checks the destination address and oif instead of all the addresses/protocol/ports. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http

Re: [RFC,NETFILTER]: Fix xfrm lookup after SNAT

2006-02-13 Thread Herbert Xu
-by: Patrick McHardy [EMAIL PROTECTED] Signed-off-by: Herbert Xu [EMAIL PROTECTED] Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe

Re: [PATCH] [IPV4] PMTU issues due to TOS field manipulation

2006-02-15 Thread Herbert Xu
the code cleaner, wouldn't it? It would too. However, routing by TOS is useful and it isn't really the problem here. The problem is its use in the hash calculations. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: [XFRM 2.6.16]: Fix policy double put

2006-02-19 Thread Herbert Xu
://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http

Re: [PATCH 01/02] change fwmark code to document use of attribute

2006-02-20 Thread Herbert Xu
the enum as RTA_FWMARK = RTA_PROTOINFO, Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line

Re: [PATCH 01/02] change fwmark code to document use of attribute

2006-02-20 Thread Herbert Xu
Michael Richardson [EMAIL PROTECTED] wrote: I'll regenerate the patch? Who is the prime on this file? Send all networking patches to [EMAIL PROTECTED] with a cc to this list. Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: Semantics of current lifetime in IPsec SA

2006-02-21 Thread Herbert Xu
this is intentional. We want to have the SAs expire after a set period of time after their first use as opposed to the last. Usually you should be to use the time on the SP instead. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: Semantics of current lifetime in IPsec SA

2006-02-21 Thread Herbert Xu
for each peer there can always be multiple SAs/policies to track. So you only want to probe when all of them go idle. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au

Re: Semantics of current lifetime in IPsec SA

2006-02-21 Thread Herbert Xu
think it's asking a lot to add a more specific policy for the peers that you want to DPD with. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert

Re: XFRM_STATE_NOPMTUDISC, was : MTU/DF problem with 2.6

2006-02-21 Thread Herbert Xu
- need to frag (mtu 1492) 22:27:00.533628 x.x.x.x y.y.y.y: ESP(spi=0x95b26645,seq=0x145) (DF) [tos 0x8] 22:27:00.578825 z.z.z.z x.x.x.x: icmp: y.y.y.y unreachable - need to frag (mtu 1492) Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home

Re: XFRM_STATE_NOPMTUDISC, was : MTU/DF problem with 2.6

2006-02-21 Thread Herbert Xu
-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt diff --git a/net/ipv4/xfrm4_policy.c b/net/ipv4/xfrm4_policy.c

Re: XFRM_STATE_NOPMTUDISC, was : MTU/DF problem with 2.6

2006-02-21 Thread Herbert Xu
On Tue, Feb 21, 2006 at 11:43:08PM +0100, Beschorner Daniel wrote: [IPSEC] Use TOS when doing tunnel lookups We should use the TOS because it's one of the routing keys. It also means that we update the correct routing cache entry when PMTU occurs. Signed-off-by: Herbert Xu [EMAIL

[IPSEC]: Use TOS when doing tunnel lookups

2006-02-21 Thread Herbert Xu
is the final patch which has been tested successfully. [IPSEC]: Use TOS when doing tunnel lookups We should use the TOS because it's one of the routing keys. It also means that we update the correct routing cache entry when PMTU occurs. Signed-off-by: Herbert Xu [EMAIL PROTECTED] BTW, we really

Re: XFRM_STATE_NOPMTUDISC, was : MTU/DF problem with 2.6

2006-02-22 Thread Herbert Xu
On Wed, Feb 22, 2006 at 10:39:03AM +0200, Ilia Sotnikov wrote: On 2/21/06, Herbert Xu [EMAIL PROTECTED] wrote: Actually, here is a better patch for your problem. We should really be using the TOS when doing IPsec lookups: [IPSEC] Use TOS when doing tunnel lookups We should use

Re: [PATCH] Allow kfree_skb to be called with a NULL argument

2006-02-23 Thread Herbert Xu
on the other hand is inlined so the you're actually adding bloat to many places that simply don't need it. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert

Re: [PATCH] Allow kfree_skb to be called with a NULL argument

2006-02-23 Thread Herbert Xu
On Thu, Feb 23, 2006 at 12:22:31PM +0100, J?rn Engel wrote: Should I merge the two patches into one and resend? Sounds good. -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http

Re: Problem with Ipsec transport mode over NAT

2006-02-24 Thread Herbert Xu
it shouldn't be too hard to fix up, for netlink at least. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line

[1/2] [IPSEC] esp: Kill unnecessary block and indentation

2006-02-24 Thread Herbert Xu
kill the block and save some indentation. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt diff

[2/2] [IPSEC]: Kill post_input hook and do NAT-T in esp_input directly

2006-02-24 Thread Herbert Xu
in the decap data structures for now since it's intricately woven into the sec_path stuff. We can kill them later too. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: IPv6: avoid dereference of null route entry in ipv6_del_addr()

2006-03-07 Thread Herbert Xu
://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info

Re: [Patch 1/1] updated: TCP/UDP getpeersec

2006-03-08 Thread Herbert Xu
and the auxiliary data mechanism of recvmsg. Looks good to me. Acked-by: Herbert Xu [EMAIL PROTECTED] Dave, perhaps we should queue this for 2.6.17? Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key

Re: Fw: [Bugme-new] [Bug 6197] New: unregister_netdevice: waiting for ppp9 to become free. Usage count = 658

2006-03-09 Thread Herbert Xu
as well. If it's possible for things to get added after the flush then isn't it pointless to flush there? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert

Re: [2.6 patch] net/decnet/dn_route.c: fix inconsequent NULL checking

2006-03-10 Thread Herbert Xu
. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message

[TCP]: Fix zero port problem in IPv6

2006-03-13 Thread Herbert Xu
, we need to make sure that we set the num/port fields so that it shows us with a non-zero port value in proc/netlink and on the wire. This code and comment is copied over from the IPv4 stack as is. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org

Re: [PATCH] [NET] : get rid of 'struct sec_path *' in skbuff if ! CONFIG_XFRM

2006-03-13 Thread Herbert Xu
. Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message

Re: [2.6 patch] hostap_{pci,plx}.c: fix memory leaks

2006-03-13 Thread Herbert Xu
)); - if (pci_enable_device(pdev)) - return -EIO; - You've just turned it into a leak of a different kind. Why not jump to the error exit instead? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP

Re: [PATCH] [NET] : get rid of 'struct sec_path *' in skbuff if ! CONFIG_XFRM

2006-03-14 Thread Herbert Xu
, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

Re: Fw: IPv6: to take care of permanent routes

2006-03-16 Thread Herbert Xu
be checking the RTF_EXPIRES flag instead. You're most welcome to fix this bug. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from

Re: [PATCH] ip6_tunnel: fix a soft lockup when there is no active tunnel for an encapsulated packet

2006-03-23 Thread Herbert Xu
); + return -1; } When xfrm6_tunnel is in use you've just made it use a freed skb. Also IPv4 has the same problem so we shold fix them both. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP

Re: [PATCH] [IPV4] PMTU issues due to TOS field manipulation

2006-03-24 Thread Herbert Xu
other route entries that differ only by TOS. Signed-off-by: Ilia Sotnikov [EMAIL PROTECTED] Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http

Re: [PATCH] ip6_tunnel: fix a soft lockup when there is no active tunnel for an encapsulated packet

2006-03-24 Thread Herbert Xu
this is a bit ugly but will do for now. Could you please do it for the ICMP packet and IPv4 as well? Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

Re: [PATCH] ip6_tunnel: fix a soft lockup when there is no active tunnel for an encapsulated packet

2006-03-24 Thread Herbert Xu
) function. Let me have a go at that. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line

[NET]: Take RTNL when unregistering notifier

2006-03-24 Thread Herbert Xu
to take the RTNL for all calls to netdev notifiers, so taking the RTNL should be sufficient to protect it. The registration path in dev.c already takes the RTNL so it's OK. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI

[SCTP]: Fix up sctp_rcv return value

2006-03-24 Thread Herbert Xu
there is to increment the IPSTATS_MIB_INDELIVERS counter which IMHO is the right thing to do. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http

Re: [NET]: Take RTNL when unregistering notifier

2006-03-25 Thread Herbert Xu
Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More

[IPSEC]: Fix tunnel error handling in ipcomp6

2006-03-26 Thread Herbert Xu
as if we succeeded! This patch fixes them both. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt diff

[INET]: Introduce tunnel4/tunnel6

2006-03-27 Thread Herbert Xu
-by: Herbert Xu [EMAIL PROTECTED] BTW the ICMP sending in case of a non-existant tunnel does not fit here and will be fixed properly in a subsequent patch. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert

Re: [INET]: Introduce tunnel4/tunnel6

2006-03-27 Thread Herbert Xu
it will be moved out in a subsequent patch. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line

Re: [INET]: Introduce tunnel4/tunnel6

2006-03-28 Thread Herbert Xu
for you. OK, will do. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line unsubscribe netdev

Re: [e1000 debug] KERNEL: assertion (!sk_forward_alloc) failed...

2006-03-30 Thread Herbert Xu
if a TSO packet wins a race against a non-TSO packet with last_tx_tso == 0 then we'll have memory corruption. Everyone, please try this patch and let us know whether the problem goes away. Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page

Re: [e1000 debug] KERNEL: assertion (!sk_forward_alloc) failed...

2006-03-30 Thread Herbert Xu
applied) patch detected! Assume -R? [n] Herbert, is that patch already included in 2.6.16.1? Not really. It's just patch being silly (or too smart :) Here it is again rediffed against 2.6.16. -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page

Re: [e1000 debug] KERNEL: assertion (!sk_forward_alloc) failed...

2006-03-31 Thread Herbert Xu
indicate that this is an SMP race. If there are anyone else out there who do have this problem and are either using something other than e1000, have disabled TSO, or are UP, please speak up now. Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home

<    1   2   3   4   5   6   7   8   9   10   >