[ovs-dev] [PATCH v3 1/5] logical-fields: New header for logical field assignments.
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>
More information about the dev