From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: 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> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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?B?N0xucXltelBxcm9PN2FLZndiejVSSm5pdG9qR2FyT2tGYzdlaDBJRnJYdCtS?= =?utf-8?B?OUFaOWd0clZyOFkzV2wyQ1NTa1FjRjFyd0tlTC9aK1RueHh6Y1d6VDdvS3l6?= =?utf-8?B?b1QvOHNCYzJLYnpndnFuNStkY1NxZnF3bnVYaWZZa3NzOVZJM3JZaW5QOVZG?= =?utf-8?B?amRjZ1JzU3FiQW5kdGlzdURVQ1FCNTdlN2FBR2kwZWtONXlJY3pKMktaM3Ew?= =?utf-8?B?K3daN0liOUg3ZFFybldmdW94SHpTRGVwZXdNbUlJN01YZW5LcTEvMUN0K0tq?= =?utf-8?B?M0NJVDNPSmc0dzF1SUhiWVlPYzJyU1pMd053U3lVUnpsaFRpQzJkcDVrNVpn?= =?utf-8?B?bVFwU05WRXlBYnlwRWJIemkweWl5MTk5dW9ybnJmZVJ3cThKdllFTHlFN2hJ?= =?utf-8?B?QUNtZ3MyWU43TjBpeUYyWmZVdFR1dGpMQllLdVFCUzNlL2g1Z1NUQ2JnMGJ6?= =?utf-8?B?bkE1M2pIdFoxNWVNR3JvWUdvZXBkZEVRM01ucXg3OVc3alUxTHhucDVWUmVS?= =?utf-8?B?N09Ed0FjU0RGdHNvSEJFZHl5ekZnOGxGaDZhQWtEanB5dnVoQ3hxbElURG1q?= =?utf-8?B?TnBNdGR5aXA5Q2JmbFFvWVdlaDZ5UGhuYW5sQUU4WVBDOTh2MGdYNjhvajN4?= =?utf-8?B?SWk0cFQzZE5ncEtzTUFQQnZ4UVVLc2F1UmhQWFZ2TUpHejIyVzRyTzUrcWZM?= =?utf-8?B?eVZoV0FpQm1UN0VKcTJ3UG1GSVA1eExEeXVoVFo0aU1JQmVRVjZyRjVwYWV1?= =?utf-8?B?cmlEa0pHSWtyTE9WUXk5WFF0ZXFHZDVPbTN5eUlHd2JlcDliVHA2eUJmbTEz?= =?utf-8?B?Uzd3NlgvNjNhbkt4RXJKODlsdC9XOTVkb3lyNlUxMnA2SEkxS2YrOUd3bmF3?= =?utf-8?B?b29vTnZrb0x1VWppU2JWZnVNZEU4aitVSkdkcVZHT2JlWTJKNU55TjNnMWdR?= =?utf-8?B?aElXOS9OZHhzSGl5WEdWam5FM2pIM25ySnhtS0ZzeFJaKzV3QWp2UEdEU255?= =?utf-8?B?K3ZFOG1CV0xvUkNocWtFK2VaYkFqaVVvSWtoOUd1aXk5bFJZL1ZMWEh0MGYv?= =?utf-8?B?UU83OU0xWkpQRjlqZW82N01KY3JVZm5YTHlleE9JMmVHN3R0SkswdzRXZHVo?= =?utf-8?B?dUdRL04xcWQzQ0pMZllBUWxDdHNrMFJza0VzQ0E2a3MvWUwwK3duVnVleFFr?= =?utf-8?B?SmMrTDdBTWJNU3RDdDYvM1pnWUk0SEFQV2RMOWVZSGY4d0pkNXlJZnlGNUYw?= =?utf-8?B?WE92RmprN295VFhGaUtVRXc2bDNtQVYyMEE2UWJsdnd3a1hXazZZa3RhVWNI?= =?utf-8?B?dFpGazFpK2FGbTY1dDR5MGUwYVZGVmVMZndzclVhUlN0RWdGL1lnKzViNUJB?= =?utf-8?B?NGNYVTJwdncwNzJZa0w0a0hSYll0YTMzV1JlSUVkdGs0cFpWL0xzcE5vbzRx?= =?utf-8?B?MEE4VWVSNVp0dU53dER5UXJKR1VsZ1BsZFdheVAvSFRGMncxMzgvVzNiM055?= =?utf-8?B?OUwzRVhNbVBDTTF1UTZlV1R1Q2xNL3JuYXJnTm4vbXU2dXB1WjVpWGRCYWJV?= =?utf-8?B?UmZIY3drd3NSMDdOMm9CTzZOMkxzbENOYWdQWWtXeFMxQUtsSGhSc0VYQ3p2?= =?utf-8?B?RnhadzRPQ1R4dXovR1Nha1MyUE16TFJrMnZkT21jSjlTQ29kbHpmZlpYUUpK?= =?utf-8?B?UVVNYWZWNlNxNm44dVFkcEZ1eW9aNmc4VEpYTWdwNk1mQzRIWjlFSVNZcUFX?= =?utf-8?B?R2hRUTZFRXFMY3VRYjRMTHhYczI4TGFHQzZUQ3Jmb2g5QUxyOTZ3aFFRZklw?= =?utf-8?B?WXJRVExkNDNMT0o4N2EyZXdxREJpWXlVYldvdDBNaTJNTE9iMVVWbU9YQ0JS?= =?utf-8?B?NHFlWTROaFM3R0R1aDlOSmw1aWZWa01DeGgvM3U0MHoxdGlZWmFHS3NkTUUr?= =?utf-8?B?bitDNkZrdHlJV0pEaEpjdDc3dkJKcElhUmQ4OThTVGZrQXlteEVCaFQrQnd2?= =?utf-8?B?U245Q09rQWZ0cW9YMkJzdzBqL0ZKN2FMK2xjM1BpMTN5SXFQL2Y2OC9TZDV3?= =?utf-8?B?aWgrVlNDNlRDQzg4NW54K1NoWnVFT3NnNXBjZXBVY3RQR3VuT3dMd2FWVUg0?= =?utf-8?B?djc4bXp5L0ViRS9veE5LMnlmU1VVcVpiZUw1UWRRNkZnYTdobUV3VEY4anFN?= =?utf-8?B?clE9PQ==?= 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; } -- 2.29.3