From: Vladimir Oltean <vladimir.olt...@nxp.com>

xdp_do_redirect already contains:
-> dev_map_enqueue
   -> __xdp_enqueue
      -> bq_enqueue
         -> bq_xmit_all // if we have more than 16 frames

So the logic from enetc will never be hit, because ENETC_DEFAULT_TX_WORK
is 128.

Signed-off-by: Vladimir Oltean <vladimir.olt...@nxp.com>
---
 drivers/net/ethernet/freescale/enetc/enetc.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/net/ethernet/freescale/enetc/enetc.c 
b/drivers/net/ethernet/freescale/enetc/enetc.c
index 469170076efa..c7b940979314 100644
--- a/drivers/net/ethernet/freescale/enetc/enetc.c
+++ b/drivers/net/ethernet/freescale/enetc/enetc.c
@@ -1324,11 +1324,6 @@ static int enetc_clean_rx_ring_xdp(struct enetc_bdr 
*rx_ring,
                                rx_ring->stats.xdp_redirect++;
                        }
 
-                       if (unlikely(xdp_redirect_frm_cnt > 
ENETC_DEFAULT_TX_WORK)) {
-                               xdp_do_flush_map();
-                               xdp_redirect_frm_cnt = 0;
-                       }
-
                        break;
                default:
                        bpf_warn_invalid_xdp_action(xdp_act);
-- 
2.25.1

Reply via email to