From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
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 <dev@dpdk.org>; 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 <ferruh.yigit@amd.com>,
Hemant Agrawal <hemant.agrawal@nxp.com>, dev@dpdk.org
Cc: Jun Yang <jun.yang@nxp.com>
References: <20240801105313.630280-1-hemant.agrawal@nxp.com>
<20240801105313.630280-12-hemant.agrawal@nxp.com>
<ddc57785-7a4a-4561-af82-d61c3402c86e@amd.com>
From: Hemant Agrawal <hemant.agrawal@oss.nxp.com>
In-Reply-To: <ddc57785-7a4a-4561-af82-d61c3402c86e@amd.com>
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 <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
<mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
<mailto:dev-request@dpdk.org?subject=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 <jun.yang@nxp.com>
>> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
>> ---
>> 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.
> <...>