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 3F11E457FE; Fri, 23 Aug 2024 09:36:48 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6D3444330A; Fri, 23 Aug 2024 09:36:38 +0200 (CEST) Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05on2078.outbound.protection.outlook.com [40.107.21.78]) by mails.dpdk.org (Postfix) with ESMTP id A40A443003 for ; Fri, 23 Aug 2024 09:36:35 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=OoOwI0Fi3G/te2MxVzymmlge1MpZPVdLmCjkXXlTJGuuPbImKAf4JbpMstuTxYNDNjiE0293O6hauYK0C1bBPHfX3PdtJDVU290pyupcBykMAAPQxxbYYzENyh1cQs3HnQDSVLc2Gfi2SnBnrSbWLtwCCjnOpuaZpMpdBjIGvI0f+AbmW6BN2J6SnXVWh/zYJo4o2h/KTTFBvrcbIEhH/QKluoCfNjyEwsY3VG7qZKxBXpFV2Pvw17uE6TvLitXAqIwwOxggAJztyGX8SLtp/ycgQZsszk7iIY81nnKxfaO8TFFd1797XtBjSm0dI/MuH/JkrldkEmFLbZC2kAXXOw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=qPY9kU4pjvt7DyOIjA3D+71FL568MkX7rr0LyANDErY=; b=Q5TFONpjfS/gua1blU35FLapiFcJDcC4Ima8VmWzynrbsTU9kfR+EabNGZd62k2T+G6pH0b39+3Ye3Snjdq0mOfGDAbO/PIM6FpfW8i6pToXCj2uoEAldl1oPLZ8gl9HgNskASzVgn1et/GFoV4+HxzANTup2forCI0vc4vguP999tJdXGygFF28oJnZBtETKJn7h90XIOnrJvJS16uQFHe/+qF9hW5wMhAaX1/y2Vu80GOjN9lbuL7+ZW+LkGA0Jjqfj7fqBxOlJsVJa2c2Bmb6NW3qHjobNJyU4OTSrVScC0wEch0YP/AQ3vGBXh1LUjI78AnGmR6kgj5cuBsXgQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oss.nxp.com; dmarc=pass action=none header.from=oss.nxp.com; dkim=pass header.d=oss.nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NXP1.onmicrosoft.com; s=selector1-NXP1-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qPY9kU4pjvt7DyOIjA3D+71FL568MkX7rr0LyANDErY=; b=H+M+sNYVOzXoQPLoNeKTOjMv6PLKZ6WxGUJw+e3upnubZWl3PxCLB8XtCct1moMQM3HMLKTIMwhWOxCG6Vh2GTCt0LMuDMs+JnvX6LNQHIjQfA6iTS9PhfEHmfdm4PjKfKrqz36vD2c3qicTYYUzblkMhoFzKusFVpHZnSlzlFHU2ovNOaTUHI1IaH8nizh3+PXxEu3jkW5lUlzA7/yOsCeipiQ05Oi+m2KzKFpH6sglS5Qb5TpgyVrk9cB3bua1uY/qR2QBCirJa2rdw5Spt5fGiUIyzJsVXOEkT1Hp/f9fwgYZg13NDrcUygr7OHktpCKz+HFL9g5R0Tg71y5pAQ== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oss.nxp.com; Received: from PAXPR04MB9328.eurprd04.prod.outlook.com (2603:10a6:102:2b6::15) by DBBPR04MB7994.eurprd04.prod.outlook.com (2603:10a6:10:1ea::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7875.20; Fri, 23 Aug 2024 07:36:33 +0000 Received: from PAXPR04MB9328.eurprd04.prod.outlook.com ([fe80::367:d59:375:fb87]) by PAXPR04MB9328.eurprd04.prod.outlook.com ([fe80::367:d59:375:fb87%3]) with mapi id 15.20.7875.019; Fri, 23 Aug 2024 07:36:33 +0000 Message-ID: <4da70a7f-1612-6df8-7ca1-fc81777f7c73@oss.nxp.com> Date: Fri, 23 Aug 2024 13:06:27 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.5.0 Subject: Re: [PATCH 11/17] net/dpaa: enhance DPAA frame display Content-Language: en-US To: Ferruh Yigit , Hemant Agrawal , dev@dpdk.org Cc: Jun Yang References: <20240801105313.630280-1-hemant.agrawal@nxp.com> <20240801105313.630280-12-hemant.agrawal@nxp.com> From: Hemant Agrawal In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-ClientProxiedBy: SG2PR04CA0162.apcprd04.prod.outlook.com (2603:1096:4::24) To PAXPR04MB9328.eurprd04.prod.outlook.com (2603:10a6:102:2b6::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PAXPR04MB9328:EE_|DBBPR04MB7994:EE_ X-MS-Office365-Filtering-Correlation-Id: 3ac97262-d3e6-4a5d-0fa2-08dcc3464fcb X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|376014|366016; X-Microsoft-Antispam-Message-Info: =?utf-8?B?WVJKMnFQVUJ1WEhxVkRDWGZMQ1FuUXZqVGt4US80MjJmUjBCKzV0dXA4MzBr?= =?utf-8?B?SUk0a3JidXVLTDFtYWdNclhrL2twNWMxYXBCL0w0T2VJNmdSRWVvK1JuRy9K?= =?utf-8?B?UUFTdlpyNG1TakVXZWFPemJzY1JoaUthVng3eFFOWklRSzRXZk93by9IcE43?= =?utf-8?B?MWlNdGtCUk5HUTErYVBYSVpEVlRTMm5ka1hlYnVkQXh1T2NyOFE5TDViM3NN?= =?utf-8?B?UWtRU3NES01EQXp2Rmd4Mko3Z283ZDZOWUJiUnRhc3JjUzB0UTRsZXNpa050?= =?utf-8?B?Y0NKRzA5R3BUTzd4K1JnT3lReTRqd2JHa1pIUU5LUnN4WWRwOVRpTEtjMWx4?= =?utf-8?B?L1gxRG1rNGJGN0U3eGdHQ1kzWW5Cd0xDSVQ5YkpMMzJlcDFxeE1jKy9lcHBx?= =?utf-8?B?U0g2Y3Y3cGdkOGltVXhma1ZOZDNFY0V5OWM5MGZSUDF6MlMzTFI0NnNacHhQ?= =?utf-8?B?OTdZb2crSXZNUUxJdFl6V3NSbFZzTk1WWUhSSHprY2w3NTRqZWYxNnI1NDIr?= =?utf-8?B?VnZISDdjL0ZjMUgrbGZrdWVJWVFOdHBmbS9kSlZ1c053eklxbVhPVG5Lb3NX?= =?utf-8?B?WlYzdG42TDJjQmFjdDJpM3BqUXFET1YwaytXOHczR3UzOUFaTVVPSm9yNkJw?= =?utf-8?B?Nm53WHJwNGhETjFDRWNSNlFESDZGZFRqUVFTdld4YXYrbnpxYWs0eEFsaFIw?= =?utf-8?B?eFdoZXFhYkswcGlyZlZNVHNHUnhlRlo5RElQZHhYaU9LNjh5dllJejdLdlov?= =?utf-8?B?cFVlMnRwNitaUDROMEVPUk0rOTFKQjBRaTdSUkJqb2hWaVNvSDBYU0NLVEEw?= =?utf-8?B?cDNrL1g0MHdJOXlTek81anRlam5mL0dpMjVMRkh5OHE2VTdjRGlreEl3RVRF?= =?utf-8?B?VVBVNDZHMm5hNFV2TU5ocmJQTVZZKzR1RTE1c2RhMmg0SGJtYlU3ZVVnVFRM?= =?utf-8?B?bi9KMTQ2RklwaFpOK2JuQzY4VVpscCs0VnlnUUkxd3VQMnlhb1RDUmNveW1D?= =?utf-8?B?N2lCaDV6OEtHQTZ3cWhVQURySEhKT0tKNFEwb0tOWnEycldwLzl3dHJKRDYz?= =?utf-8?B?RUNhVGFIcSs0TnJFZFNRc3VJd1BCR2dPNENrOUFnbWVPL2JZSVltL1poOGpt?= =?utf-8?B?dnRtbS82NkZOVFkwdHNPYUVhc1hTUFFCZTBCZGZlbWJrMXNZZkt5Y3p4d010?= =?utf-8?B?R21EaHl2U1hNWGprbkFFMlN3bDZadHB3TGRjU1V0UGRVNlQ2V1NGOXg3M1l6?= =?utf-8?B?aEF6c1JGY1cwQ2RtdVI3dWszNWRyeWRuKzB6QzVTSm5MSHkvS0l6M2dWUSsv?= =?utf-8?B?TitiUExjaGhBR05aUE44NURsanh5VUluQUZxSE84UnpzMWZoTTB3TnhwYmdR?= =?utf-8?B?SnFXVmFYZHBiY2Flb2VJZXZ6bHl3eEZ1anVscVRCNFhVYTNiR0lqdWFtYlU5?= =?utf-8?B?MXpyRUR3RnROQ1BnZGtZMTJNRG04SnRKd3NSMUZaSG11cWxrdkdIcFZpM2RS?= =?utf-8?B?YlgraDQyTVEwTGwySGFkMGxZWW5HUTdKdHFacHFpQlAxc2FsbUZvMlcraFQ2?= =?utf-8?B?YjFJY1dxRUw5cmZNTHB1ZTJtUGlGMzJjRXFrQ3g3enE2SVJTNG5CV1RnL1Ez?= =?utf-8?B?a01sUy9laVFIVDVtbFlsN3BxNjFKNi9zRlh2cjlObzBGTTc3ZzkzTW1EcnF1?= =?utf-8?B?dFhNM2N1L3IxVDYvZ0RweGFJdXJ6U2pZUmkxeXU2UGZqbjNrWHBDTW01KzV6?= =?utf-8?B?YjlwemM1RytkQjRFYm9BYmZRR0U2WnIzTU9UbDZaUG80Z1JrSWxKb1NpTjVN?= =?utf-8?B?czJmMVYrVmVLcnRWSkdsQT09?= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PAXPR04MB9328.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230040)(1800799024)(376014)(366016); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?S3QyY1k3dHRraW9FYzBzb1ppc0hwNFVBeC92WkdiUkhHcU53V214UkM4am9B?= =?utf-8?B?VWROWUxRaEVFdmRzdVVjazFMVEtkWEUrakNSZ3l3d1lCZWpKZ29RbXFuV2RT?= =?utf-8?B?VWhzakdkanJ2UEMrWkdHUjhnb0R0TFh1dFgvRk50MlJBdlFDbFBMNjh2K1Rw?= =?utf-8?B?b2tBTExhSUdSN3V3SHlMQ001R2M1cExuMHYzQjZwTi9mY20yd0lEK0Q4N1lH?= =?utf-8?B?U1JhVytnOFQrc3hBT2wvUHRDaVg5NXJJZk9JWVlxd3QxdWxMMENXMlZJV0VQ?= =?utf-8?B?RTQrRnR4dnQyMmRCVHErQ3IrdUxMVFQzb1NoYzF2d2hSMkdhQll4T3ZpQmdK?= =?utf-8?B?N0dYUWcwaC9VZlduZnlsMEhSTEI4WVcyQ2VkNjNZTktDYkNuVFNkUk1hRmxR?= =?utf-8?B?cWxmRkpFdGVDT0VWRm1FY05xZWs5YmVvRWQ1LzRaV3RSWjgrMzhhY1hqVFlJ?= =?utf-8?B?ZTlCYVhkbGdRYXI5Z0hWZ0FYQlBNS0ZCNVJCTVBpQ003WmVpQml6cFpKVDQ5?= =?utf-8?B?eDZoekQwQXV3K1UzTUw2S1BvakRiNVZkMWYyNjE5bDlhSWdzU0x1ZGx0S29F?= =?utf-8?B?NC9kQXpjYThIOEQxWXdURzNHUE1EVEE5NTRCUHdwTnJFNDMwc3NDTWo2U0tI?= =?utf-8?B?MVg1VHRjeFhXT3Z6T1BCejdEZGQrZUlPL2V4dmxJUzYvUk1jSjVLb01ra3dE?= =?utf-8?B?bXhnVURPYjJVTUdyV0o4MEJCN0Y5SEVNUXhHZnNFODJFdGJ0L1ZQNE0wR3Y4?= =?utf-8?B?dlNtMlY5d0JvRFp0UXdSSWhxNjVVblVqaWhYWXJ0d3lwK3NmVnlJb0ZFY2J3?= =?utf-8?B?aHIyWmJTNkpMOWVCTzdWMjJiYjZpd25XMDIrMGsycXRMaDZIS3RpdS9naHor?= =?utf-8?B?UFpMdzNQeWtjSkdjUU5reloxOGNDZlpzbFQycUZNSzZCZno3VXVsU1oyVm50?= =?utf-8?B?dHRmcTRqTlJZejFUQkdKd2RHL201T0JvYTZIVWE2U0FJeFRtclZSa2VNOUtn?= =?utf-8?B?VGpXZFpOdllIUHMyNkdqSCtISCtUZEpCRjVJTkNjSFVhclZjMmo1QUtZSERN?= =?utf-8?B?Rm1Eb3Q2TjFZdlFwM0diUGgrK2tHTEN2UHh5ajVrVnpsSFFac0hDb1Rzckoy?= =?utf-8?B?Umxpc2dFdUhKV3VaVURnby94dlZvNFN2SEFUZ2dTSno0T3M0VWkrL0p4Vmk0?= =?utf-8?B?UVU4a3pueStiQy9ndDJiL0NkV0ZCdk1nSHNaMDArRDNBNTkzYUFKMFRqaXhS?= =?utf-8?B?MHlBN1FjZ1h1RVZNeEdkWnZKYWJrRzVNVzRLUk5FQ1MrS1N1Wnlod2g1a0pt?= =?utf-8?B?dWJiWWdHK1BMam5YYTFHdVF4MWpUUFVGUnQ0V1dVeHl0UEVlWC9pd0orRjl5?= =?utf-8?B?WmVMc283VXlXNGRWZ0hUdVFlTklkSkExM3lUODkzT1ZmNVdlelNaNnYzS0h3?= =?utf-8?B?YklrbDlrQlB3VzRsYk1UZUlWTFZuSUE5cFJzbXMrWUczenByMDd5WkZKd0Jx?= =?utf-8?B?bS9OOG9tbWlpMFFrRVQ3eVBSYXpNTzQ5VFV4dVVlNGowczdpOUk4RG9IOGla?= =?utf-8?B?NkZrZExBSXk3QWsxNWxEOFh3aG1rQXNJV1c3WEJlc2pab1l6NGVOdXJGT1E3?= =?utf-8?B?QmhWZjBMdEI1UFdZMUlLdDV1K0FlMndmRVFJSXA4cmJsS0R6S3BJMk1EVWpD?= =?utf-8?B?aENYbHdFTHZINDdUcFl1RjJhMHR3N3VuVlVZeE5aVXp1MENibU42VEZpQllE?= =?utf-8?B?TTF6akFTTWRRUzhPUk1yUStVL1BNYXBkc1poL1hMY0NZY3NVK1JkZ1g3enVn?= =?utf-8?B?ZTc1RDdDYmxhL1d1NG5haGg2TkcyWFI1YUczOGV2dnBZN21TdmswVkFybjhO?= =?utf-8?B?NUpWVVc0L0duNVQxVHczSUMzWXN1VSs3a2VPTVFFUU1JWkxxY2M3WnMzdDlO?= =?utf-8?B?NjFNN3ZRbGlMdFd6cHlmdUFaYWdRU0hJQW9jN2ZjS01RMkw4TmllSFNaTy83?= =?utf-8?B?Tnd2ZXp1MEw1UXdoZ1lCMjdrcjBIdGxKNzNaVWdNOUd4QThkVW1DekRFL1k0?= =?utf-8?B?RWR3MUhnZGNpTWhkMkhRMGdyZW40b2VsYzRZZ3ZsdDZ5Wit4ZEswdFVjdExX?= =?utf-8?B?TURCWUFWYkdhRVMwS2IwMDN3NnJ2elBOWFVOQWlhV0Z5WlJDMmJmUnBvOGZR?= =?utf-8?B?K3c9PQ==?= X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 3ac97262-d3e6-4a5d-0fa2-08dcc3464fcb X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9328.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Aug 2024 07:36:33.6012 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: wdhl94L9mTUk/aJzNCLkRcc43sM+PPuc2T8ASvV1zm14OHNy3bQVICNFibLffWQlZJ2oyP1N/I4OqYrsnBY8og== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB7994 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 On 07-08-2024 21:09, Ferruh Yigit wrote: > On 8/1/2024 11:53 AM, Hemant Agrawal wrote: >> This patch enhances the received packet debugging capability. >> This help displaying the full packet parsing output. >> >> Signed-off-by: Jun Yang >> Signed-off-by: Hemant Agrawal >> --- >> doc/guides/nics/dpaa.rst | 5 ++ >> drivers/net/dpaa/dpaa_ethdev.c | 6 ++ >> drivers/net/dpaa/dpaa_rxtx.c | 138 +++++++++++++++++++++++++++------ >> drivers/net/dpaa/dpaa_rxtx.h | 5 ++ >> 4 files changed, 130 insertions(+), 24 deletions(-) >> >> diff --git a/doc/guides/nics/dpaa.rst b/doc/guides/nics/dpaa.rst >> index 580edd9327..448607e9ac 100644 >> --- a/doc/guides/nics/dpaa.rst >> +++ b/doc/guides/nics/dpaa.rst >> @@ -227,6 +227,11 @@ state during application initialization: >> application want to use eventdev with DPAA device. >> Currently these queues are not used for LS1023/LS1043 platform by default. >> >> +- ``DPAA_DISPLAY_FRAME_AND_PARSER_RESULT`` (default 0) >> + >> + This defines the debug flag, whether to dump the detailed frame and packet >> + parsing result for the incoming packets. >> + >> > If this is for debug, why not implement it as devarg. > > Environment variables are supported by a few drivers, and as DPDK > drivers are userspace applications we can benefit from them, true. > > But that is yet another way for configuration, I am feeling it is more > suitable way of configuration for applications. For drivers we already > have a devarg way, sticking to same configuration way provides more > consistency for users. For the debugging flags, devargs don't work well, at least for us. Customers are using EAL argument embedded in their application code. Changing devargs means they recompile their code. In the production, the env variables are working better for us. They just need to set it and they can re-use the existing production builds to collect more logs. >> >> Driver compilation and testing >> ------------------------------ >> diff --git a/drivers/net/dpaa/dpaa_ethdev.c b/drivers/net/dpaa/dpaa_ethdev.c >> index e92f1c25b2..979220a700 100644 >> --- a/drivers/net/dpaa/dpaa_ethdev.c >> +++ b/drivers/net/dpaa/dpaa_ethdev.c >> @@ -2094,6 +2094,12 @@ dpaa_dev_init(struct rte_eth_dev *eth_dev) >> td_tx_threshold = CGR_RX_PERFQ_THRESH; >> } >> >> +#ifdef RTE_LIBRTE_DPAA_DEBUG_DRIVER >> + penv = getenv("DPAA_DISPLAY_FRAME_AND_PARSER_RESULT"); >> + if (penv) >> + dpaa_force_display_frame_set(atoi(penv)); >> +#endif >> > 'penv' is not defined and build fails when macro enabled. fixed > > It seems even yourself not testing with the macro, this is why using > compile time macros is bad for long term, it is hard to detect when they > are broken. > > Can't you convert this macro to some runtime configuration, that is > enabled/disable via devargs, so easier to test, ensuring it is not broken. > Or other option can be to switch ETHDEV debug macro, which is used more, > by multiple cases, so more likely to be tested better. > run time checks are effecting datapath.  We usually ask customers to re-compile only the net_dpaa libarry and replace it in their test for debugging purpose. > <...>