mbox series

[net,0/2] net: Fix bridge enslavement failure

Message ID 20200910110127.3113683-1-idosch@idosch.org
Headers show
Series net: Fix bridge enslavement failure | expand

Message

Ido Schimmel Sept. 10, 2020, 11:01 a.m. UTC 
From: Ido Schimmel <idosch@nvidia.com>

Patch #1 fixes an issue in which an upper netdev cannot be enslaved to a
bridge when it has multiple netdevs with different parent identifiers
beneath it.

Patch #2 adds a test case using two netdevsim instances.

Ido Schimmel (2):
  net: Fix bridge enslavement failure
  selftests: rtnetlink: Test bridge enslavement with different parent
    IDs

 net/core/dev.c                           |  2 +-
 tools/testing/selftests/net/rtnetlink.sh | 47 ++++++++++++++++++++++++
 2 files changed, 48 insertions(+), 1 deletion(-)

Comments

Jakub Kicinski Sept. 10, 2020, 7:31 p.m. UTC | #1 
On Thu, 10 Sep 2020 14:01:25 +0300 Ido Schimmel wrote:
> From: Ido Schimmel <idosch@nvidia.com>
> 
> Patch #1 fixes an issue in which an upper netdev cannot be enslaved to a
> bridge when it has multiple netdevs with different parent identifiers
> beneath it.
> 
> Patch #2 adds a test case using two netdevsim instances.

Awesome, thanks Ido!

Reviewed-by: Jakub Kicinski <kuba@kernel.org>
David Miller Sept. 10, 2020, 10:07 p.m. UTC | #2 
From: Ido Schimmel <idosch@idosch.org>
Date: Thu, 10 Sep 2020 14:01:25 +0300

> From: Ido Schimmel <idosch@nvidia.com>
> 
> Patch #1 fixes an issue in which an upper netdev cannot be enslaved to a
> bridge when it has multiple netdevs with different parent identifiers
> beneath it.
> 
> Patch #2 adds a test case using two netdevsim instances.

Series applied and patch #1 queued up for -stable, thank you.