netdev - Re: [PATCH net-next 02/11] net: fec: add fec_rx_error

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <aWZqcULecGZK5nxS@lizhi-Precision-Tower-5810>
Date: Tue, 13 Jan 2026 10:53:21 -0500
From: Frank Li <Frank.li@....com>
To: Wei Fang <wei.fang@....com>
Cc: shenwei.wang@....com, xiaoning.wang@....com, andrew+netdev@...n.ch,
	davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
	pabeni@...hat.com, ast@...nel.org, daniel@...earbox.net,
	hawk@...nel.org, john.fastabend@...il.com, sdf@...ichev.me,
	netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
	imx@...ts.linux.dev, bpf@...r.kernel.org
Subject: Re: [PATCH net-next 02/11] net: fec: add fec_rx_error_check() to
 check RX errors

On Tue, Jan 13, 2026 at 11:29:30AM +0800, Wei Fang wrote:
> Extract fec_rx_error_check() from fec_enet_rx_queue(), this helper is
> used to check RX errors. And it will be used in XDP and XDP zero copy
> paths in subsequent patches.
>
> Signed-off-by: Wei Fang <wei.fang@....com>
> ---

Reviewed-by: Frank Li <Frank.Li@....com>

>  drivers/net/ethernet/freescale/fec_main.c | 58 ++++++++++++++---------
>  1 file changed, 36 insertions(+), 22 deletions(-)
>
> diff --git a/drivers/net/ethernet/freescale/fec_main.c b/drivers/net/ethernet/freescale/fec_main.c
> index 85bcca932fd2..0fa78ca9bc04 100644
> --- a/drivers/net/ethernet/freescale/fec_main.c
> +++ b/drivers/net/ethernet/freescale/fec_main.c
> @@ -1746,6 +1746,41 @@ static void fec_enet_rx_vlan(const struct net_device *ndev, struct sk_buff *skb)
>  	}
>  }
>
> +static int fec_rx_error_check(struct net_device *ndev, u16 status)
> +{
> +	if (status & (BD_ENET_RX_LG | BD_ENET_RX_SH | BD_ENET_RX_NO |
> +		      BD_ENET_RX_CR | BD_ENET_RX_OV | BD_ENET_RX_LAST |
> +		      BD_ENET_RX_CL)) {
> +		ndev->stats.rx_errors++;
> +
> +		if (status & BD_ENET_RX_OV) {
> +			/* FIFO overrun */
> +			ndev->stats.rx_fifo_errors++;
> +			return -EIO;
> +		}
> +
> +		if (status & (BD_ENET_RX_LG | BD_ENET_RX_SH |
> +			      BD_ENET_RX_LAST)) {
> +			/* Frame too long or too short. */
> +			ndev->stats.rx_length_errors++;
> +			if ((status & BD_ENET_RX_LAST) && net_ratelimit())
> +				netdev_err(ndev, "rcv is not +last\n");
> +		}
> +
> +		/* CRC Error */
> +		if (status & BD_ENET_RX_CR)
> +			ndev->stats.rx_crc_errors++;
> +
> +		/* Report late collisions as a frame error. */
> +		if (status & (BD_ENET_RX_NO | BD_ENET_RX_CL))
> +			ndev->stats.rx_frame_errors++;
> +
> +		return -EIO;
> +	}
> +
> +	return 0;
> +}
> +
>  /* During a receive, the bd_rx.cur points to the current incoming buffer.
>   * When we update through the ring, if the next incoming buffer has
>   * not been given to the system, we just set the empty indicator,
> @@ -1806,29 +1841,8 @@ fec_enet_rx_queue(struct net_device *ndev, u16 queue_id, int budget)
>
>  		/* Check for errors. */
>  		status ^= BD_ENET_RX_LAST;
> -		if (status & (BD_ENET_RX_LG | BD_ENET_RX_SH | BD_ENET_RX_NO |
> -			   BD_ENET_RX_CR | BD_ENET_RX_OV | BD_ENET_RX_LAST |
> -			   BD_ENET_RX_CL)) {
> -			ndev->stats.rx_errors++;
> -			if (status & BD_ENET_RX_OV) {
> -				/* FIFO overrun */
> -				ndev->stats.rx_fifo_errors++;
> -				goto rx_processing_done;
> -			}
> -			if (status & (BD_ENET_RX_LG | BD_ENET_RX_SH
> -						| BD_ENET_RX_LAST)) {
> -				/* Frame too long or too short. */
> -				ndev->stats.rx_length_errors++;
> -				if (status & BD_ENET_RX_LAST)
> -					netdev_err(ndev, "rcv is not +last\n");
> -			}
> -			if (status & BD_ENET_RX_CR)	/* CRC Error */
> -				ndev->stats.rx_crc_errors++;
> -			/* Report late collisions as a frame error. */
> -			if (status & (BD_ENET_RX_NO | BD_ENET_RX_CL))
> -				ndev->stats.rx_frame_errors++;
> +		if (unlikely(fec_rx_error_check(ndev, status)))
>  			goto rx_processing_done;
> -		}
>
>  		/* Process the incoming frame. */
>  		ndev->stats.rx_packets++;
> --
> 2.34.1
>