From: Hangbin Liu <liuhang...@gmail.com> Date: Wed, 17 Apr 2019 20:12:11 +0800
> If we add a bond device which is already the master of the team interface, > we will hold the team->lock in team_add_slave() first and then request the > lock in team_set_mac_address() again. The functions are called like: > > - team_add_slave() > - team_port_add() > - team_port_enter() > - team_modeop_port_enter() > - __set_port_dev_addr() > - dev_set_mac_address() > - bond_set_mac_address() > - dev_set_mac_address() > - team_set_mac_address > > Although team_upper_dev_link() would check the upper devices but it is > called too late. Fix it by adding a checking before processing the slave. > > Fixes: 3d249d4ca7d0 ("net: introduce ethernet teaming device") > Signed-off-by: Hangbin Liu <liuhang...@gmail.com> Jiri, please review. Thank you.