On 17-04-21 09:12 AM, Jiri Pirko wrote:
Fri, Apr 21, 2017 at 12:55:31PM CEST, j...@mojatatu.com wrote:
From: Jamal Hadi Salim <j...@mojatatu.com>
+#define TCA_FLAG_LARGE_DUMP_ON (1 << 0)
This is u32 "flags" that could not be extended for other use in future.
I'm missing the point. Also, you don't check the rest of the bits for 0
as requested by DaveM.
As far as this is unextendable, please have this as u8 with values 0 and 1
as I originally suggested.
I don't understand why are we running in circles about this...
If i have a 32 bit space of which i am using one bit.
The sender (user space) zeroes the bits except the one they are
interested in. The kernel checks the bits they are interested in.
Future - we add one more bit and the same philosophy applies.
Older kernels dont see this bit but they dont have the feature
to begin with. So where is the lack of extensibility?
Jiri, there is a balance between extensibility and performance.
It is senseless to use a TLV just so i can set a 0/1(true/false).
cheers,
jamal
