Re: [ovs-dev] [LNG] Re: datapath: remove HAVE_MAC_RAW

2013-09-02 Thread Viresh Kumar
On 31 August 2013 05:45, Jesse Gross wrote: > Out of curiosity, what is the actual symbol that we're finding in the RT > kernel? Its not about RT kernel really.. Following patch did this: b0e380b [SK_BUFF]: unions of just one member don't get anything done, kill them diff --git a/include/linu

[ovs-dev] [PATCH net-next 0/5] tunnels: harmonize skb scrubbing during encapsulation/decapsulation

2013-09-02 Thread Nicolas Dichtel
We talk recently about harmonizing tunnels so they behave consistently wrt. SKB orphaning, cleaning netfilter state, etc. The goal of this serie is to achieve this. Note that I test only ipip, sit and ip6_tunnels modules. v2: add patch 2/5 rebase on head remove 'RFC' prefix drivers/net/

[ovs-dev] [PATCH net-next v2 5/5] tunnels: harmonize cleanup done on skb on rx path

2013-09-02 Thread Nicolas Dichtel
The goal of this patch is to harmonize cleanup done on a skbuff on rx path. Before this patch, behaviors were different depending of the tunnel type. Signed-off-by: Nicolas Dichtel --- include/net/dst.h | 12 +++- net/ipv4/ip_tunnel.c | 3 +-- net/ipv4/ipmr.c | 3 +-- net/ip

[ovs-dev] [PATCH net-next v2 1/5] iptunnels: remove net arg from iptunnel_xmit()

2013-09-02 Thread Nicolas Dichtel
This argument is not used, let's remove it. Signed-off-by: Nicolas Dichtel --- drivers/net/vxlan.c | 3 +-- include/net/ip_tunnels.h| 3 +-- net/ipv4/ip_tunnel.c| 3 +-- net/ipv4/ip_tunnel_core.c | 3 +-- net/ipv6/sit.c | 4 ++-- net/openvswitch/vport-gre.c | 2

[ovs-dev] [PATCH net-next v2 2/5] vxlan: remove net arg from vxlan[6]_xmit_skb()

2013-09-02 Thread Nicolas Dichtel
This argument is not used, let's remove it. Signed-off-by: Nicolas Dichtel --- drivers/net/vxlan.c | 8 include/net/vxlan.h | 2 +- net/openvswitch/vport-vxlan.c | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/net/vxlan.c b/drivers/net/

[ovs-dev] [PATCH net-next v2 3/5] skb: allow skb_scrub_packet() to be used by tunnels

2013-09-02 Thread Nicolas Dichtel
This function was only used when a packet was sent to another netns. Now, it can also be used after tunnel encapsulation or decapsulation. Only skb_orphan() should not be done when a packet is not crossing netns. Signed-off-by: Nicolas Dichtel --- include/linux/skbuff.h | 2 +- net/core/dev.c

[ovs-dev] [PATCH net-next v2 4/5] tunnels: harmonize cleanup done on skb on xmit path

2013-09-02 Thread Nicolas Dichtel
The goal of this patch is to harmonize cleanup done on a skbuff on xmit path. Before this patch, behaviors were different depending of the tunnel type. Signed-off-by: Nicolas Dichtel --- drivers/net/vxlan.c | 6 -- include/net/ip6_tunnel.h| 1 - include/net/ip_tunnels.h| 2 +-

[ovs-dev] groups implementation committed--thank you!

2013-09-02 Thread Ben Pfaff
I applied the groups implementation originally contributed Centec to master now. Simon and Jarno also contributed to this implementation. Thanks a lot to all of you! Casey, the new implementation undoubtedly has flaws. If you want to contribute improvements, that perhaps move closer to the impl

[ovs-dev] 答复: groups implementation committed--thank you!

2013-09-02 Thread Neil Zhu
Our whole team are very glad to hear that! We expect to contribute much more to community in the future. Thanks, Neil -邮件原件- 发件人: Ben Pfaff [mailto:b...@nicira.com] 发送时间: 2013年9月3日 0:35 收件人: dev@openvswitch.org 抄送: Neil Zhu; Simon Horman; Jarno Rajahalme; Casey Barker 主题: groups impleme

Re: [ovs-dev] [PATCH] ofproto-dpif: Do not dpif_port_del() patch ports in port_del().

2013-09-02 Thread YAMAMOTO Takashi
this patch looks fine and works for me. thanks. YAMAMOTO Takashi > Patch ports don't have datapath ports so it doesn't make sense to try to > call dpif_port_del() on them. If we do try, it will fail, which makes the > caller think that the port wasn't really deleted, which in turn keeps > ofpro