When a port leaves a bridge, the bridge layer sets its STP state to Disabled.
But switchdev users seem to need to restore its state to Forwarding to keep the communication with the CPU port functional (see DSA and Rocker for instance). br_set_state() tries to set the switchdev port state, if correctly implemented. So call it within del_nbp() to restore the Forwarding state on port removal. Does that make sense or are there scenarios where we don't want this behavior? Thanks, Vivien Didelot (3): net: bridge: log state when setting it net: bridge: set forwarding state on port removal net: dsa: remove dsa_slave_stp_update net/bridge/br_if.c | 5 +++++ net/bridge/br_stp.c | 5 ++--- net/dsa/slave.c | 17 ----------------- 3 files changed, 7 insertions(+), 20 deletions(-) -- 2.7.1
