On Wed, Nov 01, 2017 at 08:43:20AM -0700, Greg Rose wrote: > On 10/30/2017 06:13 PM, fukaige wrote: > >From: Kaige Fu <[email protected]> > > > >IFF_NO_QUEUE is checked twice. This cause a situation > >that HAVE_IFF_NO_QUEUE is defined no matter whether > >the flag IFF_NO_QUEUE is defined in kernel or not. > > > >Signed-off-by: Kaige Fu <[email protected]> > >--- > > datapath/linux/compat/include/linux/netdevice.h | 3 --- > > 1 file changed, 3 deletions(-) > > > >diff --git a/datapath/linux/compat/include/linux/netdevice.h > >b/datapath/linux/compat/include/linux/netdevice.h > >index a5d4ee8..3c3cf42 100644 > >--- a/datapath/linux/compat/include/linux/netdevice.h > >+++ b/datapath/linux/compat/include/linux/netdevice.h > >@@ -22,9 +22,6 @@ struct net; > > #define IFF_LIVE_ADDR_CHANGE 0 > > #endif > >-#ifndef IFF_NO_QUEUE > >-#define IFF_NO_QUEUE 0 > >-#endif > > #ifndef IFF_OPENVSWITCH > > #define IFF_OPENVSWITCH 0 > > #endif > > > > Looks correct to me. > > Reviewed-by: Greg Rose <[email protected]>
Thanks Kaige and Greg. I applied this to master. Let me know if it should be backported. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
