Re: ipcomp regression in 2.6.24

2008-01-29 Thread Herbert Xu
. This patch moves the reading up so that we get the correct nexthdr value. 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: [PATCH][INET_DIAG]: Fix inet_diag_lock_handler error path

2008-01-28 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

[INET]: Prevent out-of-sync truesize on ip_fragment slow path

2008-01-28 Thread Herbert Xu
the update of skb-truesize to prevent this. 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

[0/3] [IPSEC]: Add async/aead support

2008-01-28 Thread Herbert Xu
Hi Dave: I've rebased the IPsec AEAD/async patches for the current net-2.6.25 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

[PATCH 2/3] [IPSEC]: Allow async algorithms

2008-01-28 Thread Herbert Xu
[IPSEC]: Allow async algorithms Now that ESP uses authenc we can turn on the support for async algorithms in IPsec. Signed-off-by: Herbert Xu [EMAIL PROTECTED] --- net/xfrm/xfrm_algo.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/net/xfrm/xfrm_algo.c b/net/xfrm

[PATCH 3/3] [IPSEC]: Add support for combined mode algorithms

2008-01-28 Thread Herbert Xu
is identified by its name and the ICV length. For the purposes of matching algorithms in xfrm_tmpl structures, combined mode algorithms are occupy the same name space as encryption algorithms. This is in line with how they are negotiated using IKE. Signed-off-by: Herbert Xu [EMAIL PROTECTED

[PATCH 1/3] [IPSEC]: Use crypto_aead and authenc in ESP

2008-01-28 Thread Herbert Xu
[IPSEC]: Use crypto_aead and authenc in ESP This patch converts ESP to use the crypto_aead interface and in particular the authenc algorithm. This lays the foundations for future support of combined mode algorithms. Signed-off-by: Herbert Xu [EMAIL PROTECTED] --- include/net/esp.h | 52

Re: [PATCH 2/3] virtio: Net header needs gso_hdr_len

2008-01-22 Thread Herbert Xu
this is more flexible than the Xen approach where this is essentially hard-coded to 64 bytes. 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: [PATCH 2/3] virtio: Net header needs gso_hdr_len

2008-01-22 Thread Herbert Xu
? Sounds fine to me. 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

[INET]: Fix truesize setting in ip_append_data

2008-01-22 Thread Herbert Xu
the same page gets away with no charge at all. This bug was exposed by the UDP accounting patch. This patch also fixes a similar bug in ip_append_page spotted by you :) Signed-off-by: Herbert Xu [EMAIL PROTECTED] Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL

Re: [INET]: Fix truesize setting in ip_append_data

2008-01-22 Thread Herbert Xu
and ip_append_page. 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/ipv4/ip_output.c

Re: [PATCH net-2.6.25] [XFRM] Remove unneeded export of xfrm_output_resume() and remove its declaration from net/xfrm.h.

2008-01-19 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

Re: [RFC][PATCH] Fixing SA/SP dumps on netlink/af_key

2008-01-17 Thread Herbert Xu
rip pfkey away; adding a layer to work with both pfkey and netlink would be doable, but just a lot of work. Sure racoon uses pfkey but the question is does it use pfkey dumping? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: [RFC][PATCH] Fixing SA/SP dumps on netlink/af_key

2008-01-17 Thread Herbert Xu
be discontinued in future versions of PF_KEY. Key management applications MUST NOT depend on this message for basic operation. 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][PATCH] Fixing SA/SP dumps on netlink/af_key

2008-01-17 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

Re: [PATCH] ICMP: ICMP_MIB_OUTMSGS increment duplicated

2008-01-16 Thread Herbert Xu
packets too. But we don't do that for any other protocol, e.g., raw UDP packets don't get counted. So was the inclusion of raw ICMP packets intentional? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au

Re: [RFC][PATCH] Fixing SA/SP dumps on netlink/af_key

2008-01-16 Thread Herbert Xu
is portable. So we should discourage people from using af_key wherever possible. 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

Re: [PATCH] ICMP: ICMP_MIB_OUTMSGS increment duplicated

2008-01-16 Thread Herbert Xu
the common use, at least. Fair enough. How about moving this code back into icmp.c and just add a new count call in raw.c? The push pending function is used on the UDP fast path so the leaner it is the better. Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL

Re: [RFC][PATCH] Fixing SA/SP dumps on netlink/af_key

2008-01-16 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 2/3] virtio: Net header needs gso_hdr_len

2008-01-15 Thread Herbert Xu
-gso_hdr_len); Why do we need this? When receiving GSO packets from an untrusted source the network stack will fill in the transport header offset after verifying that the headers are sane. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page

