| From 74b63c633ed6d08d0ff97c5a2ce37e8b9be49939 Mon Sep 17 00:00:00 2001 |
| From: Aaro Koskinen <aaro.koskinen@nokia.com> |
| Date: Wed, 27 Mar 2019 22:35:36 +0200 |
| Subject: net: stmmac: ratelimit RX error logs |
| |
| [ Upstream commit 972c9be784e077bc56472c78243e0326e525b689 ] |
| |
| Ratelimit RX error logs. |
| |
| Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Signed-off-by: Sasha Levin (Microsoft) <sashal@kernel.org> |
| --- |
| drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 14 ++++++++------ |
| 1 file changed, 8 insertions(+), 6 deletions(-) |
| |
| diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c |
| index f765869bd864..0ccf91a08290 100644 |
| --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c |
| +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c |
| @@ -3407,9 +3407,10 @@ static int stmmac_rx(struct stmmac_priv *priv, int limit, u32 queue) |
| * ignored |
| */ |
| if (frame_len > priv->dma_buf_sz) { |
| - netdev_err(priv->dev, |
| - "len %d larger than size (%d)\n", |
| - frame_len, priv->dma_buf_sz); |
| + if (net_ratelimit()) |
| + netdev_err(priv->dev, |
| + "len %d larger than size (%d)\n", |
| + frame_len, priv->dma_buf_sz); |
| priv->dev->stats.rx_length_errors++; |
| break; |
| } |
| @@ -3466,9 +3467,10 @@ static int stmmac_rx(struct stmmac_priv *priv, int limit, u32 queue) |
| } else { |
| skb = rx_q->rx_skbuff[entry]; |
| if (unlikely(!skb)) { |
| - netdev_err(priv->dev, |
| - "%s: Inconsistent Rx chain\n", |
| - priv->dev->name); |
| + if (net_ratelimit()) |
| + netdev_err(priv->dev, |
| + "%s: Inconsistent Rx chain\n", |
| + priv->dev->name); |
| priv->dev->stats.rx_dropped++; |
| break; |
| } |
| -- |
| 2.20.1 |
| |