Re: [PATCH 2/6] vxlan: Group Policy extension
From: Tom Herbert <hidden>
Date: 2015-01-12 18:14:58
quoted hunk ↗ jump to hunk
diff --git a/include/uapi/linux/if_link.h b/include/uapi/linux/if_link.h index f7d0d2d..9f07bf5 100644 --- a/include/uapi/linux/if_link.h +++ b/include/uapi/linux/if_link.h@@ -370,10 +370,18 @@ enum { IFLA_VXLAN_UDP_CSUM, IFLA_VXLAN_UDP_ZERO_CSUM6_TX, IFLA_VXLAN_UDP_ZERO_CSUM6_RX, + IFLA_VXLAN_EXTENSION, __IFLA_VXLAN_MAX }; #define IFLA_VXLAN_MAX (__IFLA_VXLAN_MAX - 1) +enum { + IFLA_VXLAN_EXT_UNSPEC, + IFLA_VXLAN_EXT_GBP, + __IFLA_VXLAN_EXT_MAX, +}; +#define IFLA_VXLAN_EXT_MAX (__IFLA_VXLAN_EXT_MAX - 1) +
Creating a level of indirection for extensions seems overly complicated to me. Why not just define IFLA_VXLAN_GBP as just another enum above? Tom