Re: questions on NAPI processing latency and dropped network packets

2008-01-15 Thread Herbert Xu
be borne by the company and not the community. 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: Netperf TCP_RR(loopback) 10% regression in 2.6.24-rc6, comparing with 2.6.22

2008-01-14 Thread Herbert Xu
SLAB/SLUB while testing this. 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: [PATCH v2] Documentation: add a guideline for hard_start_xmit method

2008-01-13 Thread Herbert Xu
Matti Linnanvuori [EMAIL PROTECTED] wrote: -3) Do not forget that once you return 0 from your hard_start_xmit +3) A hard_start_xmit method must not modify the shared parts of the SKB. Only if it's cloned. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI

Re: 2.6.24-rc6-mm1 - oddness with IPv4/v6 mapped sockets hanging...

2008-01-13 Thread Herbert Xu
[EMAIL PROTECTED] wrote: Any ideas? Please provide a packet dump on both sides (or at least the sender side). 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

Re: What is XFRM_POLICY_LOCALOK for?

2008-01-12 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: [RFC PATCH 8/8] [PKTGEN]: uninline getCurUs

2008-01-12 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 majordomo

Re: [RFC PATCH 8/8] [PKTGEN]: uninline getCurUs

2008-01-12 Thread Herbert Xu
that as well but no, From is correctly decoded by my ISO-8859-1'ish MUA, aha, seems that I still have something to do to deal with the Signed-off line. Yeah I did see that. Otherwise I wouldn't have noticed :) Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL

Re: [PATCH 0/3] bonding: 3 fixes for 2.6.24

2008-01-12 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

Re: [NET] ROUTE: fix rcu_dereference() uses in /proc/net/rt_cache

2008-01-11 Thread Herbert Xu
at this rt_cache_get_next() and this patch only, it's looks like the third candidate after seq-private and rtable... Perhaps we could introduce a sparse attribute for it? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au

Re: [NET] ROUTE: fix rcu_dereference() uses in /proc/net/rt_cache

2008-01-11 Thread Herbert Xu
possible... Since rcu_derference(r) is always the same as r this patch cannot change the value returned. 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: handing cloned frames to netif_rx()?

2008-01-11 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: handing cloned frames to netif_rx()?

2008-01-11 Thread Herbert Xu
because they can always run after another handler (e.g., AF_PACKET) which may have cloned the packet. 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: rp_filter and ip rule break ipsec policy

2008-01-11 Thread Herbert Xu
rp_filter can eat your packet, and only after that do we perform the output IPsec lookup. 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: questions on NAPI processing latency and dropped network packets

2008-01-11 Thread Herbert Xu
. After all we haven't made that commitment to you or your customers. 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 0/3] bonding: 3 fixes for 2.6.24

2008-01-10 Thread Herbert Xu
not be taken reliably in soft-irq context. Michael Chan fixed this here: Sure, but where do you call that function while holding the bond lock? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key

Re: [PATCH 0/3] bonding: 3 fixes for 2.6.24

2008-01-10 Thread Herbert Xu
sleep. It wasn't about the lock. Exactly, I was just about to post the same. In other words, changing read_lock on bond-lock to read_lock_bh doesn't affect this one bit. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: [NET] ROUTE: fix rcu_dereference() uses in /proc/net/rt_cache

2008-01-10 Thread Herbert Xu
the first version of this patch is more reliable. (Or alternatively additional check is needed before return.) No, while the value of r-u.dst.rt_next can change between two readings, the value of r cannot. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL

Re: [PATCH 0/3] bonding: 3 fixes for 2.6.24

2008-01-10 Thread Herbert Xu
to the rest of the bonding driver it's fine by me. BTW you only need read_lock since set_multicast_list always gets called with BH off. 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] ROUTE: fix rcu_dereference() uses in /proc/net/rt_cache

2008-01-09 Thread Herbert Xu
the memory at r was initialised. How many times you read the pointer value before the barrier is irrelevant to the effectiveness of the barrier preceding the dereference. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

[ATM]: Check IP header validity in mpc_send_packet

2008-01-09 Thread Herbert Xu
to react gracefully. This patch ensures that the skb has enough data for an IP header and that the header length field is valid. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: [PATCH 0/3] bonding: 3 fixes for 2.6.24

2008-01-09 Thread Herbert Xu
context so that you can take the RTNL. Once you have taken the RTNL you can disable BH all you want and I don't care one bit. In any case, fixing a known dead-lock is important. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http

Re: [PATCH 0/3] bonding: 3 fixes for 2.6.24

