RE: [net-next 08/15] i40e/i40evf: organize and re-number feature flags

2017-08-15 Thread David Laight
From: Keller, Jacob E > Sent: 14 August 2017 23:11 > > From: David Miller [mailto:da...@davemloft.net] > > Sent: Saturday, August 12, 2017 1:04 PM > > From: Jeff Kirsher > > Date: Sat, 12 Aug 2017 04:08:41 -0700 > > > > > Also ensure that the flags variable is actually a u64 to guarantee > > > 64b

RE: [net-next 08/15] i40e/i40evf: organize and re-number feature flags

2017-08-14 Thread Keller, Jacob E
[net-next 08/15] i40e/i40evf: organize and re-number feature > flags > > From: Jeff Kirsher > Date: Sat, 12 Aug 2017 04:08:41 -0700 > > > Also ensure that the flags variable is actually a u64 to guarantee > > 64bits of space on all architectures. > > Why? You d

Re: [net-next 08/15] i40e/i40evf: organize and re-number feature flags

2017-08-12 Thread David Miller
From: Jeff Kirsher Date: Sat, 12 Aug 2017 04:08:41 -0700 > Also ensure that the flags variable is actually a u64 to guarantee > 64bits of space on all architectures. Why? You don't need 64-bits, you only need 27. This will be unnecessarily expensive on 32-bit platforms. Please don't do this.