net: ena: Don't check if XDP program is loaded in ena_xdp_execute()
authorDavid Arinzon <darinzon@amazon.com>
Mon, 1 Jan 2024 19:08:50 +0000 (19:08 +0000)
committerJakub Kicinski <kuba@kernel.org>
Thu, 4 Jan 2024 02:00:57 +0000 (18:00 -0800)
This check is already done in ena_clean_rx_irq() which indirectly
calls it.
This function is called in napi context and the driver doesn't
allow to change the XDP program without performing destruction and
reinitialization of napi context (part of ena_down/ena_up sequence).

Signed-off-by: Shay Agroskin <shayagr@amazon.com>
Signed-off-by: David Arinzon <darinzon@amazon.com>
Link: https://lore.kernel.org/r/20240101190855.18739-7-darinzon@amazon.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/amazon/ena/ena_xdp.h

index 3fa8e80b18a9e6954ba39272a9dc7ce270e816a2..25204fb54c74fd2cc72b1d99317ec5f2f3556bad 100644 (file)
@@ -87,9 +87,6 @@ static inline int ena_xdp_execute(struct ena_ring *rx_ring, struct xdp_buff *xdp
 
        xdp_prog = READ_ONCE(rx_ring->xdp_bpf_prog);
 
-       if (!xdp_prog)
-               return verdict;
-
        verdict = bpf_prog_run_xdp(xdp_prog, xdp);
 
        switch (verdict) {