Revert "net: dsa: stop updating master MTU from master.c"
authorVladimir Oltean <vladimir.oltean@nxp.com>
Thu, 31 Mar 2022 13:28:54 +0000 (16:28 +0300)
committerDavid S. Miller <davem@davemloft.net>
Fri, 1 Apr 2022 10:59:01 +0000 (11:59 +0100)
commit066dfc4290406b1b0b014ae3267d4266a344efd1
treeb06757ae0b727e6798f325327e117e1de99a6ac7
parent1effe8ca4e34c34cdd9318436a4232dcb582ebf4
Revert "net: dsa: stop updating master MTU from c"

This reverts commit a1ff94c2973c43bc1e2677ac63ebb15b1d1ff846.

Switch drivers that don't implement ->port_change_mtu() will cause the
DSA master to remain with an MTU of 1500, since we've deleted the other
code path. In turn, this causes a regression for those systems, where
MTU-sized traffic can no longer be terminated.

Revert the change taking into account the fact that rtnl_lock() is now
taken top-level from the callers of dsa_master_setup() and
dsa_master_teardown(). Also add a comment in order for it to be
absolutely clear why it is still needed.

Fixes: a1ff94c2973c ("net: dsa: stop updating master MTU from master.c")
Reported-by: Luiz Angelo Daros de Luca <luizluca@gmail.com>
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Tested-by: Luiz Angelo Daros de Luca <luizluca@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/dsa/master.c