On 01/29/15 at 07:45pm, Pravin Shelar wrote:
> > +#ifdef HAVE_VXLAN_METADATA
> > static inline int rpl_vxlan_xmit_skb(struct vxlan_sock *vs,
> > struct rtable *rt, struct sk_buff *skb,
> > __be32 src, __be32 dst, __u8 tos, __u8 ttl, __be16 df,
> > - __be16 src_port, __be16 dst_port, __be32 vni)
> > + __be16 src_port, __be16 dst_port,
> > + struct vxlan_metadata *md)
> > {
> Can you keep vxlan_xmit_skb() in-sync with upstream kernel. I see
> couple of parameters are missing. If there is any difference in
> function prototype, we can not make use of vxlan_xmit_skb() from new
> kernel.
Sure. The vxflags was added by Tom Herbert after I submitted these.
I'll account for it an add xnet to the internal wrapper.
> > if (skb_is_gso(skb) && skb_is_encapsulated(skb)) {
> > kfree_skb(skb);
> > return -ENOSYS;
> > }
> >
> > -#ifdef HAVE_VXLAN_XMIT_SKB_XNET_ARG
>
> HAVE_VXLAN_XMIT_SKB_XNET_ARG can be removed from acinclude.m4.
Fixed
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev