[ovs-dev] [bond megaflow v2 2/5] lib/flow: add dp_hash and recirc_id to struct flow

Andy Zhou azhou at nicira.com
Wed Mar 19 00:16:03 UTC 2014


On Tue, Mar 18, 2014 at 11:10 AM, Ben Pfaff <blp at nicira.com> wrote:

> On Tue, Mar 11, 2014 at 04:56:18PM -0700, Andy Zhou wrote:
> > Signed-off-by: Andy Zhou <azhou at nicira.com>
>
> The new NXM fields are weird because OVS can report them (via
> nxm_put_32m() calls in nx-match.c) but it can't accept them at all
> (because the fields aren't listed in the table in meta-flow.c) or
> match on them.  I don't know if that's intentional but it's
> inconsistent with the comments in nicira-ext.h that talk about masking
> (a field that can't be matched definitely can't be masked).
> In previous discussions, I think we concluded that we want
> recirculation to be transparent to userspace (and to OpenFlow), so I
> suspect that the recirculation ID, at least, should not be mentioned
> in nicira-ext.h or in nx-match.c (or meta-flow.[ch]).  I'm not sure
> about dp_hash, but so far I think we only plan to use that internally
> as well, so I'd be inclined toward doing the same for it.
>
I would like those fields to be visible to the ovs-ofctl.  I can add logic
to make sure
the controller can not set them.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20140318/d6e253ae/attachment-0005.html>


More information about the dev mailing list