On Wed, Sep 23, 2015 at 2:13 PM, Jesse Gross <je...@nicira.com> wrote: > 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>
Thanks. I pushed patch to master. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
