net: dsa: mv88e6xxx: prevent interrupt storm caused by mv88e6390x_port_set_cmode
authorHeiner Kallweit <hkallweit1@gmail.com>
Thu, 28 Feb 2019 06:39:15 +0000 (07:39 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sat, 2 Mar 2019 05:37:05 +0000 (21:37 -0800)
commited8fe20205ac054bf585156709de3913d1890f30
tree825394213cada27fad8238e16eedd707334a859a
parent5e1a99eae84999a2536f50a0beaf5d5262337f40
net: dsa: mv88e6xxx: prevent interrupt storm caused by mv88e6390x_port_set_cmode

When debugging another issue I faced an interrupt storm in this
driver (88E6390, port 9 in SGMII mode), consisting of alternating
link-up / link-down interrupts. Analysis showed that the driver
wanted to set a cmode that was set already. But so far
mv88e6390x_port_set_cmode() doesn't check this and powers down
SERDES, what causes the link to break, and eventually results in
the described interrupt storm.

Fix this by checking whether the cmode actually changes. We want
that the very first call to mv88e6390x_port_set_cmode() always
configures the registers, therefore initialize port.cmode with
a value that is different from any supported cmode value.
We have to take care that we only init the ports cmode once
chip->info->num_ports is set.

v2:
- add small helper and init the number of actual ports only

Fixes: 364e9d7776a3 ("net: dsa: mv88e6xxx: Power on/off SERDES on cmode change")
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/mv88e6xxx/chip.c
drivers/net/dsa/mv88e6xxx/port.c
drivers/net/dsa/mv88e6xxx/port.h