From nobody Tue Apr 7 16:22:15 2026 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6843F3FFAA8 for ; Thu, 12 Mar 2026 18:38:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.43 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773340739; cv=none; b=dtL+6lkAcDwWPdgwFNrx/JBZjf8QZ5wOd7z1qwPCKqjM6TcWjmUj+osSvpQcEFn+Yay3FaSVk0NwDhcdsOCPbTSQabZWjTbnUOm6L2LKw9yrTYXyCZYKvSHkZiUoaVjU+rjUd3cx08DwEzF7kcyV9VrGUK1MTZ79Bc+8gzxHUSU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773340739; c=relaxed/simple; bh=3UxO3hGZusgNn8ofIA0kPIescWqHZiqa7lRiRs+eqWI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=fYtKvmQ299Qjc5KwZCnH3LfU4vxJ4Ei328s3nVIj2aw7oQlPf+/DbV2Q4GqlXlCuqcuDpDgMglWpojD67nHRKMy8BdvqO+ZwE7bA7nIG02hk7KLk8tel1JzwHrW0tumPwKpc+u2yuvlpskNLrC8gLOFW+N7CfupQsFBKc2RNeBI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Lsw11Son; arc=none smtp.client-ip=209.85.128.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Lsw11Son" Received: by mail-wm1-f43.google.com with SMTP id 5b1f17b1804b1-4852afd42ceso11655015e9.2 for ; Thu, 12 Mar 2026 11:38:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1773340736; x=1773945536; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=jSKBN+Tb2XKCGJildwBJPH5bXGsZAUeul85uzs2xixU=; b=Lsw11SonvI2upuS/evyzdmJJbBT+ilmTLQqle4qa3xMbQPphHTK/BTnRO4iYuuZXP1 Vx9Nx5Fgz/tSLDXb0mOR7Ws9oLWyJU2n8YoBHE8nKCAkBLPv1+FKLDicDVM0hf63Lb9w mVDEmF/Z4sfR7RN787AQ25uk4DFeEOdqJC7dsWUe5FGy3NO/5xl3WbQGEZNKoQ721JIp 8aJsh4au3GCs0KxAC6vXcfjwV/pACpIV7HyAKu1zSnaaBqMvtTuszlSTdgPpqf831m6k CV37DEHAFWONWA1bV/FCDA2Gp9NqB8dbuSSOGzZhZYig1m366xZVwdMbf/gXHIM7RWfF YmWg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1773340736; x=1773945536; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=jSKBN+Tb2XKCGJildwBJPH5bXGsZAUeul85uzs2xixU=; b=eEjTZTTmZc5RP54T5ILnrzEHcVyZXtbTffqJOnF3ReDnRjpSNhHx1PR7FgYI8HWdiJ gM7tHh/K9zxW5yEdJqGcIpuminy9ExNg92NG5AlWJF8Hl3SVbxHGNfu0wrHSF39TP1f6 rcy77MkE8zefLfWp4XwNcfAe5yomXy9V1sz+FIJWKf4yaBYKyYjOCbXRXiXoLaDo9Xm0 FzUsssfaf+KcKybqMhWvtx3sTjSCozk14k1fUktMez6Tk/eRxm4p4yEXXODBSyXx/4GG 3bp7xEW8lIp6Mse/G3E45kORwr8o1Veqy69qyN1lpAcC0YSPq6Sh+j8XQW0XqFY0sNEd h9mg== X-Forwarded-Encrypted: i=1; AJvYcCUEs8E4qrNTjdqh8do9lgR/E2/S6PqLEnGaFt1xCkgr+NGGdvyy86+bls1HbyONCFp8ksVwbHHqG2aVfjw=@vger.kernel.org X-Gm-Message-State: AOJu0YwFGnA/EfCcc/0RgZz9CunObo4M3nYY+jVD7PCxwKhPUA4L7j/K COqor35wK7NaRAKFLU9H6Hmr6BKFCW/rA3v7ouIXKpgvsqNZjV7dQDWR X-Gm-Gg: ATEYQzzFpN/xIYucpwqDAKb8gzdshJclZpp/xRNPanwpWLMs9yPrDue6pXO1It9RShH 9ko4/aKQzeVdLHdItETeCZuTbV7vArpLiZs2uIzQnTFJ+3xvVA0HXF3zO4iexZnv88tIYPWA778 Mbos6plX7Zv9nXX35gpj72xVp4iaoeuo+tJyLiwnLndyEtyoSzeM3WkHeBVb1GJhW4mUiW3Ug6J hzlmgpQydlU+GFBORb+Siy7Z0Tmgcpmu7o9X0J+ftWTPvQB6YPaWh3QB0UXr5bEu/PZGRwHXpvN 4Ui5879XgsFKjfQq822pSkPGde/YfLZV/gFzfzndN9YSsRXv6gpzThr0AHwcXyCozVc7SPMR7cV UKsuFDWG7KtNYur7c0qfUOQlPkh5ldUQ6JVPo2mTszGbqv8D7gNp/HO57jh6l1n2wsWrD7UQLqo 6QS1vJ4BjA5vRBO0vNxUH10Q0mxEQxDzCLdH7QkTCjYFH/HwQ7yDcA9cJQA48NXYAuHCPdTtY4j HaCzcs8unM= X-Received: by 2002:a05:600c:c4a5:b0:485:4388:348b with SMTP id 5b1f17b1804b1-485564892e6mr6973235e9.0.1773340735474; Thu, 12 Mar 2026 11:38:55 -0700 (PDT) Received: from turbo.teknoraver.net (net-5-95-156-124.cust.vodafonedsl.it. [5.95.156.124]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4854b5f6c24sm135560395e9.5.2026.03.12.11.38.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 12 Mar 2026 11:38:55 -0700 (PDT) From: Matteo Croce X-Google-Original-From: Matteo Croce To: Tony Nguyen , Przemek Kitszel , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Alexei Starovoitov , Daniel Borkmann , Jesper Dangaard Brouer , John Fastabend Cc: netdev@vger.kernel.org, bpf@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next v2 2/2] e1000e: add XDP_REDIRECT support Date: Thu, 12 Mar 2026 19:38:17 +0100 Message-ID: <20260312183817.104878-3-teknoraver@meta.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260312183817.104878-1-teknoraver@meta.com> References: <20260312183817.104878-1-teknoraver@meta.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add the ability to redirect packets to other devices via XDP_REDIRECT and to receive redirected frames from other devices via ndo_xdp_xmit. New functionality: - XDP_REDIRECT case in e1000_run_xdp() using xdp_do_redirect() - e1000_xdp_xmit() as the ndo_xdp_xmit callback for receiving redirected frames from other devices - xdp_do_flush() in e1000_finalize_xdp() for REDIR completions - xdp_features_set/clear_redirect_target() in e1000_xdp_setup() - NETDEV_XDP_ACT_REDIRECT and NETDEV_XDP_ACT_NDO_XMIT advertised Assisted-by: claude-opus-4-6 Signed-off-by: Matteo Croce --- drivers/net/ethernet/intel/e1000e/netdev.c | 85 +++++++++++++++++++++- 1 file changed, 81 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/intel/e1000e/netdev.c b/drivers/net/ether= net/intel/e1000e/netdev.c index 821b466b6673..1e28eaed7ae3 100644 --- a/drivers/net/ethernet/intel/e1000e/netdev.c +++ b/drivers/net/ethernet/intel/e1000e/netdev.c @@ -41,6 +41,7 @@ char e1000e_driver_name[] =3D "e1000e"; #define E1000_XDP_PASS 0 #define E1000_XDP_CONSUMED BIT(0) #define E1000_XDP_TX BIT(1) +#define E1000_XDP_REDIR BIT(2) =20 static int debug =3D -1; module_param(debug, int, 0); @@ -804,6 +805,9 @@ static void e1000_finalize_xdp(struct e1000_adapter *ad= apter, { struct e1000_ring *tx_ring =3D adapter->tx_ring; =20 + if (xdp_xmit & E1000_XDP_REDIR) + xdp_do_flush(); + if (xdp_xmit & E1000_XDP_TX) { /* Force memory writes to complete before letting h/w * know there are new descriptors to fetch. @@ -822,13 +826,14 @@ static void e1000_finalize_xdp(struct e1000_adapter *= adapter, * @adapter: board private structure * @xdp: XDP buffer containing packet data * - * Returns E1000_XDP_PASS, E1000_XDP_TX, or E1000_XDP_CONSUMED + * Returns E1000_XDP_PASS, E1000_XDP_TX, E1000_XDP_REDIR, or E1000_XDP_CON= SUMED **/ static int e1000_run_xdp(struct e1000_adapter *adapter, struct xdp_buff *x= dp) { struct bpf_prog *xdp_prog =3D READ_ONCE(adapter->xdp_prog); struct net_device *netdev =3D adapter->netdev; int result =3D E1000_XDP_PASS; + int err; u32 act; =20 if (!xdp_prog) @@ -845,6 +850,12 @@ static int e1000_run_xdp(struct e1000_adapter *adapter= , struct xdp_buff *xdp) if (result =3D=3D E1000_XDP_CONSUMED) goto out_failure; break; + case XDP_REDIRECT: + err =3D xdp_do_redirect(netdev, xdp, xdp_prog); + if (err) + goto out_failure; + result =3D E1000_XDP_REDIR; + break; default: bpf_warn_invalid_xdp_action(netdev, xdp_prog, act); fallthrough; @@ -1039,11 +1050,11 @@ static bool e1000_clean_rx_irq_xdp(struct e1000_rin= g *rx_ring, int *work_done, =20 /* page consumed by skb */ buffer_info->page =3D NULL; - } else if (xdp_res & E1000_XDP_TX) { + } else if (xdp_res & (E1000_XDP_TX | E1000_XDP_REDIR)) { xdp_xmit |=3D xdp_res; total_rx_bytes +=3D length; total_rx_packets++; - /* page consumed by XDP TX */ + /* page consumed by XDP TX/redirect */ buffer_info->page =3D NULL; } else { /* XDP_DROP / XDP_ABORTED - recycle page */ @@ -7815,6 +7826,11 @@ static int e1000_xdp_setup(struct net_device *netdev= , struct netdev_bpf *bpf) if (!need_reset) return 0; =20 + if (prog) + xdp_features_set_redirect_target(netdev, true); + else + xdp_features_clear_redirect_target(netdev); + if (running) e1000e_open(netdev); =20 @@ -7831,6 +7847,64 @@ static int e1000_xdp(struct net_device *netdev, stru= ct netdev_bpf *xdp) } } =20 +/** + * e1000_xdp_xmit - transmit XDP frames from another device + * @netdev: network interface device structure + * @n: number of frames to transmit + * @frames: array of XDP frame pointers + * @flags: XDP transmit flags + * + * This is the ndo_xdp_xmit callback, called when other devices redirect + * frames to this device. + **/ +static int e1000_xdp_xmit(struct net_device *netdev, int n, + struct xdp_frame **frames, u32 flags) +{ + struct e1000_adapter *adapter =3D netdev_priv(netdev); + struct e1000_ring *tx_ring =3D adapter->tx_ring; + struct netdev_queue *nq =3D netdev_get_tx_queue(netdev, 0); + int cpu =3D smp_processor_id(); + int nxmit =3D 0; + int i; + + if (unlikely(test_bit(__E1000_DOWN, &adapter->state))) + return -ENETDOWN; + + if (unlikely(flags & ~XDP_XMIT_FLAGS_MASK)) + return -EINVAL; + + if (!adapter->xdp_prog) + return -ENXIO; + + __netif_tx_lock(nq, cpu); + txq_trans_cond_update(nq); + + for (i =3D 0; i < n; i++) { + int err; + + err =3D e1000_xdp_xmit_ring(adapter, tx_ring, frames[i]); + if (err !=3D E1000_XDP_TX) + break; + nxmit++; + } + + if (unlikely(flags & XDP_XMIT_FLUSH)) { + /* Force memory writes to complete before letting h/w + * know there are new descriptors to fetch. + */ + wmb(); + if (adapter->flags2 & FLAG2_PCIM2PCI_ARBITER_WA) + e1000e_update_tdt_wa(tx_ring, + tx_ring->next_to_use); + else + writel(tx_ring->next_to_use, tx_ring->tail); + } + + __netif_tx_unlock(nq); + + return nxmit; +} + static const struct net_device_ops e1000e_netdev_ops =3D { .ndo_open =3D e1000e_open, .ndo_stop =3D e1000e_close, @@ -7854,6 +7928,7 @@ static const struct net_device_ops e1000e_netdev_ops = =3D { .ndo_hwtstamp_get =3D e1000e_hwtstamp_get, .ndo_hwtstamp_set =3D e1000e_hwtstamp_set, .ndo_bpf =3D e1000_xdp, + .ndo_xdp_xmit =3D e1000_xdp_xmit, }; =20 /** @@ -8064,7 +8139,9 @@ static int e1000_probe(struct pci_dev *pdev, const st= ruct pci_device_id *ent) netdev->max_mtu =3D adapter->max_hw_frame_size - (VLAN_ETH_HLEN + ETH_FCS_LEN); =20 - netdev->xdp_features =3D NETDEV_XDP_ACT_BASIC; + netdev->xdp_features =3D NETDEV_XDP_ACT_BASIC | + NETDEV_XDP_ACT_REDIRECT | + NETDEV_XDP_ACT_NDO_XMIT; =20 if (e1000e_enable_mng_pass_thru(&adapter->hw)) adapter->flags |=3D FLAG_MNG_PT_ENABLED; --=20 2.53.0