From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesper Dangaard Brouer Subject: [net PATCH] mlx4: fix XDP_TX is acting like XDP_PASS on TX ring full Date: Fri, 16 Sep 2016 22:36:12 +0200 Message-ID: <20160916203535.20016.45146.stgit@firesoul> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Cc: Eric Dumazet , tom@herbertland.com, bblanco@plumgrid.com, Jesper Dangaard Brouer , rana.shahot@gmail.com, "David S. Miller" To: netdev@vger.kernel.org, tariqt@mellanox.com Return-path: Received: from mx1.redhat.com ([209.132.183.28]:56038 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965465AbcIPUgP (ORCPT ); Fri, 16 Sep 2016 16:36:15 -0400 Sender: netdev-owner@vger.kernel.org List-ID: The XDP_TX action can fail transmitting the frame in case the TX ring is full or port is down. In case of TX failure it should drop the frame, and not as now call 'break' which is the same as XDP_PASS. Fixes: 9ecc2d86171a ("net/mlx4_en: add xdp forwarding and data write support") Signed-off-by: Jesper Dangaard Brouer --- Note, this fix have nothing to do with the page-refcnt bug I just reported. drivers/net/ethernet/mellanox/mlx4/en_rx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/mellanox/mlx4/en_rx.c b/drivers/net/ethernet/mellanox/mlx4/en_rx.c index 2040dad8611d..adcd55c655db 100644 --- a/drivers/net/ethernet/mellanox/mlx4/en_rx.c +++ b/drivers/net/ethernet/mellanox/mlx4/en_rx.c @@ -906,7 +906,7 @@ int mlx4_en_process_rx_cq(struct net_device *dev, struct mlx4_en_cq *cq, int bud length, tx_index, &doorbell_pending)) goto consumed; - break; + goto next; /* Drop on xmit failure */ default: bpf_warn_invalid_xdp_action(act); case XDP_ABORTED: