net: fec: use netdev_err_once() instead of netdev_err()
authorWei Fang <wei.fang@nxp.com>
Thu, 6 Jul 2023 08:10:12 +0000 (16:10 +0800)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 11 Jul 2023 08:00:49 +0000 (10:00 +0200)
commit84a10947198792d038527af9c3994782ecb37c82
treed7ed94c15cf9748f65008bfa85a08683f1a9205b
parent56b3c6ba53d0e9649ea5e4089b39cadde13aaef8
net: fec: use netdev_err_once() instead of netdev_err()

In the case of heavy XDP traffic to be transmitted, the console
will print the error log continuously if there are lack of enough
BDs to accommodate the frames. The log looks like below.

[  160.013112] fec 30be0000.ethernet eth0: NOT enough BD for SG!
[  160.023116] fec 30be0000.ethernet eth0: NOT enough BD for SG!
[  160.028926] fec 30be0000.ethernet eth0: NOT enough BD for SG!
[  160.038946] fec 30be0000.ethernet eth0: NOT enough BD for SG!
[  160.044758] fec 30be0000.ethernet eth0: NOT enough BD for SG!

Not only will this log be replicated and redundant, it will also
degrade XDP performance. So we use netdev_err_once() instead of
netdev_err() now.

Fixes: 6d6b39f180b8 ("net: fec: add initial XDP support")
Signed-off-by: Wei Fang <wei.fang@nxp.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/freescale/fec_main.c