> > From: Stefan Chulski <stef...@marvell.com> > > > > This patch doesn't change any functionality, but just extend MIB > > counter register and ethtool-statistic names with "err". > > > > The counter MVPP2_MIB_FRAGMENTS_RCVD in fact is Error counter. > > Extend REG name and appropriated ethtool statistic reg-name with the > > ERR/err. > > > --- a/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c > > +++ b/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c > > @@ -1566,7 +1566,7 @@ static u32 mvpp2_read_index(struct mvpp2 > *priv, u32 index, u32 reg) > > { MVPP2_MIB_FC_RCVD, "fc_received" }, > > { MVPP2_MIB_RX_FIFO_OVERRUN, "rx_fifo_overrun" }, > > { MVPP2_MIB_UNDERSIZE_RCVD, "undersize_received" }, > > - { MVPP2_MIB_FRAGMENTS_RCVD, "fragments_received" }, > > + { MVPP2_MIB_FRAGMENTS_ERR_RCVD, "fragments_err_received" }, > > Hi Stefan > > I suspect this is now ABI and you cannot change it. You at least need to argue > why it is not ABI. > > Andrew
Hi Andrew, I not familiar with ABI concept. Does this mean we cannot change, fix or extend driver ethtool counters? Thanks, Stefan.