On 5/22/2018 1:18 AM, Dan Gora wrote: > Fix compilation errors on Centos 7.4 when CONFIG_RTE_KNI_KMOD_ETHTOOL > is set to 'y'. > > Centos 7.4 needs HAVE_NDO_BRIDGE_GETLINK_FILTER_MASK_VLAN_FILL: > > igb_main.c: In function ‘igb_ndo_bridge_getlink’: > igb_main.c:2289:2: error: too few arguments to function > ‘ndo_dflt_bridge_getlink’ > return ndo_dflt_bridge_getlink(skb, pid, seq, dev, mode, 0, 0, nlflags); > ^ > > Centos 7.4 needs HAVE_VF_VLAN_PROTO and needs to redefine > ndo_set_vf_vlan to .extended.ndo_set_vf_vlan: > > igb_main.c:2318:2: error: unknown field ‘ndo_set_vf_vlan’ specified > in initializer > .ndo_set_vf_vlan = igb_ndo_set_vf_vlan, > ^ > > Signed-off-by: Dan Gora <d...@adax.com>
<...> > @@ -3920,6 +3921,11 @@ skb_set_hash(struct sk_buff *skb, __u32 hash, > __always_unused int type) > #if ((LINUX_VERSION_CODE >= KERNEL_VERSION(4, 9, 0)) || \ > (SLE_VERSION_CODE && SLE_VERSION_CODE >= SLE_VERSION(12, 3, 0))) > #define HAVE_VF_VLAN_PROTO > +#elif ( RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7,4) ) > +/* In RHEL/Centos 7.4, the "new" version of ndo_set_vf_vlan > + * is in the net_device_ops_extended struct */ > +#define HAVE_VF_VLAN_PROTO > +#define ndo_set_vf_vlan extended.ndo_set_vf_vlan This works but it is not really an "else" condition logically, what do you think about something like: #if ((LINUX_VERSION_CODE >= KERNEL_VERSION(4, 9, 0)) || \ (SLE_VERSION_CODE && SLE_VERSION_CODE >= SLE_VERSION(12, 3, 0)) || \ ( RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7,4) )) #define HAVE_VF_VLAN_PROTO #if ( RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7,4) ) /* In RHEL/Centos 7.4, the "new" version of ndo_set_vf_vlan * is in the net_device_ops_extended struct */ #define ndo_set_vf_vlan extended.ndo_set_vf_vlan #endif /* RHEL >= 7.4 */ #endif /* >= 4.9.0, >= SLES12SP3 */ > #endif /* >= 4.9.0, >= SLES12SP3 */ > > #if LINUX_VERSION_CODE < KERNEL_VERSION(4, 8, 0) >