[PATCH 3/3] bridge: Add support for VLANs

David Ahern dsa at cumulusnetworks.com
Wed Nov 25 11:13:51 PST 2015


On 11/25/15 12:11 PM, Tobias Jungel wrote:
> On Mi, 2015-11-25 at 10:11 -0700, David Ahern wrote:
>>> afaik RTEXT_FILTER_BRVLAN_COMPRESSED is the default for the kernel
>> to
>>> send notifications, so I see this rather mandatory to have.
>>
>> sure, can add that one in time. Right now the request below is for
>> non-compressed, that's why it is marked as unexpected here. The
>> complication with the compressed version is that it crosses
>> consecutive
>> attributes: start is in the first, end in the second.
>>
>> -----8<-----
>
> I can offer to do this tomorrow.

Ok. I'll send v2 now. As you noted RTEXT_FILTER_BRVLAN_COMPRESSED is the 
default for notifications, so it is needed. The problem is that 
ao_parse_af is called per nested attribute, where for bridge to handle 
it needs the attr starting at the nest.




More information about the libnl mailing list