2008-01-09 Thread Herbert Xu
. It does not since the point of the move is to be able to take the RTNL, which you can still do as long as you do it before you disable BH. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert/ PGP Key

Re: AF_UNIX MSG_PEEK bug?

2008-01-09 Thread Herbert Xu
if you had attached a patch :) 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

Re: [IPV4] ROUTE: ip_rt_dump() is unecessary slow

2008-01-08 Thread Herbert Xu
-bucket].chain); } return r; Slight optimisation: please move both barriers onto the return statement, i.e., return rcu_dereference(r); Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au

Re: [PATCH 0/3] bonding: 3 fixes for 2.6.24

2008-01-08 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: [XFRM]: xfrm_algo_clone() allocates too much memory

2008-01-08 Thread Herbert Xu
(struct xfrm_algo *alg) Could you please add an xfrm prefix to this? 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

Re: [SOCK]: Adds a rcu_dereference() in sk_filter

2008-01-08 Thread Herbert Xu
. Signed-off-by: Eric Dumazet [EMAIL PROTECTED] Acked-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

Re: [PATCH 3/4] [XFRM]: Kill some bloat

2008-01-07 Thread Herbert Xu
goes from 1 byte to 4 when you go over 128 bytes. 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: [IPV4] ROUTE: Avoid sparse warnings

2008-01-07 Thread Herbert Xu
of them to hold locks unconditionally. 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: [IPV4] ROUTE: Avoid sparse warnings

2008-01-07 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: [IPV4] ROUTE: Avoid sparse warnings

2008-01-07 Thread Herbert Xu
to avoid all uses of conditional locking. 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][XFRM] Statistics: Add outbound-dropping error.

2008-01-07 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] xfrm_policy_destroy: rename and relative fixes

2008-01-07 Thread Herbert Xu
instances that call xfrm_policy_alloc but not using xfrm_policy_destroy to destroy the resource, fix them. Cc: David Miller [EMAIL PROTECTED] Cc: Herbert Xu [EMAIL PROTECTED] Signed-off-by: WANG Cong [EMAIL PROTECTED] This patch looks OK to me. Thanks, -- Visit Openswan at http

Re: [PATCH 3/4] [XFRM]: Kill some bloat

2008-01-06 Thread Herbert Xu
them all. Here's the other end of the list: Right. If the inline helps the compiler perform code size optimisations that it otherwise couldn't then it might be worthwhile. So when in doubt, check the assembly :) Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI

Re: sparc oops in ip_fast_csum

2008-01-05 Thread Herbert Xu
that which is quite reasonable. 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: [PATCH 3/4] [XFRM]: Kill some bloat

2008-01-05 Thread Herbert Xu
Ilpo J??rvinen [EMAIL PROTECTED] wrote: Signed-off-by: Ilpo J??rvinen [EMAIL PROTECTED] Acked-by: Herbert Xu [EMAIL PROTECTED] #ifdef CONFIG_AUDITSYSCALL -static inline void xfrm_audit_helper_sainfo(struct xfrm_state *x, - struct audit_buffer

Re: sparc oops in ip_fast_csum

2008-01-05 Thread Herbert Xu
) { +iphlen = iph-ihl * 4; +if (iphlen = sizeof(*iph) iphlen = length) { Humm, this could use ip_hdrlen(skb) :-) That would not necessarily be an improvement since we'd have to reload the values from skb. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI

Re: sparc oops in ip_fast_csum

2008-01-05 Thread Herbert Xu
complain too much because I wasn't able to spend any time in 2007 working on it. But it's definitely high on my todo list for 2008. Oh and Al, your effort in going through all the ip_fast_csum callers is much appreciated! Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu

Re: [IPSEC]: Return EOVERFLOW when output sequence number overflows

2008-01-02 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] Force UNIX domain sockets to be built in

2008-01-02 Thread Herbert Xu
. This I think is quite pointless. 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.24-rc6-mm1

2008-01-02 Thread Herbert Xu
be git-nfsd and git-net. 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

Re: 2.6.24-rc6-mm1

2008-01-01 Thread Herbert Xu
this, we really need your help to track this down. Since bisecting the mm tree is not practical, you could start by checking whether the bug is in mm only or whether it affects rc6 too. Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page

[IPSEC]: Kill duplicate xfrm_policy_flush prototype

2008-01-01 Thread Herbert Xu
Hi Dave: [IPSEC]: Kill duplicate xfrm_policy_flush prototype For five years we had two xfrm_policy_flush prototypes and every time that function's signature changed people have been diligently updating both of them without noticing :) Signed-off-by: Herbert Xu [EMAIL PROTECTED] diff --git

