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 9D7DF42486; Wed, 25 Jan 2023 14:21:46 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 915D142D74; Wed, 25 Jan 2023 14:21:46 +0100 (CET) Received: from NAM10-MW2-obe.outbound.protection.outlook.com (mail-mw2nam10on2080.outbound.protection.outlook.com [40.107.94.80]) by mails.dpdk.org (Postfix) with ESMTP id D94EA42D31 for ; Wed, 25 Jan 2023 14:21:44 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=PVOrBNYBeSo03Ve0PMbXEnfnQSwHSSTLSJWanbM5f1s8fctF6DZ6Ghts4vYyBoyiw5sOwTcXgZCVvrBoFu2dVepzXxzbM30j1dvw3mo+FmdQbDwL88bq6iezCdl1natVQE1QR/ACiLJ5zfGXDARyT3WLgKTjxKnN5xBpKgS10J6JQBpL6S6ITqj66HpYPts48ZonC5EM+bQgHEHEe5sO33Nif6A2Av5Mbla1MCayy8yELzhfIvlW3OH38727LGEYcbMBJXvm5HOviSwjK2bNHUk+2g+5N0jM9Xe55tvhW0OQ9SqBtAc/hiDtqFK0oMai+KbL/PGqwvQ2+F7OmH1dPA== 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=H/zj8QrwWFEWPxKc21xSKPXvad2XPi8Ol7cQOkSuHGU=; b=YGOSrNbPfc1Sb9ccIaTUliMmOteJKKwYZnKj2c2k/20XwHVCirCJq2JqFotrAsmgp2o4pxhl+RmiSU78HUTUK6Mn0ix0hgldEZMOEyiX8Hj7vvXTdSWvak7H4ORFgtnots79X23B7Qz2zAMZAy5qjxXGgxZCCIJAGbaG/WpGmfqwMG6Slg641/XehXmTpawE3fs1ASIW/ReeGzichnUvN7FV0UdJY4VMXPbZGUpbkLk5dr5ZmbjbsD1AGVB/oyZPhzpjrRuoA1RLpA+d92Bc01PX2XfISZWHHjiOmIdJ6c0z5SiIOykx/g6RdZg/pxC950lcLF+ZD7y1T0w1VNJ7mg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=H/zj8QrwWFEWPxKc21xSKPXvad2XPi8Ol7cQOkSuHGU=; b=z7g0swdzSrZc0OHk03Gem7bTPpAWpCs7DfaqgSiZ31Y8hsUWfSFwaDAcemCmtAYZIVmIxwraAuXU77N9OuEMjbRnVHunJx66QmsKESqRKits/ZTLm1yEcxf7cM3gxIn3IHkMluDRIH3014+w/eOreCGw1VrRE4m6M2nrds++zGk= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com; Received: from CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) by IA1PR12MB7568.namprd12.prod.outlook.com (2603:10b6:208:42c::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6002.33; Wed, 25 Jan 2023 13:21:42 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::4807:1f44:5e04:e05a]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::4807:1f44:5e04:e05a%7]) with mapi id 15.20.6002.033; Wed, 25 Jan 2023 13:21:42 +0000 Message-ID: Date: Wed, 25 Jan 2023 13:21:36 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 Subject: Re: [EXT] Re: [PATCH v5 2/2] app/testpmd: add command to process Rx metadata negotiation Content-Language: en-US To: Hanumanth Reddy Pothula , Aman Singh , Yuying Zhang , Ivan Malov , Andrew Rybchenko Cc: "dev@dpdk.org" , "viacheslavo@nvidia.com" , Jerin Jacob Kollanukkaran , Nithin Kumar Dabilpuram References: <20221220200250.2413443-1-hpothula@marvell.com> <20221221020713.2803232-1-hpothula@marvell.com> <20221221020713.2803232-2-hpothula@marvell.com> <11fa3448-7348-09d6-b7e0-321aef1d8953@amd.com> From: Ferruh Yigit In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: LO2P265CA0488.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:13a::13) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|IA1PR12MB7568:EE_ X-MS-Office365-Filtering-Correlation-Id: 8226aef5-43c8-4768-1376-08dafed7192b X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: rhvcV+8J2CarsKe6xhnV1IxY0Sxw7mhQ7RUejtQbPmfI9MCRMZS/Ft+3eZvD9CTwpXa28sXbS9mxhYLmkaLVpZnjbpZRGkwZbg07v5KQASaVNFl9Nm5vL2H29KVQWEM2wJLVi069gqKzV0241wWitM7ZbxJJoE5RLkDi8mKbyvvaMy/2M9RL43K6hiVft1g52Kgg0fKqquNoa2z7LLZVDWHt+r34exLYV/Q2hHqNC3O9cgTPIcBbtOqQeImRXWiG+NyFD9yf4zPwXibrHGHcS9WKyB3ToFeiOpraZGfzFl4H5xDwEJBu02hD5Etw/LJuA/CJ1H79QSBu0hFXpbTZMgrDDzpm7/aL27n6t8Zl9Y5wqK31XKS/Y4Poo7G/SrQyCnzh0j5vTuLiJ7OzmEJLr4Ygmd1C5fynXNYXLQDsk5F7lqNhbNBcprC++m1rNPuSZCAeZwA3hw760CMFQME4AueqYzE5QU5INW9hatS/PeQfsuApPub+pLwvIzBleuYplikjfj+teCmW+oddmZJ+zX9Yo7pTh7TwkuGOOfQ2ZtGRXxHm+Rglssn2C/HEC8KN3KmoL5bUNt09fRoHSdBWDO1Km7CXdIseyhqG/K9x6xrGg/nbWo63bLmtlR8sN3C3gtBeOCH94mAKGI6oGCSmuDJ5McKd+WOZL8zk92DDtYxw1EakMRsPTDwFgnB4WluAT0SGgYsDhiL+OM+YAsMF1b/eee0TIWiOHL3oS8FR3DhgoViNFtwp2FmTSUqpPCnURXCzr7O8aYiNb9KDfpQnBg== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CH2PR12MB4294.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230025)(4636009)(366004)(346002)(39860400002)(136003)(396003)(376002)(451199018)(38100700002)(36756003)(110136005)(966005)(6486002)(31696002)(478600001)(316002)(83380400001)(66556008)(44832011)(66476007)(8676002)(4326008)(2616005)(54906003)(2906002)(31686004)(6506007)(53546011)(86362001)(186003)(8936002)(41300700001)(26005)(66946007)(5660300002)(6512007)(6666004)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?U1E1Vnkya0tLYzlLaVVmNktmOFNkUVZMYlI3M0NPVnJwL1g3TmhhdWRpaE5V?= =?utf-8?B?SmpLdjMybzRtRkhYQzd1UEVtU21uM04zSXF6QkVDd3JXYnV2RGJCZ2VmbUwr?= =?utf-8?B?UTZuZjhwOW9TTEhXZWRKK2FjMHNKSEk4bkdYZllqSWNmQVlMSStCMnRlV3RD?= =?utf-8?B?MFk5VGE0WlJScy9BRjd3T3V6WjYxK21QbHJmSDNxM3lXY3Y5MnBRYUgwWTJ3?= =?utf-8?B?ZXdIbUVMY095cy9EQ3ZTbDBvRUV2d3lET21wYUZzQkE3UWw5d1BpYW1COHVC?= =?utf-8?B?VXA5K3ZjcnFpSTd4TzRMM0lnT1lBN2hZTFRScW8zcWsvZ3NTOGhydmw1OUVB?= =?utf-8?B?a29mcjdoQmZEdkdiY1QyUmgxMkpPWXlLanVINmhvdTV5ZGxYTkJZRVhGOGI4?= =?utf-8?B?Z0dBRnovUlhmQUJ2bjJxbmZZZE50eVowMEQwakJKaE1Lc0NwaTgxU0tPZTFR?= =?utf-8?B?VmQ1MTZRdkMwM0RUU3lyU0Uycm9YR1h2UnBHZGZnRGRoQmV6b0Z5QzZWMXg5?= =?utf-8?B?VnMyUnhZNW1HeXpIWENlcFFvMEh3N2Z4TzlUaTFYQVlScGR2QS9GZHUwUFI3?= =?utf-8?B?VTErOWJROFN2d3cvYlFZWlZSUmY4cGFQaWtocHBkTnlRTUo3Q1k0QXM4aEhZ?= =?utf-8?B?UjVNWjJxVWVWNFIrcnFiY2t1b0dPbU1QT2VtbndEKzJZVFBIZWpCZ2lGTVFF?= =?utf-8?B?VTVZR0hQcll3L0huN3JlTEIyb2NYYnJHSmZjcXVacWdEWWs5aUZhTmU2eE5h?= =?utf-8?B?MWpyTGNJeXY4RExlMVZuWVZ4YytEM3FoUE9zUmlhODludk9CcUpYbWFWZ3d0?= =?utf-8?B?ck1zZ1NiVEFQSTFISlB5dFVQWlhnS2tBdGtDejE5eWp4dHRvRktER3BkNStI?= =?utf-8?B?dzhjU0JWem95bzlzS3RuWWRtd0RVNXlSTkExMVVuZ0FBd1lFc1RQVkNyWnlZ?= =?utf-8?B?eEpFMVZnVFNrRmozM1d3OG1FTVVnS0k2NHVna3NXQTVGU1JkWENsOFlzVHRr?= =?utf-8?B?WVlaODF0bUR3ZlZVOXl3c2ZGUTZlb3NvRFk5MW16OTk0c1A3L0h5b1R4M3lB?= =?utf-8?B?aTh4WVJHWHlLTkJIc0lWcElQQ21aTnZFcGo0Rk0ram0vVlUxNE9JeTBXVkZw?= =?utf-8?B?ck0rQ1lBQjNyZ2ttVUovRk96cEFlbGdjQjFmd0tGODZsWVkzY09yait6c3Vx?= =?utf-8?B?bFVUdzRaU2kxaDEwNjVCeW9CU2YyVWZOMUplTUFCMzRENW43WGZIMzV5VWc1?= =?utf-8?B?WHJHZXJ5c2p3RDNzdzk5VGRQVjNYWStiaUg4YUc0NkNYeGdtYUhPTDhORnUz?= =?utf-8?B?ajV5MkRMV1lzLy9BNUJMQWFkYS8rK0xiQ2kxNUwySisxVFFNcUtsVy9QNzRj?= =?utf-8?B?cXhqRjVwc0kyUWg4Z3ovVit0eHZ6MEFrRWFEZTM0aVRFZlB3TXBnRTJNVW5P?= =?utf-8?B?c0RHSzhkNllIQkovWmp0Y3g2UXYyaHVBTUhQMlZPNlJ2Tzhmc3hGd0VWMW1l?= =?utf-8?B?aHZYMG00K3dvMGZJOHBjbkJlVzRyMDFVc3l5SHo1M0R2YVY4ekJ0cUJJTytV?= =?utf-8?B?bnFtZm5LWkhBRFNkUXhuSFlwQzNvMlgyZXRVVHFIYXpXb0JHb3BzbTAwTlFr?= =?utf-8?B?ZUdDclBpVlFSTkpVczR3TUl2KytidWFrMUVLSUE2VGZEL3Vrc2NmeVQ3T2Zi?= =?utf-8?B?UUpCUlVMWjh5RjFOR3czcUlKSVp0dmZlZ3VNRnlqb05DQy9zSmlMci9QWkRL?= =?utf-8?B?aXpmbnExMjAzWXpmMzhmRE9wQlBwWVNyaUROclp3ZDNaSitEZVZXT0JiR0hC?= =?utf-8?B?M3dFeGV6Ti8zUmJmVVFscWR1T0p4dVBFYVlYRGZHNHRNeXJiQTBPR3hrNFhY?= =?utf-8?B?eHVUWEIyaGM4bXh6NmNhNmtaeVRxTDNHQmlNZVdDZ0x5Z1NWS1dQMXlnUzJ5?= =?utf-8?B?bzBVOGJmWEs5Qzc3YzNjRDhCSjh1Zko0WGFjLzdYS2lPa3B4LzBjQVlNWmd2?= =?utf-8?B?NFJiQXZpUzEzK0FSai9zazAzdC9aWTlsUjQ2anRsU1pRWnJTNUp2b3ozMDBy?= =?utf-8?B?ZnlacExwMURiMkJTR0hDRnlIV0xTVU5wbUlXQ3Q2K3dxaGVCWDM0NFZXOFhy?= =?utf-8?Q?Vg37SPwhCDVbj7SiM3Zrk/t+w?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8226aef5-43c8-4768-1376-08dafed7192b X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Jan 2023 13:21:42.2367 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: IOcSTZONvUjVT1Z6Py1BS3RmPYrkADLvxo7PQLmKIGuHjeBeRJzV/eqq+IDmxQg3 X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR12MB7568 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 1/25/2023 9:30 AM, Hanumanth Reddy Pothula wrote: > ++ Ivan Malov and Andrew Rybchenko > >> -----Original Message----- >> From: Ferruh Yigit >> Sent: Tuesday, January 24, 2023 11:34 PM >> To: Hanumanth Reddy Pothula ; Aman Singh >> ; Yuying Zhang >> Cc: dev@dpdk.org; andrew.rybchenko@oktetlabs.ru; >> viacheslavo@nvidia.com; Jerin Jacob Kollanukkaran ; >> Nithin Kumar Dabilpuram >> Subject: [EXT] Re: [PATCH v5 2/2] app/testpmd: add command to process >> Rx metadata negotiation >> >> External Email >> >> ---------------------------------------------------------------------- >> On 12/21/2022 2:07 AM, Hanumanth Pothula wrote: >>> Presently, Rx metadata is sent to PMD by default, leading to a >>> performance drop as processing for the same in Rx path takes extra >>> cycles. >>> >>> Hence, add new testpmd command, >>> 'enable port nic_to_pmd_rx_metadata' >>> >>> This command helps in sending Rx metadata to PMD and thereby Rx >>> metadata flow command requests are processed. >>> >>> Signed-off-by: Hanumanth Pothula >> >> Hi Hanumanth, >> >> I agree with Thomas for the patch. >> >> 'eth_rx_metadata_negotiate_mp()' requests all Rx metadata offloads to be >> enabled, but at this stage if there is no flow rule for Rx metadata why it is >> consuming extra cycles? >> >> Can you update driver code to process Rx metadata when it is enabled by >> application (via 'rte_eth_rx_metadata_negotiate()') AND there is at least >> one flow rule for it? > > #1 What is the purpose of rte_eth_rx_metadata_negotiate() API if it is always called by testpmd. > We thought it was added so that when that metadata is not needed, application need not call this > thereby saving cycles/bandwidth. > > #2 We use this API similar to Rx/Tx offload flags so that we can set things up before device is > configured. We thought that is the purpose of having this negotiate API and avoid depleting offload flags. > > #3 Generally any new offloads added to DPDK would be in disabled state in testpmd and we would have > an option to enable it. In this case, testpmd is by default calling this negotiation. > > We can update the driver if the purpose of this API is clear. Hi Hanumanth, After looking the history of the API again, you may be right. One of the previous version commit log describes the intention better [1], because of negative performance impact of enabling Rx metadata offload by default and difficulty to switch configuration dynamically, there is a desire to learn application intention before configuration. Although I have some concerns with this API [2] it is already there as stable API. So it sounds reasonable to make this configurable for a test application, indeed intention of the API is to get this configuration from application and operate based on it. Next question is what should be the default value, I am not sure about it, there are only a few drivers impacted from this overall. For the Thomas' point, it helps to test the feature of its impact if it is enabled by default. Will it work to enable them all by default and add capability to disable it in testpmd, which helps to run performance tests also to verify the impact of the API? Thanks, ferruh [1] https://inbox.dpdk.org/dev/20210902142359.28138-2-ivan.malov@oktetlabs.ru/ [2] API does two things: a) Learn Rx Meta capability of HW b) Configure Rx Meta feature for HW Functionality (a) conflicts with rest of the flow rules that capability checked via `rte_flow_validate()` API. Functionality (b) conflicts with configuring flow actions, this configuration should be controlled by flow rule not with a specific API, although I understand the reasoning behind the API. RSS_HASH offload seems given example in the discussions but it is still controlled via offload flag, there is no specific API to configure RSS hash functionality, same could be done here. [/2]