Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Stephen Hemminger
On Fri, 7 Apr 2017 21:16:49 +0300 Ido Schimmel wrote: > On Fri, Apr 07, 2017 at 01:37:53PM -0400, Stephen Hemminger wrote: > > Fix it right now. The patch you propose is too half baked. > > Given this patch needs to be back ported to three different stable > kernels I tried

Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Stephen Hemminger
On Fri, 7 Apr 2017 21:16:49 +0300 Ido Schimmel wrote: > On Fri, Apr 07, 2017 at 01:37:53PM -0400, Stephen Hemminger wrote: > > Fix it right now. The patch you propose is too half baked. > > Given this patch needs to be back ported to three different stable > kernels I tried

Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Ido Schimmel
On Fri, Apr 07, 2017 at 01:37:53PM -0400, Stephen Hemminger wrote: > Fix it right now. The patch you propose is too half baked. Given this patch needs to be back ported to three different stable kernels I tried to keep the amount of changes to a minimum while fixing the bug. I'll work on the

Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Stephen Hemminger
On Fri, 7 Apr 2017 18:43:06 +0300 Nikolay Aleksandrov wrote: > On 07/04/17 18:36, Stephen Hemminger wrote: > > On Fri, 7 Apr 2017 18:27:37 +0300 > > Nikolay Aleksandrov wrote: > > > >> On 07/04/17 18:22, Stephen Hemminger wrote: >

Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Nikolay Aleksandrov
On 07/04/17 18:36, Stephen Hemminger wrote: > On Fri, 7 Apr 2017 18:27:37 +0300 > Nikolay Aleksandrov wrote: > >> On 07/04/17 18:22, Stephen Hemminger wrote: >>> On Fri, 7 Apr 2017 17:19:48 +0300 >>> Nikolay Aleksandrov wrote: >>>

Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Stephen Hemminger
On Fri, 7 Apr 2017 18:27:37 +0300 Nikolay Aleksandrov wrote: > On 07/04/17 18:22, Stephen Hemminger wrote: > > On Fri, 7 Apr 2017 17:19:48 +0300 > > Nikolay Aleksandrov wrote: > > > >> On 07/04/17 17:10, Stephen Hemminger wrote: >

Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Nikolay Aleksandrov
On 07/04/17 18:22, Stephen Hemminger wrote: > On Fri, 7 Apr 2017 17:19:48 +0300 > Nikolay Aleksandrov wrote: > >> On 07/04/17 17:10, Stephen Hemminger wrote: >>> On Fri, 7 Apr 2017 15:49:15 +0300 >>> wrote: >>> From: Ido Schimmel

Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Stephen Hemminger
On Fri, 7 Apr 2017 17:19:48 +0300 Nikolay Aleksandrov wrote: > On 07/04/17 17:10, Stephen Hemminger wrote: > > On Fri, 7 Apr 2017 15:49:15 +0300 > > wrote: > > > >> From: Ido Schimmel > >> > >> Peter reported a kernel

Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Nikolay Aleksandrov
On 07/04/17 17:10, Stephen Hemminger wrote: > On Fri, 7 Apr 2017 15:49:15 +0300 > wrote: > >> From: Ido Schimmel >> >> Peter reported a kernel oops when executing the following command: >> >> $ ip link add name test type bridge vlan_default_pvid 1 >> >>

Re: [PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread Stephen Hemminger
On Fri, 7 Apr 2017 15:49:15 +0300 wrote: > From: Ido Schimmel > > Peter reported a kernel oops when executing the following command: > > $ ip link add name test type bridge vlan_default_pvid 1 > > [13634.939408] BUG: unable to handle kernel NULL

[PATCH net] bridge: netlink: register netdevice before executing changelink

2017-04-07 Thread idosch
From: Ido Schimmel Peter reported a kernel oops when executing the following command: $ ip link add name test type bridge vlan_default_pvid 1 [13634.939408] BUG: unable to handle kernel NULL pointer dereference at 0190 [13634.939436] IP: __vlan_add+0x73/0x5f0