On 03/09/2016 04:29 AM, Tom Herbert wrote: [...]
diff --git a/include/uapi/linux/if_link.h b/include/uapi/linux/if_link.h index d452cea..6bebc97 100644 --- a/include/uapi/linux/if_link.h +++ b/include/uapi/linux/if_link.h @@ -456,6 +456,7 @@ enum { IFLA_VXLAN_GBP, IFLA_VXLAN_REMCSUM_NOPARTIAL, IFLA_VXLAN_COLLECT_METADATA, + IFLA_VXLAN_LABEL,Should this interface align with ip_gre that uses IFLA_GRE_FLOWINFO to set both traffic class and flow label in one shot?
For vxlan and geneve by the way, there's already IFLA_VXLAN_TOS and IFLA_GENEVE_TOS attribute that configure the tos for IPv4 or traffic class for IPv6 case on the device, and have special handling in the xmit path that they can inherit the tos/tc when configured as such on the device in non collect metadata case. Seems like grev6 is handled differently and also still without tunnel key support, if I see this correctly. Agree with you that some more consolidation would be nice.
__IFLA_VXLAN_MAX }; #define IFLA_VXLAN_MAX (__IFLA_VXLAN_MAX - 1) -- 1.9.3