[IPSEC]: Return EOVERFLOW when output sequence number overflows

2008-01-01 Thread Herbert Xu
Hi Dave: [IPSEC]: Return EOVERFLOW when output sequence number overflows Previously we made it an error on the output path if the sequence number overflowed. However we did not set the err variable accordingly. This patch sets err to -EOVERFLOW in that case. Signed-off-by: Herbert Xu [EMAIL

Re: ipv4_devconf.arp_accept mystery

2007-12-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 net-2.6][NEIGH] Updating affected neighbours when about MAC address change

2007-12-23 Thread Herbert Xu
().) I had IPv4 in mind, there is no reason that it will no be also in IPv6. Hmm, why can't you do this from user-space? 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: TSO trimming question

2007-12-21 Thread Herbert Xu
a branch. 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: [SOCK] Avoid integer divides where not necessary in include/net/sock.h

2007-12-21 Thread Herbert Xu
for the last 18 months. 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: [XFRM]: Fix outbound statistics.

2007-12-21 Thread Herbert Xu
: 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://vger.kernel.org

Re: TSO trimming question

2007-12-20 Thread Herbert Xu
have to divide in the unlikely case of us being limited by the receiver window. In that case speed is probably not of the essence 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

Re: [PATCH] [XFRM] IPv6: Fix dst/routing check at transformation.

2007-12-20 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: Inline local_bh_disable when TRACE_IRQFLAGS

2007-12-19 Thread Herbert Xu
preemptible softirq work. However I'm mostly thinking about the existing callers of local_bh_disable in the networking stack. 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

Re: missing dir in and dir fwd policy on 2.6.23

2007-12-19 Thread Herbert Xu
have already fixed 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/pubkey.txt -- To unsubscribe from this list: send the line unsubscribe netdev

Re: TSO trimming question

2007-12-19 Thread Herbert Xu
avoid the divide for the case when we're not bound by the receiver window. 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

[IPSEC]: Avoid undefined shift operation when testing algorithm ID

2007-12-19 Thread Herbert Xu
-space. Besides, this whole interface is obsolete anyway in favour of the xfrm_user interface which doesn't use this bit mask in templates (well not within the kernel anyway). Signed-off-by: Herbert Xu [EMAIL PROTECTED] diff --git a/net/key/af_key.c b/net/key/af_key.c index 878039b..26d5e63 100644

Re: [PATCH 3/3] [UDP6]: Counter increment on BH mode

2007-12-18 Thread Herbert Xu
new percpu code will greatly simplify this and we won't need to disable preemption or do the fancy softirq/user splitting that we have right now. I eagerly await its acceptance into the Linux kernel. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL

Re: ip neigh show not showing arp cache entries?

2007-12-18 Thread Herbert Xu
looks like to determine the problem. BTW my emails to you are bouncing so you might want to fix that. 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

Inline local_bh_disable when TRACE_IRQFLAGS

2007-12-18 Thread Herbert Xu
: 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://vger.kernel.org

[IPSEC]: Do xfrm_state_check_space before encapsulation

2007-12-18 Thread Herbert Xu
goto should the encapsulation output fail. Thanks to Kazunori MIYAZAWA for finding this bug. Signed-off-by: Herbert Xu [EMAIL PROTECTED] diff --git a/net/xfrm/xfrm_output.c b/net/xfrm/xfrm_output.c index 3c277a4..26fa0cb 100644 --- a/net/xfrm/xfrm_output.c +++ b/net/xfrm/xfrm_output.c @@ -33,16

[IPSEC]: Rename tunnel-mode functions to avoid collisions with tunnels

2007-12-18 Thread Herbert Xu
which lives in the files xfrmX_mode_tunnel.c. This patch renames functions from the latter to use the xfrmX_mode_tunnel prefix to avoid name-space conflicts. Signed-off-by: Herbert Xu [EMAIL PROTECTED] diff --git a/net/ipv4/xfrm4_mode_tunnel.c b/net/ipv4/xfrm4_mode_tunnel.c index aa335db..8dee617

Re: missing dir in and dir fwd policy on 2.6.23

2007-12-18 Thread Herbert Xu
the management of policies is entirely up to the user-space Key Manager. 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: ip neigh show not showing arp cache entries?

2007-12-17 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: [PACKET]: Fix /proc/net/packet crash due to bogus private pointer

2007-12-16 Thread Herbert Xu
On Sat, Dec 15, 2007 at 11:56:04PM -0800, Andrew Morton wrote: On Sun, 16 Dec 2007 01:37:01 -0500 Miles Lane [EMAIL PROTECTED] wrote: On Sun, Dec 16, 2007 at 11:07:07AM +0800, Herbert Xu wrote: So I posted this patch after 19:00 PST on 15 Dec. Dec 15 13:44:39 syntropy kernel: #0: (p

