On Thu, Feb 02, 2017 at 02:59:42PM -0800, Joe Stringer wrote:
> On 18 January 2017 at 11:45, Eric Garver <e...@erig.me> wrote:
> > Creates VXLAN devices using rtnetlink and tunnel metadata.
> >
> > Co-Authored-by: Thadeu Lima de Souza Cascardo <casca...@redhat.com>
> > Co-Authored-by: Eric Garver <e...@erig.me>
> > Signed-off-by: Eric Garver <e...@erig.me>
> > ---
> > lib/dpif-netlink.c | 194
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++-
> > 1 file changed, 193 insertions(+), 1 deletion(-)
>
> I think that the vast majority of this code is linux-specific and
> should not exist in dpif-netlink.c.
>
> Perhaps we should add a new lib/netdev-lwt.[ch], where the .h file has
> the #ifdef __linux__ logic to either declare or define the functions,
> then lib/netdev-lwt.c has the implementations of these functions. The
> .h would always be added to the build in lib/automake.mk and the .c
> file would only be added in the #if LINUX section.
That's a nice cleanup. Thanks for the suggestion.
> One might even argue that it'd be tidier if the
> dpif_netlink_port_{create,destroy} functions were moved here (and
> renamed to something more apt).
I'll look into this as well.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
