From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id D7761E6BF01 for ; Fri, 30 Jan 2026 11:45:20 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0F57140E1C; Fri, 30 Jan 2026 12:42:59 +0100 (CET) Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.10]) by mails.dpdk.org (Postfix) with ESMTP id ECD3E40DDC for ; Fri, 30 Jan 2026 12:42:53 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1769773374; x=1801309374; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=pb2TPM7kh1qKwRY4ZIerxHbXkDzfxU5lQBbDTi4127Q=; b=kWIeKanJCrXb7z/hjkZ5rrFLOb9OkPvDsMsqDuKX0iHWhnlu1oK2smC7 TcjpL+mrS7dZarhoweELBKzCULC55ZZiDYERE+liC9ZJQVqyxRrAJrHfR lkk7p6cE1Zfhf1cxcZMNF6AoAybxRF/vhSEZyaG6AJSwyuHt1qYroNd5o +1HPtAtrTS+nyBT3vGJFy7I3ayGA/+OX5nj6DR8qL4wsAoooUd19c2q3W C84tWWfjmRumO0PI2wBCQAxWwJpZpCQmkpn3FCPjNdCZhqqYZqz199dJv NnrF4ZGWdQGNe+CGnK/944UFA16f5KxR0t9C69d7yqtlgR0ogvb8qGxaL A==; X-CSE-ConnectionGUID: veoVtyEDQXSh+TdwVoPTPA== X-CSE-MsgGUID: r5OZqg0/T8m3zmWqSbQQ+Q== X-IronPort-AV: E=McAfee;i="6800,10657,11686"; a="82392332" X-IronPort-AV: E=Sophos;i="6.21,262,1763452800"; d="scan'208";a="82392332" Received: from fmviesa010.fm.intel.com ([10.60.135.150]) by fmvoesa104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Jan 2026 03:42:53 -0800 X-CSE-ConnectionGUID: lRbJU+LtSQumwrUSrz/A6g== X-CSE-MsgGUID: /rr45lu/Qzabw8Lp/j4XwQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.21,262,1763452800"; d="scan'208";a="209190566" Received: from silpixa00401385.ir.intel.com ([10.20.224.226]) by fmviesa010.fm.intel.com with ESMTP; 30 Jan 2026 03:42:52 -0800 From: Bruce Richardson To: dev@dpdk.org Cc: Bruce Richardson , Vladimir Medvedkin , Anatoly Burakov , Jingjing Wu , Praveen Shetty Subject: [PATCH v3 28/36] net/intel: remove index for tracking end of packet Date: Fri, 30 Jan 2026 11:41:55 +0000 Message-ID: <20260130114207.1126032-29-bruce.richardson@intel.com> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20260130114207.1126032-1-bruce.richardson@intel.com> References: <20251219172548.2660777-1-bruce.richardson@intel.com> <20260130114207.1126032-1-bruce.richardson@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org The last_id value in each tx_sw_queue entry was no longer used in the datapath, remove it and its initialization. For the function releasing packets back, rather than relying on "last_id" to identify end of packet, instead check for the next pointer being NULL. Signed-off-by: Bruce Richardson --- drivers/net/intel/common/tx.h | 1 - drivers/net/intel/i40e/i40e_rxtx.c | 8 +++----- drivers/net/intel/iavf/iavf_rxtx.c | 9 ++++----- drivers/net/intel/ice/ice_dcf_ethdev.c | 1 - drivers/net/intel/ice/ice_rxtx.c | 9 ++++----- drivers/net/intel/idpf/idpf_common_rxtx.c | 2 -- drivers/net/intel/ixgbe/ixgbe_rxtx.c | 9 ++++----- 7 files changed, 15 insertions(+), 24 deletions(-) diff --git a/drivers/net/intel/common/tx.h b/drivers/net/intel/common/tx.h index 0d11fcc142..16f8256304 100644 --- a/drivers/net/intel/common/tx.h +++ b/drivers/net/intel/common/tx.h @@ -105,7 +105,6 @@ struct ci_tx_queue; struct ci_tx_entry { struct rte_mbuf *mbuf; /* mbuf associated with TX desc, if any. */ uint16_t next_id; /* Index of next descriptor in ring. */ - uint16_t last_id; /* Index of last scattered descriptor. */ }; /** diff --git a/drivers/net/intel/i40e/i40e_rxtx.c b/drivers/net/intel/i40e/i40e_rxtx.c index 33a0ae332a..792fb5a86a 100644 --- a/drivers/net/intel/i40e/i40e_rxtx.c +++ b/drivers/net/intel/i40e/i40e_rxtx.c @@ -2536,14 +2536,13 @@ i40e_tx_done_cleanup_full(struct ci_tx_queue *txq, pkt_cnt < free_cnt && tx_id != tx_last; i++) { if (swr_ring[tx_id].mbuf != NULL) { - rte_pktmbuf_free_seg(swr_ring[tx_id].mbuf); - swr_ring[tx_id].mbuf = NULL; - /* * last segment in the packet, * increment packet count */ - pkt_cnt += (swr_ring[tx_id].last_id == tx_id); + pkt_cnt += (swr_ring[tx_id].mbuf->next == NULL) ? 1 : 0; + rte_pktmbuf_free_seg(swr_ring[tx_id].mbuf); + swr_ring[tx_id].mbuf = NULL; } tx_id = swr_ring[tx_id].next_id; @@ -2636,7 +2635,6 @@ i40e_reset_tx_queue(struct ci_tx_queue *txq) txd->cmd_type_offset_bsz = rte_cpu_to_le_64(CI_TX_DESC_DTYPE_DESC_DONE); txe[i].mbuf = NULL; - txe[i].last_id = i; txe[prev].next_id = i; prev = i; } diff --git a/drivers/net/intel/iavf/iavf_rxtx.c b/drivers/net/intel/iavf/iavf_rxtx.c index 3fcb8d7b79..cb3b579d20 100644 --- a/drivers/net/intel/iavf/iavf_rxtx.c +++ b/drivers/net/intel/iavf/iavf_rxtx.c @@ -282,7 +282,6 @@ reset_tx_queue(struct ci_tx_queue *txq) txq->ci_tx_ring[i].cmd_type_offset_bsz = rte_cpu_to_le_64(CI_TX_DESC_DTYPE_DESC_DONE); txe[i].mbuf = NULL; - txe[i].last_id = i; txe[prev].next_id = i; prev = i; } @@ -3960,14 +3959,14 @@ iavf_tx_done_cleanup_full(struct ci_tx_queue *txq, while (pkt_cnt < free_cnt) { do { if (swr_ring[tx_id].mbuf != NULL) { - rte_pktmbuf_free_seg(swr_ring[tx_id].mbuf); - swr_ring[tx_id].mbuf = NULL; - /* * last segment in the packet, * increment packet count */ - pkt_cnt += (swr_ring[tx_id].last_id == tx_id); + pkt_cnt += (swr_ring[tx_id].mbuf->next == NULL) ? 1 : 0; + rte_pktmbuf_free_seg(swr_ring[tx_id].mbuf); + swr_ring[tx_id].mbuf = NULL; + } tx_id = swr_ring[tx_id].next_id; diff --git a/drivers/net/intel/ice/ice_dcf_ethdev.c b/drivers/net/intel/ice/ice_dcf_ethdev.c index 02a23629d6..abd7875e7b 100644 --- a/drivers/net/intel/ice/ice_dcf_ethdev.c +++ b/drivers/net/intel/ice/ice_dcf_ethdev.c @@ -408,7 +408,6 @@ reset_tx_queue(struct ci_tx_queue *txq) txq->ci_tx_ring[i].cmd_type_offset_bsz = rte_cpu_to_le_64(CI_TX_DESC_DTYPE_DESC_DONE); txe[i].mbuf = NULL; - txe[i].last_id = i; txe[prev].next_id = i; prev = i; } diff --git a/drivers/net/intel/ice/ice_rxtx.c b/drivers/net/intel/ice/ice_rxtx.c index 14c4683ad7..b79e195b5c 100644 --- a/drivers/net/intel/ice/ice_rxtx.c +++ b/drivers/net/intel/ice/ice_rxtx.c @@ -1121,7 +1121,6 @@ ice_reset_tx_queue(struct ci_tx_queue *txq) txd->cmd_type_offset_bsz = rte_cpu_to_le_64(CI_TX_DESC_DTYPE_DESC_DONE); txe[i].mbuf = NULL; - txe[i].last_id = i; txe[prev].next_id = i; prev = i; } @@ -3201,14 +3200,14 @@ ice_tx_done_cleanup_full(struct ci_tx_queue *txq, pkt_cnt < free_cnt && tx_id != tx_last; i++) { if (swr_ring[tx_id].mbuf != NULL) { - rte_pktmbuf_free_seg(swr_ring[tx_id].mbuf); - swr_ring[tx_id].mbuf = NULL; - /* * last segment in the packet, * increment packet count */ - pkt_cnt += (swr_ring[tx_id].last_id == tx_id); + pkt_cnt += swr_ring[tx_id].mbuf->next == NULL ? 1 : 0; + rte_pktmbuf_free_seg(swr_ring[tx_id].mbuf); + swr_ring[tx_id].mbuf = NULL; + } tx_id = swr_ring[tx_id].next_id; diff --git a/drivers/net/intel/idpf/idpf_common_rxtx.c b/drivers/net/intel/idpf/idpf_common_rxtx.c index 95f2e1deea..bd77113551 100644 --- a/drivers/net/intel/idpf/idpf_common_rxtx.c +++ b/drivers/net/intel/idpf/idpf_common_rxtx.c @@ -218,7 +218,6 @@ idpf_qc_split_tx_descq_reset(struct ci_tx_queue *txq) prev = (uint16_t)(txq->sw_nb_desc - 1); for (i = 0; i < txq->sw_nb_desc; i++) { txe[i].mbuf = NULL; - txe[i].last_id = i; txe[prev].next_id = i; prev = i; } @@ -277,7 +276,6 @@ idpf_qc_single_tx_queue_reset(struct ci_tx_queue *txq) txq->ci_tx_ring[i].cmd_type_offset_bsz = rte_cpu_to_le_64(CI_TX_DESC_DTYPE_DESC_DONE); txe[i].mbuf = NULL; - txe[i].last_id = i; txe[prev].next_id = i; prev = i; } diff --git a/drivers/net/intel/ixgbe/ixgbe_rxtx.c b/drivers/net/intel/ixgbe/ixgbe_rxtx.c index ea609d926a..dc9fda8e21 100644 --- a/drivers/net/intel/ixgbe/ixgbe_rxtx.c +++ b/drivers/net/intel/ixgbe/ixgbe_rxtx.c @@ -2407,14 +2407,14 @@ ixgbe_tx_done_cleanup_full(struct ci_tx_queue *txq, uint32_t free_cnt) pkt_cnt < free_cnt && tx_id != tx_last; i++) { if (swr_ring[tx_id].mbuf != NULL) { - rte_pktmbuf_free_seg(swr_ring[tx_id].mbuf); - swr_ring[tx_id].mbuf = NULL; - /* * last segment in the packet, * increment packet count */ - pkt_cnt += (swr_ring[tx_id].last_id == tx_id); + pkt_cnt += swr_ring[tx_id].mbuf->next == NULL ? 1 : 0; + rte_pktmbuf_free_seg(swr_ring[tx_id].mbuf); + swr_ring[tx_id].mbuf = NULL; + } tx_id = swr_ring[tx_id].next_id; @@ -2535,7 +2535,6 @@ ixgbe_reset_tx_queue(struct ci_tx_queue *txq) txd->wb.status = rte_cpu_to_le_32(IXGBE_TXD_STAT_DD); txe[i].mbuf = NULL; - txe[i].last_id = i; txe[prev].next_id = i; prev = i; } -- 2.51.0