Re: oops with 2.6.23.1, marvel, software raid, reiserfs and samba

2007-12-16 Thread Herbert Xu
on exactly the same line in tcp_recvmsg. 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: oops with 2.6.23.1, marvel, software raid, reiserfs and samba

2007-12-16 Thread Herbert Xu
On Sun, Dec 16, 2007 at 07:56:56PM +0800, Herbert Xu wrote: What's spooky is that I just did a google and we've had reports since 1998 all crashing on exactly the same line in tcp_recvmsg. However, there's been no reports at all since 2000 apart from this one so the earlier ones are probably

Re: [patch 2/4] net: use mutex_is_locked() for ASSERT_RTNL()

2007-12-16 Thread Herbert Xu
this change without a might_sleep. 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

Re: [patch 2/4] net: use mutex_is_locked() for ASSERT_RTNL()

2007-12-16 Thread Herbert Xu
for GFP is/should_be there already? On some paths this may be buried a conditional. Also if you replace it with a mutex_is_locked without the may_sleep it won't catch the case that started all this, namely the promiscuous path. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert

Re: [patch 2/4] net: use mutex_is_locked() for ASSERT_RTNL()

2007-12-15 Thread Herbert Xu
him an apology. 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: [PATCH 3/3] [UDP6]: Counter increment on BH mode

2007-12-15 Thread Herbert Xu
Ob Tue, Dec 04, 2007 at 12:17:23AM +1100, Herbert Xu wrote: Never mind, we already have that in local_t and as Alexey correctly points out, USER is still going to be the expensive variant with the preempt_disable (well until BH gets threaded). So how about this patch? I didn't hear any

Re: [PATCH 2/4] datagram: mem_scheudle functions

2007-12-15 Thread Herbert Xu
(). ip_append_data can't possibly work for stream protocols. So which stream protocol is calling 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 2/4] [CORE]: datagram: mem_scheudle functions

2007-12-15 Thread Herbert Xu
, +struct sock* nsk)) Just make a new function for this rather than playing with function pointers. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page: http://gondor.apana.org.au/~herbert

Re: [Bugme-new] [Bug 9543] New: RTNL: assertion failed at net/ipv6/addrconf.c (2164)/RTNL: assertion failed at net/ipv4/devinet.c (1055)

2007-12-15 Thread Herbert Xu
output again. 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

[SNMP]: Fix SNMP counters with PREEMPT

2007-12-15 Thread Herbert Xu
SNMP counters with PREEMPT The SNMP macros use raw_smp_processor_id() in process context which is illegal because the process may be preempted and then migrated to another CPU. This patch makes it use get_cpu/put_cpu to disable preemption. Signed-off-by: Herbert Xu [EMAIL PROTECTED] Thanks

Re: [PATCH 3/3] [UDP6]: Counter increment on BH mode

2007-12-15 Thread Herbert Xu
%rax would be the offset with %gs as the base. This would obviate the need for the CPU ID and therefore avoid disabling preemption. Hmm, wasn't Christoph working on something like that? Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmVHI~} [EMAIL PROTECTED] Home Page

Re: 2.6.24-rc5-mm1: cat /proc/net/packet - oops

2007-12-15 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

Re: 2.6.24-rc5-mm1 -- inconsistent {in-softirq-W} - {softirq-on-R} usage.

2007-12-15 Thread Herbert Xu
with the namespace work. I'll let you know when I have digged further. 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: [Bugme-new] [Bug 9543] New: RTNL: assertion failed at net/ipv6/addrconf.c (2164)/RTNL: assertion failed at net/ipv4/devinet.c (1055)

2007-12-15 Thread Herbert Xu
: 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

[PACKET]: Fix /proc/net/packet crash due to bogus private pointer

2007-12-15 Thread Herbert Xu
On Sun, Dec 16, 2007 at 11:07:07AM +0800, Herbert Xu wrote: I suspect namespace borkage. But just because you pin-pointed my patch I'll try to track it down :) Surprise surprise. The namespace seq patch missed two spots in AF_PACKET. [PACKET]: Fix /proc/net/packet crash due to bogus private

Re: [patch 2/4] net: use mutex_is_locked() for ASSERT_RTNL()

2007-12-15 Thread Herbert Xu
ASSERT_RTNL users and then add the might_sleep check. 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: ip xfrm bug

2007-12-15 Thread Herbert Xu
is up-index % 8 = 3 This is true iff it's a socket policy. 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

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