net: dsa: suppress appending ethtool stats to LAG DSA masters
authorVladimir Oltean <vladimir.oltean@nxp.com>
Sun, 11 Sep 2022 01:07:01 +0000 (04:07 +0300)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 20 Sep 2022 08:32:36 +0000 (10:32 +0200)
commitcfeb84a52fcbdb12c7364c2cab4529c5c5558c35
tree9e15cd542b288f37cf5bb20a3e22ee0a9d89e643
parent6e61b55c6d7f20f5619e831fa171f65bfbcf03c7
net: dsa: suppress appending ethtool stats to LAG DSA masters

Similar to the discussion about tracking the admin/oper state of LAG DSA
masters, we have the problem here that struct dsa_port *cpu_dp caches a
single pair of orig_ethtool_ops and netdev_ops pointers.

So if we call dsa_master_setup(bond0, cpu_dp) where cpu_dp is also the
dev->dsa_ptr of one of the physical DSA masters, we'd effectively
overwrite what we cached from that physical netdev with what replaced
from the bonding interface.

We don't need DSA ethtool stats on the bonding interface when used as
DSA master, it's good enough to have them just on the physical DSA
masters, so suppress this logic.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
net/dsa/master.c