[ovs-dev] [PATCH 15/16] tunneling: Add userspace support for new GRE implementation.
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
> 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...
More information about the dev