[ovs-dev] [PATCH v10 0/5] userspace: Support for layer 3 encapsulated packets

[Simon Horman]

On Wed, May 18, 2016 at 05:28:52PM -0700, Ben Pfaff wrote:
> On Wed, May 04, 2016 at 04:34:20PM +0900, Simon Horman wrote:
> > This series implements support for layer 3 encapsulated packets.  At the
> > core of this change is removing the assumption that all packets/flows have
> > Ethernet header. Support for layer 3 packets in GRE tunnels is also added
> > by this patchset.
> > 
> > The implementation automatically adds appropriate pop_eth and push_eth
> > actions to datapath flows. This may change in the future when OpenFlow
> > support for these actions will be added, since discussions on EXT-112 in
> > the ONF showed preference for explicitly requiring the pop_eth and push_eth
> > actions in OpenFlow rules.
> > 
> > This series is based on work by Lorand Jakab, Thomas Morin and others.
> > 
> > Unlike Lorand's work this series does not update the kernel datapath
> > nor the LISP vport implementation in the version of the kernel datapath
> > in the OVS tree. Rather it focuses only on userspace including
> > the user-space datapath, and provides layer 3 GRE support. It should,
> > however, be in keeping with his goal of providing a clean path for
> > porting the LISP datapath support to the mainline kernel for inclusion
> > there.
> 
> What part of this series is the next logical part to review?

Hi Ben,

there has been some review of the accompanying kernel patch set and
at this point the main changes there which affect user-space are:

* Removing the eth_type field from push_eth actions
* Discussion of how to correctly handle VLANs in conjunction
  with layer3 vports/packets: not concluded

>From my point of view aspects of this patch set which aren't affected
by the above could be reviewed as-is. Which is to say, everything
except the push/pop eth implementation (including related paths in the
user-space datapath).