[ovs-dev] [PATCH v3 1/5] logical-fields: New header for logical field assignments.

Ben Pfaff blp at nicira.com
Mon Oct 19 15:48:16 UTC 2015


On Mon, Oct 19, 2015 at 12:17:04AM -0700, Justin Pettit wrote:
> 
> > On Oct 18, 2015, at 12:52 PM, Ben Pfaff <blp at nicira.com> wrote:
> > 
> > The original concept for "expr" and "actions" was that they should not need
> > to know anything about the mapping between physical and logical fields,
> > that instead everything should be provided via the symbol table.  In
> > practice this has proven difficult because a couple of actions need to know
> > about logical fields.  For now, it seems reasonable to put the logical
> > field mapping into a header of its own.  Later, maybe we'll figure out
> > whether there's value in a less leaky abstraction.
> > 
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> 
> Thanks for doing this.  I looked at the changes since the last
> version, and they look good to me.  I think you have ACKs on
> everything then.
> 
> Acked-by: Justin Pettit <jpettit at nicira.com>

Thanks, I applied this series to master.



More information about the dev mailing list