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
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to