[ovs-dev] [PATCH] openvswitch.h: Introduce new fields for mega flow.

Andy Zhou azhou at nicira.com
Thu May 2 21:13:19 UTC 2013


OVS_KEY_ATTR_MEGA may not be strictly required in the end, but it makes the
of ovs-dpctl changes easier.

I will try to remove it in the next few commits.


On Thu, May 2, 2013 at 1:35 PM, Ben Pfaff <blp at nicira.com> wrote:

> On Thu, May 02, 2013 at 01:30:05PM -0700, Andy Zhou wrote:
> > +/**
> > + * enum ovs_mask_attr - attributes for %OVS_KEY_ATTR_MASK.
> > + * @OVS_MASK_ATTR_TYPE: The %OVS_KEY_ATTR_* whose value is matched with
> a mask.
> > + * Attributes that nested attributes may not be masked; specify masks
> on the
>
> s/that nested/that have nested/
>
> What is OVS_KEY_ATTR_MEGA used for?  If it is a flag that indicates a
> megaflow key, why is OVS_FLOW_ATTR_MATCH also present?  I would have
> expected to see OVS_FLOW_ATTR_MATCH but not OVS_KEY_ATTR_MEGA.
>
> Thanks,
>
> Ben.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20130502/327a2f21/attachment-0003.html>


More information about the dev mailing list