Hi Andrew, On Wed, 6 Feb 2019 00:02:58 +0100, Andrew Lunn <and...@lunn.ch> wrote: > 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.he...@zii.aero> > Tested-by: Chris Healy <chris.he...@zii.aero> > Fixes: 65f60e4582bd ("net: dsa: mv88e6xxx: Keep ATU/VTU violation statistics") > Signed-off-by: Andrew Lunn <and...@lunn.ch>
Hey I guess you forgot to Cc me on this one. FWIW: Reviewed-by: Vivien Didelot <vivien.dide...@gmail.com>