[ovs-dev] [PATCH 15/16] tunneling: Add userspace support for new GRE implementation.

Jesse Gross jesse at nicira.com
Fri Apr 16 13:23:35 UTC 2010


On Thu, Apr 15, 2010 at 1:57 PM, Ben Pfaff <blp at nicira.com> wrote:

> On Tue, Apr 13, 2010 at 10:41:17AM -0400, Jesse Gross wrote:
> > Add a netdev that supports the new datapath GRE implementation.
> > It currently coexists with the old implementation so it is named
> > "grenew".
>
> In netdev_gre_create(), I think that the cast is unneeded on "(struct
> gre_port_config *)&port_config".
>
> In netdev_gre_reconfigure(), I would write &port_config instead of
> "(struct gre_port_config *)ovm.config".
>

Yeah, I guess I forgot to update everything when I changed the config
parsing.


>
> In netdev_gre_destroy(), I do not see why the xstrdup() is necessary.
>

It's not really.  I originally did it to make the compiler shut up about
const-ness but I just threw in a cast instead.


>
> In netdev_gre_update_flags(), I was surprised about NETDEV_LOOPBACK.
> The only other reference I see to it is in netdev.h.  I guess that we
> should delete it.  It is not, at any rate, a configuration parameter
> that can be turned on and off.
>

Yeah, obviously I wasn't planning on actually supporting it.  I see that
everyone else ignores it though, so I just deleted that check.


>
> "Review fatigue" set in when I arrived at the netdev_gre_poll_*()
> functions.  I guess that these are correct.
>

They're not too exciting.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20100416/d9058b9a/attachment-0003.html>


More information about the dev mailing list