On Wed, Sep 25, 2013 at 9:42 AM, Jarno Rajahalme <jrajaha...@nicira.com> wrote: > diff --git a/include/linux/openvswitch.h b/include/linux/openvswitch.h > index 09c26b5..2642ca5 100644 > --- a/include/linux/openvswitch.h > +++ b/include/linux/openvswitch.h > @@ -291,6 +291,8 @@ enum ovs_key_attr { > OVS_KEY_ATTR_MPLS = 62, /* array of struct ovs_key_mpls. > * The implementation may restrict > * the accepted length of the array. */ > + OVS_KEY_ATTR_TCP_FLAGS, /* be16 TCP flags. */
I would put this after SCTP and not use a temporary value. The MPLS patch should hopefully land soon but since there's no kernel code currently checked-in for that, there's no interface breakage when moving things around. We can just allocate IDs based on the order that the patches are applied. Otherwise this looks OK to me. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev