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

Justin Pettit jpettit at nicira.com
Mon Oct 19 07:17:04 UTC 2015


> 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>

--Justin





More information about the dev mailing list