On Tue, Sep 22, 2015 at 7:40 PM, Pravin Shelar <pshe...@nicira.com> wrote: > On Tue, Sep 22, 2015 at 7:21 PM, Jesse Gross <je...@nicira.com> wrote: >> On Fri, Sep 18, 2015 at 3:23 PM, Pravin B Shelar <pshe...@nicira.com> wrote: >>> diff --git a/datapath/linux/compat/include/linux/net.h >>> b/datapath/linux/compat/include/linux/net.h >>> index 9c94745..2a6903d 100644 >>> --- a/datapath/linux/compat/include/linux/net.h >>> +++ b/datapath/linux/compat/include/linux/net.h >> [...] >>> +#ifndef HAVE_SOCK_CREATE_KERN_NET >>> +int ovs_sock_create_kern(struct net *net, int family, int type, int >>> protocol, struct socket **res); >>> +void ovs_sock_release(struct socket *sock); >>> +#define sock_create_kern ovs_sock_create_kern >>> +#define sock_release ovs_sock_release >>> +#endif >> >> Is there a reason why this is ovs_sock_create_kern() instead of using >> rpl_ like in other places? And do we even need sock_release() all now? > > sock_release macro required for backport. > I used ovs_ prefix since there is already sock_release() defined on > older kernel which is pretty different from the one backported.
OK, I understand now. Acked-by: Jesse Gross <je...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
