[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