On 06/04/17 23:44, Ido Schimmel wrote:
> On Thu, Apr 06, 2017 at 11:26:06PM +0300, Nikolay Aleksandrov wrote:
>> Actually making br_vlan_init() idempotent might work, keep the code as-is
>> just init the
>> the vlans before the changelink() in newlink(), then the second vlan_init()
>> inside
On Thu, Apr 06, 2017 at 11:26:06PM +0300, Nikolay Aleksandrov wrote:
> Actually making br_vlan_init() idempotent might work, keep the code as-is
> just init the
> the vlans before the changelink() in newlink(), then the second vlan_init()
> inside would
> be a no-op, but it will work for the old
On 06/04/17 21:58, Ido Schimmel wrote:
> +Nik
>
Thanks!
> On Thu, Apr 06, 2017 at 08:19:35PM +0200, Peter V. Saveliev wrote:
>> Operation:
>>
>> # ip link add name test type bridge vlan_default_pvid 1
>>
>> Result:
>>
>> Kernel oops. Minimal required netlink packet structure:
>>
>> ifinfmsg:
>>
+Nik
On Thu, Apr 06, 2017 at 08:19:35PM +0200, Peter V. Saveliev wrote:
> Operation:
>
> # ip link add name test type bridge vlan_default_pvid 1
>
> Result:
>
> Kernel oops. Minimal required netlink packet structure:
>
> ifinfmsg:
> - IFLA_IFNAME: "test"
> - IFLA_LINKINFO:
>
Operation:
# ip link add name test type bridge vlan_default_pvid 1
Result:
Kernel oops. Minimal required netlink packet structure:
ifinfmsg:
- IFLA_IFNAME: "test"
- IFLA_LINKINFO:
= IFLA_INFO_KIND: "bridge"
= IFLA_INFO_DATA: