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 51B84A0C4D; Tue, 24 Aug 2021 15:00:35 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C91CB406A3; Tue, 24 Aug 2021 15:00:34 +0200 (CEST) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mails.dpdk.org (Postfix) with ESMTP id 31A3F40687 for ; Tue, 24 Aug 2021 15:00:31 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10085"; a="214179075" X-IronPort-AV: E=Sophos;i="5.84,347,1620716400"; d="scan'208";a="214179075" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Aug 2021 06:00:31 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.84,347,1620716400"; d="scan'208";a="464424673" Received: from orsmsx601.amr.corp.intel.com ([10.22.229.14]) by orsmga007.jf.intel.com with ESMTP; 24 Aug 2021 06:00:30 -0700 Received: from orsmsx607.amr.corp.intel.com (10.22.229.20) by ORSMSX601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10; Tue, 24 Aug 2021 06:00:30 -0700 Received: from orsmsx603.amr.corp.intel.com (10.22.229.16) by ORSMSX607.amr.corp.intel.com (10.22.229.20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10; Tue, 24 Aug 2021 06:00:29 -0700 Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) by orsmsx603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.10 via Frontend Transport; Tue, 24 Aug 2021 06:00:29 -0700 Received: from NAM12-BN8-obe.outbound.protection.outlook.com (104.47.55.170) by edgegateway.intel.com (134.134.137.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.10; Tue, 24 Aug 2021 06:00:29 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=PPRVVlsuLlezooYONLRvH6WI5/IiyyVyfw2xylMygjunTKy7lVttD3uMXiARr9zhtZ6qDGTYI5KVqGPCSiB3Zl0+Vn5bLV5TZ+wrDy5739CZPW14tXlj+6itdUmFyaCDL2U+xg1EHEc2mmnAQzN4gGKhwJNQ7tz4MXN6pC+yM9f26Ebe5m43cSAL8Z9DJUpbGZ1imymcb625KYvg0EBHTbqw8jHUoFcH9kKX3e5dRLs3/NNy5bYoyQuDTtBNU6hpMWdYVjMMzlB3OttvX+8ne7rYWPTyFHmvjcHLlC+vXOr1+OCxX2LLy1MpmF5C8HSpFEm/fB4KOUiL3Gv34W98FA== 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-SenderADCheck; bh=nApxbGh59MgzfPhgV22WrYYo4Un2Tv7d+fubNrbl+f4=; b=k0HM59bhRfkYG+TV+7FxoXkRe6bzBGwsZ+sMgPeSwLcVeSW9nGy+zXDO8BZZY0S2+Q5xC3qMJzbsKgB2b7lXgOz5c4v8NZl9CPrsSfp++9owP5Rd6aL1hNwNQhF9gmBmILMPER3TMsq3buZTrcTPU5qMf6cWnAN+xxZ1yC7LYj3oQRZ3l9xJPiHH0eUml0yTSsIEAWli3wP8igw3uoCPMYTf+2Iy0MYMv+qu5SaMDXVvXCJQ8Ozose/pv0UF8IgeuzBuPjHdBWI4kT5tEWvyAscCJrYvBJBGConHVhYY/H6DnYk8qfV7K314FmVXD+GiJer8SYym2VXdET8mIwQtJg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=nApxbGh59MgzfPhgV22WrYYo4Un2Tv7d+fubNrbl+f4=; b=qTkB6DMK5ZmSSsTLXOigQeTydqf/kJpvRiuLuVhSIlC0qVXiLZMaCMwBlUxU5oqpRCAoScFUwM9ldbbeC4rFyAwRwRj/1foF6mAAGY4p7+7aMMqEviXFM7tGRYRIWB4O6IKxugKY3DMhtJ0GYKTxISqhA7CZtrssuw9tSKEoIgw= Authentication-Results: microsoft.com; dkim=none (message not signed) header.d=none;microsoft.com; dmarc=none action=none header.from=intel.com; Received: from PH0PR11MB5000.namprd11.prod.outlook.com (2603:10b6:510:41::19) by PH0PR11MB4791.namprd11.prod.outlook.com (2603:10b6:510:43::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4436.19; Tue, 24 Aug 2021 13:00:28 +0000 Received: from PH0PR11MB5000.namprd11.prod.outlook.com ([fe80::2979:70ca:38a:dbaf]) by PH0PR11MB5000.namprd11.prod.outlook.com ([fe80::2979:70ca:38a:dbaf%6]) with mapi id 15.20.4436.024; Tue, 24 Aug 2021 13:00:28 +0000 To: Andrew Rybchenko , Kathleen Capella , Xiaoyun Li CC: , , , , Thomas Monjalon , "Ananyev, Konstantin" , Jerin Jacob Kollanukkaran , Stephen Hemminger References: <20210430213747.41530-1-kathleen.capella@arm.com> From: Ferruh Yigit X-User: ferruhy Message-ID: <1da457e8-4cf7-289d-01da-d68b8b0e6cf4@intel.com> Date: Tue, 24 Aug 2021 14:00:21 +0100 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit X-ClientProxiedBy: DB6PR0301CA0083.eurprd03.prod.outlook.com (2603:10a6:6:30::30) To PH0PR11MB5000.namprd11.prod.outlook.com (2603:10b6:510:41::19) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from [192.168.0.206] (37.228.236.146) by DB6PR0301CA0083.eurprd03.prod.outlook.com (2603:10a6:6:30::30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4436.19 via Frontend Transport; Tue, 24 Aug 2021 13:00:26 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 89272e66-c014-43fb-d6bd-08d966ff2573 X-MS-TrafficTypeDiagnostic: PH0PR11MB4791: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:9508; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: BC+VtaCgf5bGbULatwZ5DyFdMCVnVohhqyXJ6h3xNUsYvQZAiQBM4R1IMKAG0e4wVtJqsesH4CX1nTCk4Gv/JA7eCzFy9UgyxLZI1UynLjw2VvvsyRlBRsCB25BylmOQhtaDqVBP0jjJI4LD7bHhgtePktdVB+AN9MN8kS6Xngn6nKLyXL3YqqkNn8Sj+nmpFwMmrz/FHXa3uynRi/EDiDQ1o4F1oG0u1y2lRE8mP01EMWgly8w8DxjpdCE2lfHc4H+OipFbYuTU3PlKinNUzOTFQcaRyTEk6ef5iE5LD3Fj0P1gd7V8ugv5ZELLaH8IYoFpkWXQla12uRYP7xhuJFilQ2H/EOMWdo5UXfwA7QUsdF82VjWONWOaX2FXzsRANaRo1lS182A7vNX6ZEnSHQLDdZYLUQjKdET0zL4nDD9PVXym414K1m3THBJkBcfClx4PQkoFYT5M0r1MW4vENhoeGGS8vxuPh3eg+8qlCwVTzYUrex13kHH4HWjxsVTbbAVRQPqRz8ue1NLbi6IoAP2wJgqh8Lgy7UQrrl6mhlGdHNU1YxunMzOJ8GEF/FPpt++OjEaNXdbsNaaL/NeOXR6KHbpwNK+7ItSbsvyQVTkyGzBkVL3m5L5BFF2k39nSYlIW9rwmogQrGg/kpjCMDddrwzdutppg20zRlaffvbewrGB0ddMFwa01GM4qR2TlkXBDduqYiMQ00Y87OMtWmBQyTaDC4RvMCPE0w8wUqHQlHFkwszsm2Ike+q38sfJ3ujfpvRDqI6RVVxnlPyT31g== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PH0PR11MB5000.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(376002)(39860400002)(136003)(396003)(346002)(83380400001)(478600001)(8936002)(5660300002)(86362001)(66946007)(186003)(53546011)(66476007)(66556008)(36756003)(6636002)(4326008)(44832011)(38100700002)(2616005)(8676002)(31696002)(6666004)(2906002)(16576012)(316002)(54906003)(110136005)(956004)(31686004)(6486002)(26005)(42413003)(32563001)(45980500001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?a3NPSEZHeHpXS21uUEhlellmMWZHWW1sb21uZjJBb21QeXF2ZFlybHAxYnNa?= =?utf-8?B?RFVtYWtwUzJWZldRb25sMWYvOFpKdU5tdFpRMUd2WWlZb1ZnQmJQSXNQUEc2?= =?utf-8?B?djJpU2dKVThBVjZlVWEvZXFibnpLT0RvUmMxenp0bEpNenRRYzRGd2RydFNz?= =?utf-8?B?dyt3NjJzektPMzI4TktwMllpT2xzUzc2SS9QdU44SkZ2YUU3YjYwY2t3R2dT?= =?utf-8?B?Y3IreXp0SHhTR08zTnRLOE1laDZZclhiNmNzOHp1TXVkWFR5QmpkSVM2bnh5?= =?utf-8?B?ay8zVG9WdGpIcFV4N2lONXo3dXUyR2ZvMXdzTGF2OGlNUUpGUG1GbTM1TEJN?= =?utf-8?B?L2c0V1FsRHpJM2JSOWhpanByWjByS2NQQnVib3BIeFlQU0NXTVdCeGhlU2k1?= =?utf-8?B?Q0MxR1g0T3U5bmdOSFhvVDVHRENubEhzQ3pDWHZENXhTVURGWnN4emVQYXhV?= =?utf-8?B?ZDAwQ1hFVytQWmJKWkYxQVJXa2hCa2lTUCtBVDM1WVB2QndJVVJOVWJQZmc3?= =?utf-8?B?THp1N1JGOWR5bnEyaFM5VnFaQ3kxVGpxZ0hpVnI1OS82bGhxUkdQZFJSekIx?= =?utf-8?B?Tm11c2Zqalc0KzJmMlVwaWJoeUlkbElpc0pRazZhTkwreEFTaUsxRHl4b2c1?= =?utf-8?B?UytQNEZUOHhSYWZQcWJwZjF6NFRhd2hhZnFnQ1EwN0I1ajdhWDEzMEZLTTYy?= =?utf-8?B?ZjRYK0ZDUFUwOHRMSlRDYlZmL1JiREd1b2RsOTVYNEZzdkMvY2NFaHJma3JY?= =?utf-8?B?eTgvWE85di9ibGdwR2NIcUpveE1KZlhNMUVHaDd6NkNla0NIQmJkemxoaFdV?= =?utf-8?B?TVFXYVlEVVppbGxaaXdyRU8yaGdTdk12b3F4TC9rZnRXaW9MbWJZSWVRYTRI?= =?utf-8?B?T1BqQURHa2xXZEhlaEFwSzhpbU0zend3SmpDZ0ZDOFhrb2QyMVZKR2E1aElx?= =?utf-8?B?blNVa3hnWGZXOUNmS2wreGUyR2NNc0NDbDFZTEZ4eXN2cWx3aVZIZEtoTUVt?= =?utf-8?B?V0pUcUVBZG1LUkMwNEx4OFZUQ1R0UGNrNkM2dDdMYnVwb3NYQUt2SmxKSUZm?= =?utf-8?B?dmh5RkNxKzN3OFVPN0p6dlpmMTZTUU43Sk84K3hpM2pVeFRVRDdsc1VLMkx0?= =?utf-8?B?VUoxM2FlMmJrYjZLNTk3dXpnOWkrNUYvUVR2VW5NRWtKd1FYeTRyY0xWd2pN?= =?utf-8?B?cVBURXlCd1BqUlQ5MjdwdmZqVE05VEpFcHYwaEMwQmwydTBSd0F1ZjlnVmVs?= =?utf-8?B?RUJBYW81VVZ2QWFqTktJUjdzaEVEMkd6NlBPSWVNaXZUQXhkNStYTXNPeWZK?= =?utf-8?B?bDhxRFFkOFFjY3hHOE5jSmZpYkV5ZzdrdUtSbXc0RXFpNTNaTjJDM1J4UjRs?= =?utf-8?B?T3IydnN5QTErd2hOOVkxWnZPeXRVc2d3OVFjTTByN1pQOVFqbUx2UURISjNq?= =?utf-8?B?N3FyczdyMm9Vd3ZHdU9KYzVQb24xTWZZMFRRejd0a0NDdExzcHR1VzVtN1NM?= =?utf-8?B?dnpoWjZ3eHQzL0JjK3ZmZWhrNlN0OWtFU2N1SmFWbDJWMGV0NVY4N0t5YWRF?= =?utf-8?B?cUhocUVtdTcvd3Y4dFpqVlVhSGNqam9TTkdWaFZnR3VtYnQ4VHlYbEdscGYx?= =?utf-8?B?WEY1a0pyU0pHRVJRVFJ3YjRYMkJiaDBSNXFUNGJGY2pxVFN6RFAzdHk4SXNK?= =?utf-8?B?VktrZFZ4VE9OcVQ1eUVyM21TNi9vMjA1V3VEN1hlK0pzTjZjNzBiWERpdFZ6?= =?utf-8?Q?DOZvtdRqMjZq7r76w8Bpg9tMLWQvNKBld+ANkuE?= X-MS-Exchange-CrossTenant-Network-Message-Id: 89272e66-c014-43fb-d6bd-08d966ff2573 X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5000.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Aug 2021 13:00:28.2500 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: XVx6WckSi4ne76khJFSEppHdU1/ONX0mVZMNe0r496arM46DkZKYcXDcggqDK8HZqc4JRDWBVsyAWJkK8neK5Q== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB4791 X-OriginatorOrg: intel.com Subject: Re: [dpdk-dev] [RFC 0/1] app/testpmd: add l3fwd mode to testpmd 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 Sender: "dev" On 7/2/2021 11:15 AM, Andrew Rybchenko wrote: > @Xiaoyun could you share your thoughts on it? > > As far as I remember there is no agreement on the topic. Adding > more people in Cc. > I was OK for adding simple l3fwd forwarding engine to testpmd, to benefit from configuration/debugging/measurement benefits testpmd brings as patch mentions. But adding neon will likely bring other architecture specific implementations, and there will be more code duplicates, which is not good. Also it is possible that people may want to add more lookup methods (em, fib..) making things worse. If we get the feature, what about limiting it to scalar implementation and LPM? Still it is questionable to have the feature in the testpmd, but at least we limit the scope. For performance measurements can use the l3fwd sample application. Another point is testing, this feature should come with dts updates to test testpmd l3fwd, otherwise it may not be tested and turn into dead code easily. > On 5/1/21 12:37 AM, Kathleen Capella wrote: >> Performance of the LPM mode in L3fwd example application is used as an industry >> standard to compare between various platforms. >> >> Unfortunately, L3fwd example application lacks debugging capabilities to >> understand the performance bottlenecks and fix them. >> >> While debugging performance issues we need all the flexibility possible. >> Some of the capabilities we have used are: >> 1) ability to print hardware and software statistics - xstats, stats at >> port/queue level, burst stats to identify any headroom available, >> CPU cycles/packet etc >> 2) ability to modify all possible configurable parameters for the PMD >> as well as the application at run time without recompiling the code. >> Some of the parameters we have used are RX/TX queue depths, burst size, >> number of receive queues, PMD specific parameters etc. This >> configurability at runtime helps to understand and debug L3fwd >> performance issues quickly and effectively. >> >> It is possible to add all these capabilities to L3fwd example application. >> However, doing that we will result in L3fwd example application losing >> its purpose (of being a sample application). At the same time, testpmd >> application has all these capabilities to debug an application. In my opinion >> it makes sense to add L3fwd mode to testpmd. >> >> This patch adds l3fwd mode into testpmd to take advantage of the >> existing infrastructure in testpmd. >> >> I'd like to hear from the community if the structure of this change makes sense, >> namely, adding l3fwd as a separate fwd_engine into testpmd. >> >> This feature is not yet implemeted for SSE or AltiVec. >> >> Kathleen Capella (1): >> app/testpmd: add l3fwd mode to testpmd >> >> app/test-pmd/config.c | 66 +++++++ >> app/test-pmd/l3fwd.c | 356 ++++++++++++++++++++++++++++++++++ >> app/test-pmd/l3fwd.h | 143 ++++++++++++++ >> app/test-pmd/l3fwd_common.h | 268 +++++++++++++++++++++++++ >> app/test-pmd/l3fwd_lpm.h | 107 ++++++++++ >> app/test-pmd/l3fwd_lpm_neon.h | 169 ++++++++++++++++ >> app/test-pmd/l3fwd_neon.h | 234 ++++++++++++++++++++++ >> app/test-pmd/meson.build | 3 +- >> app/test-pmd/testpmd.c | 4 +- >> app/test-pmd/testpmd.h | 20 ++ >> 10 files changed, 1368 insertions(+), 2 deletions(-) >> create mode 100644 app/test-pmd/l3fwd.c >> create mode 100644 app/test-pmd/l3fwd.h >> create mode 100644 app/test-pmd/l3fwd_common.h >> create mode 100644 app/test-pmd/l3fwd_lpm.h >> create mode 100644 app/test-pmd/l3fwd_lpm_neon.h >> create mode 100644 app/test-pmd/l3fwd_neon.h >> >