From patchwork Fri Feb 3 07:57:34 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 123001 X-Patchwork-Delegate: ferruh.yigit@amd.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 1370F41BB9; Fri, 3 Feb 2023 08:58:07 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D8328410D7; Fri, 3 Feb 2023 08:58:02 +0100 (CET) Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam04on2112.outbound.protection.outlook.com [40.107.100.112]) by mails.dpdk.org (Postfix) with ESMTP id 1D5A64014F for ; Fri, 3 Feb 2023 08:58:01 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=oFLqvAv6dO9RD5zZh7hnBAqSxVb29snWs7cUQe+SJWPXmFShYSslaRDGOMnjpauxhTDTXgoJVsP3OmnguMfg/KrLPWv9HMwy5x9wnBexTQuQA/zY2v/0asPQFXw0THahUBwkrCyjMn2iR+3wjhoLzQtSDSx1pEsfzb94eYujwCv8zUKpfyl1pmG5cm60LDvFZKbHt4gFVPFeZbtnLULyWMzhALuF1Ov5Hm0cJONT9C6QQ0Fhc8JpfwOC8jlFx7Qp9ETxtTVewffxK5dSx+4bFRYklsX7Strb1iIU1YuPApM1umGJGmpzugwlkMZwRxfUcu+qahmcI6HSPA7m7JD3fg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=b8j3wqzu7kxS5gZkqLqq7q0RIiVMfwB/asQbtfd5jkI=; b=afOfNElmJFPOpzThAKuPrqk+ingX7/e29LKHEWb1mJqUG+1zpENE1CBZYXp5+y8iSiQEFoKixkAD23XTlcX+dMfsCQvck1BFyo7jxP/sX39fily1FU0FuJ4eoRj9M31NYN+RhsLSR9dG6W+dtJxr5eODiEMvosOT7eMzxq5zJXiXOrnzgMwjznJkrPY3RQ4HcTFI5crJhP1M0/JLb5ndzEez/ib/Jaypbk5g0fQq7HZkUCJOxarWcjD/83RHkE/vVzHUmCYMso7UOH4UGoILYuxQu4uV3qooHlKr21Mc5oCVDMVxKOUIIjhHrnHbG8Zg806muD4SkkEEyCTf0VMWXA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=b8j3wqzu7kxS5gZkqLqq7q0RIiVMfwB/asQbtfd5jkI=; b=m89htRMtXFeiXiU8taOQ32t/jcY7nNCbq7XF81wFNKcjqdmkUWQb49lwNB1n6uC5aDpt15tOPnPJCKh2FsdA/AvaI4fmP/14VKjP5l1cXCvKiPHNchMZf5ksPv3c2s6CrpDdcH4k9rHeEmPFyMLh/0kaAiLXSp/P6x3T08K3MRE= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by PH8PR13MB6292.namprd13.prod.outlook.com (2603:10b6:510:257::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6064.27; Fri, 3 Feb 2023 07:57:59 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::7ea4:e421:5dd5:9f4d]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::7ea4:e421:5dd5:9f4d%4]) with mapi id 15.20.6064.027; Fri, 3 Feb 2023 07:57:59 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, James Hershaw , Chaoyong He Subject: [PATCH 1/3] net/nfp: remove panic usage during reconfig Date: Fri, 3 Feb 2023 15:57:34 +0800 Message-Id: <20230203075736.18616-2-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.29.3 In-Reply-To: <20230203075736.18616-1-chaoyong.he@corigine.com> References: <20230203075736.18616-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SI2PR01CA0014.apcprd01.prod.exchangelabs.com (2603:1096:4:191::17) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|PH8PR13MB6292:EE_ X-MS-Office365-Filtering-Correlation-Id: 9044f897-351a-4c5c-4ac6-08db05bc5e31 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 0fzOHcLZt/M2y/q1m33f5VfpglINnCJoiAM4Mm+IfzwNevzlvpJro7p4Bh30qSyAXBfSFIxKRc4hW2UBxN74zmHg/cpnDmYzZobn5aKZpa+aI+RfT2927h9/Du/GB+ytYHLA/+YkuwSOgYiqJegk308Z+98Mt/+ozbfPYzdFAsCkN6ClnAlgR5HWN7dqKIng8FdUwlX7jSJj1Q+6GxB88ynVJviiv3W/lo4tOz+v3sxEN02KZZ40v26PYJgsQsDG65slodIdEsidMxJfWuOMFp+4pyLNXfe6SDvCGmCMEl8EE/2lDBz836pdr+PHU12l+sCybqwvK1gz2M+bTkrEMuQmLLr6bFNSKrw6BtkZbwHCyEtM5la8Exmq8tj+UiwBybBsBBShtcc+ukjtdzwakYorZ5mEIpXy9cqEUURZXu9jXUSaflCXHUYrrjwi2zme+h+H7b5T+9Qj5AtsqU2CeI8yXtAtXKR1NLc/09gTHJmSehEO952XPmhE8zWn6+q36h6ZzoX6WXwwGYulIfufGsKvBBwVKt7RGyexnwfbLYI+WetuIF6DKoMmuyqt12VbPqUjMQauZhSs213vWY9b6CcvInzawCie95eEgSfutZ8f1aYrkdQ4WD8rbq1f22yCoC51NHdiKGrh3AHao7ucw3sALEUNWcjh6/9AXckHbGDd//Mr/jIkg8Bo8TpNI9LmsL7Xn9KXz9my6Khvc22m5dFlZomky5/cROXv4l485ceZTbmzMYs6aApYZEFQo99g X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230025)(4636009)(346002)(366004)(39840400004)(136003)(376002)(396003)(451199018)(26005)(186003)(86362001)(6512007)(6506007)(1076003)(107886003)(6666004)(478600001)(6486002)(38100700002)(38350700002)(52116002)(2616005)(36756003)(2906002)(8936002)(5660300002)(44832011)(41300700001)(83380400001)(66556008)(66946007)(4326008)(66476007)(6916009)(8676002)(54906003)(316002)(66574015); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?lFZOB6CKWsOEirsYIyv2bYnCkOPa?= =?utf-8?q?pd/40DA027DzmNnJGXdW74S0YMus+H7ydTu/wD2Ul2rL72bpSJ4TwCTg5vSxxq5NR?= =?utf-8?q?nrrju84YIQ4h27JLr1xOO5Lc/yDN5+3qZxDqrU596dVwsZisNyA94D2hZlEkwQ4Qp?= =?utf-8?q?ycFBZJVLUr78UfU1xtBuTapT10fO7+2HJdauszRU85QvD203dPQEJ7PeDuXhwsYZn?= =?utf-8?q?7CZ/OMoyl7KEVZ06BtLXj3LQo3GfBgDvhltykbfQPKQ7smp9mFgQym5JTPWpO3XsT?= =?utf-8?q?pJuPkhwCis/69Tvt8sRUxjNgBGOb9SqrrlbSMvT5tRWdNimVKUlSBeZt+KDvkZG76?= =?utf-8?q?zST2XvIIciFeRQThWHltPHPY4/ApHSik7KRbVNtKAfvUpRBm/83LJtSEEaXQImFjS?= =?utf-8?q?dIvt29eOMJrD3YjSBrMkBiApUmbZVaMoYvg5RfLTWgJNmfUn1U5ZiOGAdxdJHh7yG?= =?utf-8?q?JgHRAoMOPlbKH+7BJmujjybUk4PWM2363InSuWjLwrZyIedVgGAduDo4cfcPubPyA?= =?utf-8?q?Q/6eAWEkIwZPJnoZzIXluVIPHxLNaO/dCPU9A8nItcUTuMQmisOXoOTB+tCzsTMi9?= =?utf-8?q?1fORl2HKXQUMYOSB6GnyioRV2fhuC9iZ5kyYEG7UASXp4V6EG60Slw8+RwTGy+k/z?= =?utf-8?q?JOCvZO2Cn8InoSQXd4nAFP3LkpYxwnMU+K0sFztLkpxymkfe0aQHXYqjK6E461yCw?= =?utf-8?q?VczkXxiPBLCIFiZJgKg6oBnRoXlXRkR4e9dTIAuOztBuKhAl9TMvbQ3g+5Cd9S2CH?= =?utf-8?q?KXSvSate+Zn1cbBTtbXkHMkDKHZi4f6pKIMm/Soy0TJ4iyXXQ/w4jX+V4rTZi0Ugg?= =?utf-8?q?/UUF2IudmfrAZ2aqOrUzx517JJUJNf06Of3K5w+Jw2Mend1xTeWmAjBRKe0qhEUrm?= =?utf-8?q?yYYAdv5Y3HC3XAoWhelMZKl3MxJdgfYO3mQxwgpOj6f7j8xTKp+Cki6SMw0UBht52?= =?utf-8?q?j377KK1GOIqJEmYpL9f5oGyt5CiHMwLYkJPSTrhf2mgWmaS43pjDwddmfytKLRs9A?= =?utf-8?q?cZTzFWJvsNWF02WJKUgEIPNSXEAhUzyXVOyvlP5JBE1ACkf6OrfrOgwrIQtS/k1P2?= =?utf-8?q?Y+xplzlWYibCsgn5MDuPpKXbUPfYXrhMnR2EzQ6kmophxZFl7wRezooJZ0peh8frd?= =?utf-8?q?mc4vqckPFIofjhCaAj20EzmmMcVa89CTdjnCALFgJT3qi7TTCPfD5cRrSA5SKga9T?= =?utf-8?q?HbAjvAZ++AH1OeEf5xBk2N6Von2AI/c784p6VOzMYTIwq40ZRpK/+yWbNC+NJ9ywG?= =?utf-8?q?3nAiwhm3Xu3i0Qc8vmj8JCPK61Tj2BXLV3zwehFem1XrC+b0svMMDXyJ2+CTVuE7e?= =?utf-8?q?H6/cEp98IQVXt8IRfhcAk3HoH6yl1XUu92YGvi5QQj+knD2s2VrAJ8mfmWULHT3US?= =?utf-8?q?JC6VsjV73TcrqjZrsm7xlupdLnTVBN8z/ekdzALtY6bb9TypAPMF29bJxC+ZyRhjf?= =?utf-8?q?oqKvPmuSDQIcBUQxu5tMynK/0zv4aZ7WaT2l4ET1oADP1bH+av/PLUllyqZCAfq/p?= =?utf-8?q?XVJUZXkimFB53iwZZAyw2yYCqnLcnIyETA=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9044f897-351a-4c5c-4ac6-08db05bc5e31 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Feb 2023 07:57:59.6898 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: /HdOXVybSF7JrPAxNWSczWDjXTWu3obeXn7mD9a7eEIo+7Xa5ZerZei7vhUZh1fvlx8KXOq3vHmr6hznKKlAggt4TGYSp8vD1zi4Rwz2GlI= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR13MB6292 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: James Hershaw Remove usage of the rte_panic() function from the nfp PMD. It is better to return error values and allow the application to handle the error. Signed-off-by: James Hershaw Reviewed-by: Niklas Söderlund Reviewed-by: Chaoyong He --- drivers/net/nfp/nfp_common.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/net/nfp/nfp_common.c b/drivers/net/nfp/nfp_common.c index 4f21d9978d..7a60add3a6 100644 --- a/drivers/net/nfp/nfp_common.c +++ b/drivers/net/nfp/nfp_common.c @@ -63,8 +63,10 @@ __nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t update) PMD_DRV_LOG(DEBUG, "Writing to the configuration queue (%p)...", hw->qcp_cfg); - if (hw->qcp_cfg == NULL) - rte_panic("Bad configuration queue pointer\n"); + if (hw->qcp_cfg == NULL) { + PMD_INIT_LOG(ERR, "Bad configuration queue pointer"); + return -ENXIO; + } nfp_qcp_ptr_add(hw->qcp_cfg, NFP_QCP_WRITE_PTR, 1); @@ -85,7 +87,7 @@ __nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t update) if (cnt >= NFP_NET_POLL_TIMEOUT) { PMD_INIT_LOG(ERR, "Reconfig timeout for 0x%08x after" " %dms", update, cnt); - rte_panic("Exiting\n"); + return -EIO; } nanosleep(&wait, 0); /* waiting for a 1ms */ } @@ -121,16 +123,14 @@ nfp_net_reconfig(struct nfp_net_hw *hw, uint32_t ctrl, uint32_t update) rte_spinlock_unlock(&hw->reconfig_lock); - if (!err) - return 0; + if (err != 0) { + PMD_INIT_LOG(ERR, "Error nfp_net reconfig for ctrl: %x update: %x", + ctrl, update); + return -EIO; + } + + return 0; - /* - * Reconfig errors imply situations where they can be handled. - * Otherwise, rte_panic is called inside __nfp_net_reconfig - */ - PMD_INIT_LOG(ERR, "Error nfp_net reconfig for ctrl: %x update: %x", - ctrl, update); - return -EIO; } /* From patchwork Fri Feb 3 07:57:35 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 123002 X-Patchwork-Delegate: ferruh.yigit@amd.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 2583641BB9; Fri, 3 Feb 2023 08:58:15 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 4257C42BFE; Fri, 3 Feb 2023 08:58:04 +0100 (CET) Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam04on2104.outbound.protection.outlook.com [40.107.100.104]) by mails.dpdk.org (Postfix) with ESMTP id 4D62A41140 for ; Fri, 3 Feb 2023 08:58:03 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jfc6GJ1WGVFDTLbvjelffISNuWaGNMksMHY3CkeVN3eYmgJnS+hwx9WJbOfqPPEJdZkyzhexUqRI3AffbI+TeN0iiC0DauSKiD1aRCMOVBk2D6wLfUMQieUDFkW9eNByOZz4owSk0+Kp2hkSK6N0eP5tYueE9W3h0l+ucXcp1W1XePsF23ND4l8+l//kcwlHQhbtWg7TJScg0pbZBqqV1+wuEhcugheqk2jKdQ6gzXnUKGPImHCl+z2dhKsaEmuUy9sjas3nsmwriSWnpoSVLLMVvrOgPfn6pPv7GUlop4FDxYCOsKARIBseDfOwxHEiXCT1R/svERvlqhaNLbUr4w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=jQLbovZoLx+AWwoqMkZxe+q+6k/RpFNxc0/lp/dID2I=; b=Q39dfhnYGo1eTzzOWzDIw2OHofyiDm0uomjE2cmw2jR3UCsK3F1SGiOZXb7ym2BHJG2Y2bqp7jCxm7k0c2wx09ijmmDE3M/GZtp/VVZEhnCTUl74AkYpBL3/Ja4zhk1yZwujDceJK7dnFeSRGZb51Un7CQyPtX9Mns0bHlebAzRMj/TkLAKFwb5MiIYqVPdB3mhxJHDOUdmYPo0MHesdSX+/qt4Qt7kA11/wdFWbAak8VtqLjOcD541wztqdvVVAF5TmQkjJ4C0a3Lldj6WSA68PLRBwfgnAk+/0gR5AUPexu1UXqi/SEZ63Nti3ZkdToZMv5x6muFpL8PlycNysHA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=jQLbovZoLx+AWwoqMkZxe+q+6k/RpFNxc0/lp/dID2I=; b=ZqeQwf3ccYVRNGSLvkh1EKhYoy0FlxImxHps2Maq+VPFkdsf69k2IBRbKYv0AVVZhXAhc5ylxJzp/QxSMf7tFs0bI10Zrr38Q+nb33RlyrrXy6DEkpC4p2wafrSGwIbtHh679XB0sUktMi/YY0QZh+/n19F4NOROwhR2HtkNg44= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by PH8PR13MB6292.namprd13.prod.outlook.com (2603:10b6:510:257::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6064.27; Fri, 3 Feb 2023 07:58:02 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::7ea4:e421:5dd5:9f4d]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::7ea4:e421:5dd5:9f4d%4]) with mapi id 15.20.6064.027; Fri, 3 Feb 2023 07:58:01 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, James Hershaw , Chaoyong He Subject: [PATCH 2/3] net/nfp: remove panic usage for multisegment Tx Date: Fri, 3 Feb 2023 15:57:35 +0800 Message-Id: <20230203075736.18616-3-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.29.3 In-Reply-To: <20230203075736.18616-1-chaoyong.he@corigine.com> References: <20230203075736.18616-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SI2PR01CA0014.apcprd01.prod.exchangelabs.com (2603:1096:4:191::17) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|PH8PR13MB6292:EE_ X-MS-Office365-Filtering-Correlation-Id: 1305119b-b8e5-4879-5eb0-08db05bc5f75 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: DIs1uyEt3t4HJE0YggxwmwDHyjUfQXi7fLyFmdSOOr3AZegJP/2ccjwSSuqJKQ3EnXqXGsjZC74zKhautI/1AnxC3Hxju4o1dYKdqZp2BZr0hwwgYC6ELuiZT5g96JdS+fKikfg/a72Yb7MuaQc/opA/4/ATvoMB8AiOTbNZ0mYrqRnDsXSRKg+m0HRyEIv1SU/zSuShVJv3ib3aATmOUz64IXYZi2a14t09kKlOHeJWmfVse5Yp8eYE5MN4kjb3wgEn79JlDVbB9nQQXQfsRLYh63ejNpKTq/4cSyMJlSchHnGIt+/D27JNk/pjw7ls+oycVVvkZVfLDVSX/v8HVJkJl7p7xD2J2tHjRP3+dTJuwakPO2ZuS4wWYatHRom4Wo3GIIwCFm3iFxfBEI/Y7IxoriXVDWK8+y3MPddLiDbJEHBO0YcJTrNIMXuE+v2L3bjMEIP0+qSHnBSjrJR8zNcvUs9sFbkiUEU+l8iqNNjgeqNGCEuG3dDXX2rlYCwCVFkTd8NGwZ1HeMzrtkode/4yGF8nAw2BbPFD0/jyiHp222chJImYgeBzoSIg1arTltPnFoRa3nj2Ig6LBt5yv72ScCyJgKSF31t3/DW2IdBgann5LC7oRhT4tX1s6o6jF3c9sBj3D0OkVx5loToE3RrpsKlbuLdZFU5j1osRxwyqy8gIe/D6Xd3vvRQcH8Lrm0IHN123KG1FXUMceT6I3fwJ/2yrbqcdKf8i+g7V8UeAS/8mQoaly58PnLEHeqAz X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230025)(4636009)(346002)(366004)(39840400004)(136003)(376002)(396003)(451199018)(26005)(186003)(86362001)(6512007)(6506007)(1076003)(107886003)(6666004)(478600001)(6486002)(38100700002)(38350700002)(52116002)(2616005)(36756003)(2906002)(8936002)(5660300002)(44832011)(41300700001)(83380400001)(66556008)(66946007)(4326008)(66476007)(6916009)(8676002)(54906003)(316002)(66574015); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?7LnqymzPqroO7aKfwbz5RJnitojG?= =?utf-8?q?arOkFc7eh0IFrXt+R9AZ9gtrVr8Y3Wl2CSSkQcF1rwKeL/Z+TnxxzcWzT7oKyzoT/?= =?utf-8?q?8sBc2Kbzgvqn5+dcSqfqwnuXifYkss9VI3rYinP9VFjdcgRsSqbAndtisuDUCQB57?= =?utf-8?q?e7aAGi0ekN5yIczJ2KZ3q0+wZ7Ib9H7dQrnWfuoxHzSDepewMmII7MXenKq1/1Ct+?= =?utf-8?q?Kj3CIT3OJg4w1uIHbYYOc2rSZLwNwSyURzlhTiC2dp5k5ZgmQpSNVEyAbypEbHzi0?= =?utf-8?q?yiy199uornrfeRwq8JvYELyE7hIACmgs2YN7N0iyF2ZfUtTutjLBYKuQBS3e/h5gS?= =?utf-8?q?TCbg0bznA53jHtZ15eMGroYGoepddEQ3Mnqx79W7jU1Lxnp5VReR7ODwAcSDFtsoH?= =?utf-8?q?BEdyyzFg8lFh6aAkDjpyvuhCxqlITDmjNpMtdyip9CbflQoYWeh6yPhnanlAE8YPC?= =?utf-8?q?98v0gX68oj3xIi4pT3dNgpKsMAPBvxQUKsauRhPXVvMJGz22W4rO5+qfLyVhWAiBm?= =?utf-8?q?T7EJq2wPmFIP5xLDyuhTZ4iMIBeQV6rF5paeuriDkJGIkrLOVQy9XQteqGd5Om3yy?= =?utf-8?q?IGwbep9bTp6yBfm13S7w6X/63anKxErJ89lt/W95doyr6U12p6HI1Kf+9Gwnawooo?= =?utf-8?q?NvkoLuUjiSbVfuMdE8j+UJGdqVGObeY2J5NyN3g1gQhIW9/NdxsHiyXGVjnE3jH3n?= =?utf-8?q?rJxmKFsxRZ+5wAjvPGDSny+vE8mBWLoRChqkE+eZbAjiUoIkh9Guiy9lRY/VLXHt0?= =?utf-8?q?f/QO79M1ZJPF9jeo67MJcrUfnXLyexOI2eG7ttJK0w4WduhuGQ/N1qd3CJLfYAQlC?= =?utf-8?q?tsk0RskEsCA6ks/YL0+wnVuexQkJc+L7AMbMStCt6/3ZgYI4HAPWdL9eYHf8wJd5y?= =?utf-8?q?IfyF5F0XOvFjk7oyTXFiKUEw6l3mAV20A6QblvwwkXWk6YktaUcHtZFk1i+aFm65t?= =?utf-8?q?4y0e0aVFVeLfwsrUaRStEgF/Yg+5b5BA4cXU2pvw072YkL4kHRbYta33WReIEdtk4?= =?utf-8?q?pZV/LspNoo4q0A8UeR5ZtuNwtDyQrJGUlgPldWayP/HTF2w138/W3b3Ny9L3EXMmP?= =?utf-8?q?CM1uQ6eWTuClM/rnargNn/mu6upuZ5iXdBabURfHcwkwsR07N2oBO6N2LslCNagPY?= =?utf-8?q?kWxS1AKlHhRsEXCzvFxZw4OCTxuz/GSakS2PMzLRk2vdOmcJ9SCodlzffZXQJJQUM?= =?utf-8?q?afV6Sq6n8uQdpFuyoZ6g8TJXMgp6MfC4HZ9EISYqAWGhQQ6EEqLcuQb4LLxXs28La?= =?utf-8?q?GC6TCrfoh9ALr96whQQfIpYrQTLd43LOJ87a2ewqDBiYyUbWot0Mi2MLOb1UVmOXC?= =?utf-8?q?BR4qeY4NhS7GDuh9NJl5ifVkMCxh/3u40z1tiYZaGKsdME+n+C6FktyIWJDhJct77?= =?utf-8?q?vBJpIaRd898STfkAymxEBhT+BwvSn9COkAftqoX2Bsw0j/FJ7aL+lc3Pi13yIqP/f?= =?utf-8?q?68/Sd5wih+VSC6TCC885nx+ShZuEOsg5pcepUctPGunOwLwaVUH4v78mzy/EbE/ox?= =?utf-8?q?NK2yfSUUqZbeL5QdQ6Fga7hmEwTF8jqMrQ=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1305119b-b8e5-4879-5eb0-08db05bc5f75 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Feb 2023 07:58:01.8606 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: +OPWlfGCJBuvMmnCiRyiGRcik9PT2MwqlimfNRZVpQHL9COzHN6I33dvyrsMPzJd4AgQI60YMsnRDcqqjb8XKfZjguM/0Yql5YOyn1tmySU= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR13MB6292 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: James Hershaw Remove usage of the rte_panic() function from the nfp PMD. Update the nfp_net_nfd3_xmit_pkts() function to match the functionality of the nfp_net_nfdk_xmit_pkts(), rather than use rte_panic() Update the logging within both the nfp_net_nfd3_xmit_pkts() and nfp_net_nfdk_xmit_pkts() functions to only make use of a single log when encountering a transmission with multiple segments when NFP_NET_CFG_CTRL_GATHER is not set, and to change the log-level to ERR. Signed-off-by: James Hershaw Reviewed-by: Niklas Söderlund Reviewed-by: Chaoyong He --- drivers/net/nfp/flower/nfp_flower.c | 3 +-- drivers/net/nfp/nfp_rxtx.c | 7 +++---- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/net/nfp/flower/nfp_flower.c b/drivers/net/nfp/flower/nfp_flower.c index 7b46dc0f6a..e0e94e0a36 100644 --- a/drivers/net/nfp/flower/nfp_flower.c +++ b/drivers/net/nfp/flower/nfp_flower.c @@ -532,8 +532,7 @@ nfp_flower_pf_xmit_pkts(void *tx_queue, if (unlikely(pkt->nb_segs > 1 && !(hw->cap & NFP_NET_CFG_CTRL_GATHER))) { - PMD_INIT_LOG(INFO, "NFP_NET_CFG_CTRL_GATHER not set"); - PMD_INIT_LOG(INFO, "Multisegment packet unsupported"); + PMD_INIT_LOG(ERR, "Multisegment packet not supported"); goto xmit_end; } diff --git a/drivers/net/nfp/nfp_rxtx.c b/drivers/net/nfp/nfp_rxtx.c index 4a7574fd65..5f23128ac9 100644 --- a/drivers/net/nfp/nfp_rxtx.c +++ b/drivers/net/nfp/nfp_rxtx.c @@ -986,8 +986,8 @@ nfp_net_nfd3_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pk if (unlikely(pkt->nb_segs > 1 && !(hw->cap & NFP_NET_CFG_CTRL_GATHER))) { - PMD_INIT_LOG(INFO, "NFP_NET_CFG_CTRL_GATHER not set"); - rte_panic("Multisegment packet unsupported\n"); + PMD_INIT_LOG(ERR, "Multisegment packet not supported"); + goto xmit_end; } /* Checking if we have enough descriptors */ @@ -1466,8 +1466,7 @@ nfp_net_nfdk_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pk if (unlikely(pkt->nb_segs > 1 && !(hw->cap & NFP_NET_CFG_CTRL_GATHER))) { - PMD_INIT_LOG(INFO, "NFP_NET_CFG_CTRL_GATHER not set"); - PMD_INIT_LOG(INFO, "Multisegment packet unsupported"); + PMD_INIT_LOG(ERR, "Multisegment packet not supported"); goto xmit_end; } From patchwork Fri Feb 3 07:57:36 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 123003 X-Patchwork-Delegate: ferruh.yigit@amd.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 5050441BB9; Fri, 3 Feb 2023 08:58:21 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 5066E41611; Fri, 3 Feb 2023 08:58:07 +0100 (CET) Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam04on2124.outbound.protection.outlook.com [40.107.100.124]) by mails.dpdk.org (Postfix) with ESMTP id B0B7842D0D for ; Fri, 3 Feb 2023 08:58:05 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=UKZfjeebhSPB3ricA0ZOkTwqUORfHgsYsWI4Zsa8cqrNQUOt6uYLh7L5V2NdCDcXSaOmICgQw91rcREnSHkmWEnwD4xduIYfd+cBuBKEWBh30xcnEzvTEeW6fhR6eaxaniSzwTkOyko+x5/OXWt10s/Ifu3iYYVhp/HPQyG9DrNc/PpreQrE8wiFBMqCVBALKBbgmiS1KwUJLewo6jP2MdGe4wPk3AWiWDLL1+mpn0ld+xaRJCCVop6yGC7JDFaXkbLlhh+RoHieacT4xG0rA/UqV+CqQRr4EvqzrA34e5RSwchvYYm5KxCZdlHMsxZTwzMjxlNCxChf44LDqKKDIA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=WYX7J5v+kYPMpV/x3NNfLGY7lubWXg+NaQc8+ID2azM=; b=JCKSrKJhtVu8x0DahyE3mzo1XHcD6YSFLsUyH5hkJ6Y1bu+xqd0Oj7GIBpxBQp7vIseeYYB4d9+9I1FBfNIlMae7JBD2Sq7blysUKdOe4nSs/74zjUOmtHDYGDdAAfVaVTDYi4ePnHt6/9ixK2Q1NcSacPILRe7pSHUPkUAYV6m25u4+kfte9Se5IUxelDf1i7qZnLZQDXZG0KHS49NAkLYYCHhaRxPpliqExICn3m9Nbhb77XAyvzqe+I4E87cBt10Mi7VHwjwZ0R7yJJFlQgc0iFXfYjj7GTsOOioF0LTPRiXXr3TlyTbCGHFmAEtBnSBIPhnQ712xuzEZD9lTfQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=WYX7J5v+kYPMpV/x3NNfLGY7lubWXg+NaQc8+ID2azM=; b=ZC6ATfMgQg/aem/9ZlXxY5UAnzcnsVh1wuxK3/03bz8wa/sxvc1ri9FWtprQalfbiJXAz7ud9hg8EViClBio0AScekt/RnrZS5IoWdhLKLF4msmovVJ7/f7UbtVdYW8wfomvCx6Djk1zgL9NQ9+3lTHRm7nSyyGu+FeDpO0Hc+c= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by PH8PR13MB6292.namprd13.prod.outlook.com (2603:10b6:510:257::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6064.27; Fri, 3 Feb 2023 07:58:04 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::7ea4:e421:5dd5:9f4d]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::7ea4:e421:5dd5:9f4d%4]) with mapi id 15.20.6064.027; Fri, 3 Feb 2023 07:58:04 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, James Hershaw , Chaoyong He Subject: [PATCH 3/3] net/nfp: remove ASSERT() macro Date: Fri, 3 Feb 2023 15:57:36 +0800 Message-Id: <20230203075736.18616-4-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.29.3 In-Reply-To: <20230203075736.18616-1-chaoyong.he@corigine.com> References: <20230203075736.18616-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SI2PR01CA0014.apcprd01.prod.exchangelabs.com (2603:1096:4:191::17) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|PH8PR13MB6292:EE_ X-MS-Office365-Filtering-Correlation-Id: b29dfd03-f725-4134-ab46-08db05bc60c5 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: N68WohMb5xHmzGalor4RrwrLSba4w3L703wRqiqxS4Q+Y/FPasP1cDbHHVPXvVLIhpR+bYyRWzuvqImMzMSEepx1B1C0zEaGBUuFb0G2cG4ypsJn6dhdfOjxZd38uwMlwBpFied7uTJgx8K539nbGHDsONnPTLzIVxHUPN92PUNGb76kvTFPbdyPcft/1BL3Y93+Z/DqYehLaKOuUYdOTzBaL2yTL+bqUVtmabWNxByYZYoJLHV1i8VL4qkShp2zJHZqT/EpLKRdEbMmhddmRr4s/Z0Vl123GWgqYisRubS0AFgPeGsE18TwT3KyPgRfMLTew05b4JLyMdaeLbq4OSuBcmEDDoBMNBwwt/KvrUnA/XQrWPyqKH4O+6MfHBWL4ajckiOlHYXXQCjgljJZadgSLfVUtnCjG3S7SEKYeSQ9B53gJ7/OWVOfjkdMNw9ki2OT1u2/lrvl6grcaPwPeK/D6Bv2RBGbRuS+Yc6OeR2+ypepqTEdGQ+oE26UYctObd7V0P1x/HAZWURuoDEX+uciUxCIwTh6BWEQwNDVqrRRIVn35ACHoPYcQ7hofo9EUvmRRcm+9KlpCIRdQRaRu2JyMD7y8AuMa5LI7YgdgyCmibUrLfI0tvf2vl8d6MxRXYrc1E7sm9ONZFLYPbBfNqPab0vzNq+ALt82vOtdBZz49KYk48uzzZKOJrBTYeM5gLBolhBfEr/f59eiTrw5IQPE+OYcKN9Ahf7acrjtnz/rNNAbD9qzBsS2SRarbyCE X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230025)(4636009)(346002)(366004)(39840400004)(136003)(376002)(396003)(451199018)(26005)(186003)(86362001)(6512007)(6506007)(1076003)(107886003)(6666004)(478600001)(6486002)(38100700002)(38350700002)(52116002)(2616005)(36756003)(2906002)(8936002)(5660300002)(44832011)(41300700001)(83380400001)(66556008)(66946007)(4326008)(66476007)(6916009)(8676002)(54906003)(316002); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?Gl8fvd5RSbC8ttUu9QGN/j6y9Td8?= =?utf-8?q?T0JkJqlgWK8cIkZejoky8OiMNMGpVggvY8/1X+IdIbmVl4HfxQ4eda0X4ZS6GqPPp?= =?utf-8?q?2H+GzZMAtGWjHWK+BThqqjO2/DYdrcl1lVfnCFcOSbz1o7DEuN6GVjZrzNU9Fw9cB?= =?utf-8?q?7frSVSmXHsgGcS8gq3o7vVAaMpExS/k0y1knT5X2O43OpH/4xVICB6L0P5Xq9BolN?= =?utf-8?q?bracKGu9IqvY2dEWd/GqHw+IPfeTxJClDXyCQK9qkNdh4+j/oWd+lDNadKwfwJbFl?= =?utf-8?q?bGsZqCD7dvKaQ2cRUxYoD6OReDvOF011HcVwOdSpK9ya9/ndmwsGUht2OcfaDIhT/?= =?utf-8?q?g4qlMj/2iTagnJ7rX8rV06jAnK2pjki5bTIyQscEIoCgfN1IkbVJobE57QcmavYcJ?= =?utf-8?q?lAlKTrtSt4RGvStXmVz6rilJEWph5FHjJ5F2WqH+p0YcfCPKI1RLw+BT93m8KBZzr?= =?utf-8?q?1jWqBEqwl7IyjlutoidB2wM3PLVWVA2n7utH1Z7f0y/slHqcWZl5Z5x370pkCaIH0?= =?utf-8?q?1jY6tJcgiCNL1haNmCnbaiUmu0Qhxzur9ui0rgaPHLojbxwnY2UNbIqQq37Q5InkA?= =?utf-8?q?l1kqn8RPEGt7hUzFz1Ma67BLVOvxQ0mUg6J7Sp/8kb9iBOpKGKUAO9J5TDIq1q/F6?= =?utf-8?q?/2OGVlvgzRUcK04qOdRxubiw4xZ4asHK/dbDYwzdcm4NtWKrexYKT9/q5eUt5DM2Z?= =?utf-8?q?PxKL+5opsjmt9I43i1OT2mB6ygQ7OPPNziTfeb4BGUOKI8xco0eZTgABBhyQcbk28?= =?utf-8?q?DNRlqZQZHoXeAKx2u3e4BY7+FqnrEbZm/6cCiW6KWEk2e4XcgH6NuoJvweH9XQqlg?= =?utf-8?q?B1d0YaicoSpowcLXkEHiYKdOdK1wlYzkQbplqvkcggaUarOLf7z9d6A0GtFvjIWiJ?= =?utf-8?q?17i30/EjSFCtCn2GX/d7xVg42ci+KXmm5MEic328g1vwyg2mdHGNa6EnL2WRrfZXj?= =?utf-8?q?ZjOHB9j2VH6gkUR+03D+S3Q+l9m2auhd4TlsUQ/q4rlLlekKltwFbLANOvwE7On4U?= =?utf-8?q?RfGEhtjxz7hJYGMpx/JMPSzNrPQ9+gH/lO7aMCYRZqcXbIiXVH2peXTv9R+I2nSHl?= =?utf-8?q?W8nKtrOek6R24TtwOSfnuEizl2pJezSbfGC5nYOjlCDD/4oiauUDoj5NQlmTrS54d?= =?utf-8?q?D92iqwaYxrBvyIdVeXLIj2IoA1mcGdbTi8Wyj9R9KZk5nPoKxoei+Djv5TyxtKw08?= =?utf-8?q?RQEL39o/OLL9R80Iy5GNccjTf+yuHQEwPkoXvMfnSRaeAFf60iuCvTIJX89QiT+5+?= =?utf-8?q?0oR+uhPUMRg3KnmoYFzWZaq4lnn9W0j9wMIGqGf/KD1/GULFrcdMpWYK+ERm3HnUo?= =?utf-8?q?AQKkZwPpTmm/U19JAqyADUhbWIr2Q/eC7rYsYeOVRrH6FJ7r4cCA0lNrT4WEToY4c?= =?utf-8?q?GBzfrsOddFGOxxN/WhPYZin9yNykCsoZQilho2AGiQNnIxArXqpYVyqJQguEOXZM9?= =?utf-8?q?IQpgZVF+wOTWFjRrN7sy5Uy6zdUTG3m4KS2rZRqu62eC66oa+EmfEOrpoNZYem/VV?= =?utf-8?q?7LdyMQv3JqeXtYhU5mcz6XoK+IhmOanuCg=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: b29dfd03-f725-4134-ab46-08db05bc60c5 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Feb 2023 07:58:04.0646 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: xi6LaDLA09h7x60eXb18sGQm19RY/lGi3ML2w6PzjAb+aXLRVNM2+CzTywfIRlAAniTda01CR92bIP+biUPvpI5QKl2wM+tbcfHp68plImc= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR13MB6292 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: James Hershaw Remove usage of the rte_panic() function from the nfp PMD. As the ASSERT(x) macro was only used twice in the PMD, it is better remove it altogether and rework the code where it was used as it is a very small change to make. Signed-off-by: James Hershaw Reviewed-by: Niklas Söderlund Reviewed-by: Chaoyong He --- drivers/net/nfp/flower/nfp_flower.c | 8 +++----- drivers/net/nfp/nfp_logs.h | 2 -- drivers/net/nfp/nfp_rxtx.c | 7 ++----- 3 files changed, 5 insertions(+), 12 deletions(-) diff --git a/drivers/net/nfp/flower/nfp_flower.c b/drivers/net/nfp/flower/nfp_flower.c index e0e94e0a36..f1424a010d 100644 --- a/drivers/net/nfp/flower/nfp_flower.c +++ b/drivers/net/nfp/flower/nfp_flower.c @@ -491,7 +491,7 @@ nfp_flower_pf_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pkts) { - int i = 0; + int i; int pkt_size; int dma_size; uint64_t dma_addr; @@ -522,7 +522,7 @@ nfp_flower_pf_xmit_pkts(void *tx_queue, issued_descs = 0; /* Sending packets */ - while ((i < nb_pkts) && free_descs) { + for (i = 0; i < nb_pkts && free_descs > 0; i++) { /* Grabbing the mbuf linked to the current descriptor */ lmbuf = &txq->txbufs[txq->wr_p].mbuf; /* Warming the cache for releasing the mbuf later on */ @@ -561,7 +561,7 @@ nfp_flower_pf_xmit_pkts(void *tx_queue, */ pkt_size = pkt->pkt_len; - while (pkt != NULL) { + while (pkt != NULL && free_descs > 0) { /* Copying TSO, VLAN and cksum info */ *txds = txd; @@ -583,7 +583,6 @@ nfp_flower_pf_xmit_pkts(void *tx_queue, txds->data_len = txd.data_len; txds->dma_addr_hi = (dma_addr >> 32) & 0xff; txds->dma_addr_lo = (dma_addr & 0xffffffff); - ASSERT(free_descs > 0); free_descs--; txq->wr_p++; @@ -607,7 +606,6 @@ nfp_flower_pf_xmit_pkts(void *tx_queue, lmbuf = &txq->txbufs[txq->wr_p].mbuf; issued_descs++; } - i++; } xmit_end: diff --git a/drivers/net/nfp/nfp_logs.h b/drivers/net/nfp/nfp_logs.h index 5da384e53f..b7632ee72c 100644 --- a/drivers/net/nfp/nfp_logs.h +++ b/drivers/net/nfp/nfp_logs.h @@ -24,10 +24,8 @@ extern int nfp_logtype_init; #ifdef RTE_ETHDEV_DEBUG_TX #define PMD_TX_LOG(level, fmt, args...) \ RTE_LOG(level, PMD, "%s() tx: " fmt "\n", __func__, ## args) -#define ASSERT(x) if (!(x)) rte_panic("NFP_NET: x") #else #define PMD_TX_LOG(level, fmt, args...) do { } while (0) -#define ASSERT(x) do { } while (0) #endif extern int nfp_logtype_cpp; diff --git a/drivers/net/nfp/nfp_rxtx.c b/drivers/net/nfp/nfp_rxtx.c index 5f23128ac9..a5907cc428 100644 --- a/drivers/net/nfp/nfp_rxtx.c +++ b/drivers/net/nfp/nfp_rxtx.c @@ -968,12 +968,11 @@ nfp_net_nfd3_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pk pkt = *tx_pkts; - i = 0; issued_descs = 0; PMD_TX_LOG(DEBUG, "queue: %u. Sending %u packets", txq->qidx, nb_pkts); /* Sending packets */ - while ((i < nb_pkts) && free_descs) { + for (i = 0; i < nb_pkts && free_descs > 0; i++) { memset(&meta_data, 0, sizeof(meta_data)); /* Grabbing the mbuf linked to the current descriptor */ lmbuf = &txq->txbufs[txq->wr_p].mbuf; @@ -1010,7 +1009,7 @@ nfp_net_nfd3_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pk */ pkt_size = pkt->pkt_len; - while (pkt) { + while (pkt != NULL && free_descs > 0) { /* Copying TSO, VLAN and cksum info */ *txds = txd; @@ -1034,7 +1033,6 @@ nfp_net_nfd3_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pk txds->data_len = txd.data_len; txds->dma_addr_hi = (dma_addr >> 32) & 0xff; txds->dma_addr_lo = (dma_addr & 0xffffffff); - ASSERT(free_descs > 0); free_descs--; txq->wr_p++; @@ -1061,7 +1059,6 @@ nfp_net_nfd3_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pk lmbuf = &txq->txbufs[txq->wr_p].mbuf; issued_descs++; } - i++; } xmit_end: