From patchwork Fri May 17 07:44:41 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nithin Dabilpuram X-Patchwork-Id: 140163 X-Patchwork-Delegate: jerinj@marvell.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 812CF44048; Fri, 17 May 2024 09:45:11 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id F203B4067C; Fri, 17 May 2024 09:45:05 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by mails.dpdk.org (Postfix) with ESMTP id A11D640649; Fri, 17 May 2024 09:45:03 +0200 (CEST) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 44GLNLaK025405; Fri, 17 May 2024 00:45:02 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s= pfpt0220; bh=Gn3SbWVMcnQasd99Myk9Z705OSN8rsCfdNRvIGpDtzY=; b=YHD /BgtrULASh2FUP1BZqchJmNCF+O29TQXCzUplm+nmD1/33tGTKlP0H4fJYtOkwcQ 4iva8AGudIP0lHOKaOzfK5m8jPHoqLVrnCpDydqS/uUIjIt59odwSMEl1oT4V3Oy 8NHfbCAbvH18C3+aHimiEDagh9gkisaDRxf4dwvlJGMJxVIkofZ6rDg0hOjhEjEI r8lWs6CW+KMqBhzX4blg1j8xVOIPI443Dd2rykUpkdlIXgEbycf9YksN1tQw64Qh yuWCDrbOiw7oGPEWzXHHWANsPS90+h26StPfMQtvTpXLmttnse3Qj/5X6SOyTt8e 0uqfziR4TjzTwmIh9hg== Received: from dc6wp-exch02.marvell.com ([4.21.29.225]) by mx0a-0016f401.pphosted.com (PPS) with ESMTPS id 3y5t0vhc31-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 17 May 2024 00:45:02 -0700 (PDT) Received: from DC6WP-EXCH02.marvell.com (10.76.176.209) by DC6WP-EXCH02.marvell.com (10.76.176.209) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.4; Fri, 17 May 2024 00:45:01 -0700 Received: from maili.marvell.com (10.69.176.80) by DC6WP-EXCH02.marvell.com (10.76.176.209) with Microsoft SMTP Server id 15.2.1544.4 via Frontend Transport; Fri, 17 May 2024 00:45:01 -0700 Received: from hyd1588t430.caveonetworks.com (unknown [10.29.52.204]) by maili.marvell.com (Postfix) with ESMTP id 7D1BA3F7059; Fri, 17 May 2024 00:44:58 -0700 (PDT) From: Nithin Dabilpuram To: Nithin Dabilpuram , Kiran Kumar K , Sunil Kumar Kori , Satha Rao , Harman Kalra CC: , , Rakesh Kudurumalla , Subject: [PATCH 03/10] net/cnxk: fix extbuf handling for multisegment packet Date: Fri, 17 May 2024 13:14:41 +0530 Message-ID: <20240517074448.3146611-3-ndabilpuram@marvell.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20240517074448.3146611-1-ndabilpuram@marvell.com> References: <20240517074448.3146611-1-ndabilpuram@marvell.com> MIME-Version: 1.0 X-Proofpoint-ORIG-GUID: OjRmdkoXywHrIujX0KjzxvM3H9eStHq_ X-Proofpoint-GUID: OjRmdkoXywHrIujX0KjzxvM3H9eStHq_ X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.650,FMLib:17.11.176.26 definitions=2024-05-16_07,2024-05-15_01,2023-05-22_02 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 From: Rakesh Kudurumalla Avoid double free of extbuf when during TX path in multisegmented packet with extbuf as one of segment. Fixes: dd9446991212 ("net/cnxk: add transmit completion handler") Cc: stable@dpdk.org Signed-off-by: Rakesh Kudurumalla --- drivers/net/cnxk/cn9k_tx.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/cnxk/cn9k_tx.h b/drivers/net/cnxk/cn9k_tx.h index 6fc9e4d758..b56881c561 100644 --- a/drivers/net/cnxk/cn9k_tx.h +++ b/drivers/net/cnxk/cn9k_tx.h @@ -117,6 +117,7 @@ cn9k_nix_prefree_seg(struct rte_mbuf *m, struct rte_mbuf **extm, struct cn9k_eth send_hdr->w1.sqe_id = sqe_id & txq->tx_compl.nb_desc_mask; txq->tx_compl.ptr[send_hdr->w1.sqe_id] = m; + m->next = NULL; } return 1; } else {