Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1570555

Re: [PATCH 12/17] net: stmmac: rename rx_crc to rx_crc_errors

Show all headers | View raw

On 1/31/2017 10:11 AM, Corentin Labbe wrote:
> The ethtool stat counter rx_crc from stmmac is mis-named, the name
> seems to speak about the number of RX CRC done, but in fact it is about
> errors.
>
> This patch rename it to rx_crc_errors, just like the same ifconfig
> counter.
>
> Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>

Acked-by: Giuseppe Cavallaro <peppe.cavallaro@st.com>

> ---
>  drivers/net/ethernet/stmicro/stmmac/common.h         | 2 +-
>  drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c   | 2 +-
>  drivers/net/ethernet/stmicro/stmmac/enh_desc.c       | 2 +-
>  drivers/net/ethernet/stmicro/stmmac/norm_desc.c      | 2 +-
>  drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c | 2 +-
>  5 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/net/ethernet/stmicro/stmmac/common.h b/drivers/net/ethernet/stmicro/stmmac/common.h
> index 9da4877..b7ee15a 100644
> --- a/drivers/net/ethernet/stmicro/stmmac/common.h
> +++ b/drivers/net/ethernet/stmicro/stmmac/common.h
> @@ -67,7 +67,7 @@ struct stmmac_extra_stats {
>  	unsigned long overflow_error;
>  	unsigned long ipc_csum_error;
>  	unsigned long rx_collision;
> -	unsigned long rx_crc;
> +	unsigned long rx_crc_errors;
>  	unsigned long dribbling_bit;
>  	unsigned long rx_length;
>  	unsigned long rx_mii;
> diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c b/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c
> index 8816515..843ec69 100644
> --- a/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c
> +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c
> @@ -103,7 +103,7 @@ static int dwmac4_wrback_get_rx_status(void *data, struct stmmac_extra_stats *x,
>  			x->rx_mii++;
>
>  		if (unlikely(rdes3 & RDES3_CRC_ERROR)) {
> -			x->rx_crc++;
> +			x->rx_crc_errors++;
>  			stats->rx_crc_errors++;
>  		}
>
> diff --git a/drivers/net/ethernet/stmicro/stmmac/enh_desc.c b/drivers/net/ethernet/stmicro/stmmac/enh_desc.c
> index 8427643..323b59e 100644
> --- a/drivers/net/ethernet/stmicro/stmmac/enh_desc.c
> +++ b/drivers/net/ethernet/stmicro/stmmac/enh_desc.c
> @@ -221,7 +221,7 @@ static int enh_desc_get_rx_status(void *data, struct stmmac_extra_stats *x,
>  			x->rx_mii++;
>
>  		if (unlikely(rdes0 & RDES0_CRC_ERROR)) {
> -			x->rx_crc++;
> +			x->rx_crc_errors++;
>  			stats->rx_crc_errors++;
>  		}
>  		ret = discard_frame;
> diff --git a/drivers/net/ethernet/stmicro/stmmac/norm_desc.c b/drivers/net/ethernet/stmicro/stmmac/norm_desc.c
> index 5a0d4b0..efb818e 100644
> --- a/drivers/net/ethernet/stmicro/stmmac/norm_desc.c
> +++ b/drivers/net/ethernet/stmicro/stmmac/norm_desc.c
> @@ -111,7 +111,7 @@ static int ndesc_get_rx_status(void *data, struct stmmac_extra_stats *x,
>  			stats->collisions++;
>  		}
>  		if (unlikely(rdes0 & RDES0_CRC_ERROR)) {
> -			x->rx_crc++;
> +			x->rx_crc_errors++;
>  			stats->rx_crc_errors++;
>  		}
>  		ret = discard_frame;
> diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c
> index 69db8cb..6ca0a10 100644
> --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c
> +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_ethtool.c
> @@ -61,7 +61,7 @@ static const struct stmmac_stats stmmac_gstrings_stats[] = {
>  	STMMAC_STAT(overflow_error),
>  	STMMAC_STAT(ipc_csum_error),
>  	STMMAC_STAT(rx_collision),
> -	STMMAC_STAT(rx_crc),
> +	STMMAC_STAT(rx_crc_errors),
>  	STMMAC_STAT(dribbling_bit),
>  	STMMAC_STAT(rx_length),
>  	STMMAC_STAT(rx_mii),
>

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 12/17] net: stmmac: rename rx_crc to rx_crc_errors Corentin Labbe <clabbe.montjoie@gmail.com> - 2017-01-31 10:30 +0100
  Re: [PATCH 12/17] net: stmmac: rename rx_crc to rx_crc_errors Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:10 +0100

csiph-web