net: dsa: mv88e6xxx: Fix counting of ATU violations
authorAndrew Lunn <andrew@lunn.ch>
Tue, 5 Feb 2019 23:02:58 +0000 (00:02 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 6 Feb 2019 00:37:52 +0000 (16:37 -0800)
commit75c05a74e745ae7d663b04d75777af80ada2233c
tree1d1d2945642d71dccad3b7c4b5c1a6e3a3924cfd
parent9c0bda64abb880f5b93418af768dd7dde4e238fe
net: dsa: mv88e6xxx: Fix counting of ATU violations

The ATU port vector contains a bit per port of the switch. The code
wrongly used it as a port number, and incremented a port counter. This
resulted in the wrong interfaces counter being incremented, and
potentially going off the end of the array of ports.

Fix this by using the source port ID for the violation, which really
is a port number.

Reported-by: Chris Healy <Chris.Healy@zii.aero>
Tested-by: Chris Healy <Chris.Healy@zii.aero>
Fixes: 65f60e4582bd ("net: dsa: mv88e6xxx: Keep ATU/VTU violation statistics")
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/mv88e6xxx/global1_atu.c