[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <21a6a034-31e2-b74e-731c-33f2e41f6546@mojatatu.com>
Date: Mon, 24 Apr 2017 21:25:27 -0400
From: Jamal Hadi Salim <jhs@...atatu.com>
To: Benjamin LaHaise <benjamin.lahaise@...ronome.com>
Cc: David Miller <davem@...emloft.net>, netdev@...r.kernel.org,
bcrl@...ck.org, Jiri Pirko <jiri@...nulli.us>
Subject: Re: [PATCH net-next 0/2] flower: add MPLS matching support
On 17-04-24 09:05 PM, Benjamin LaHaise wrote:
> On Mon, Apr 24, 2017 at 08:58:18PM -0400, Jamal Hadi Salim wrote:
>> On 17-04-24 02:32 PM, David Miller wrote:
>> You have 3 TLVs, one of which is u8 that only allows use of 3 bits.
>> The other is a u32 which allows only 20 bits to be set.
>
> What are the new rules for TLVs -- do you want a new type added for the
> subset values that are limited to the number of bits actually used? A
> pointed here would be helpful.
>
The rules are to check for bits that are not used.
For example in the BOS TLV it would be required to
verify that the user did not set any other bit other than the
bos field. It is required to reject the whole thing if the user
set any other of the u8 bits.
cheers,
jamal
Powered by blists - more mailing lists