From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM04-SN1-obe.outbound.protection.outlook.com (mail-eopbgr700065.outbound.protection.outlook.com [40.107.70.65]) by dpdk.org (Postfix) with ESMTP id D73434CA6 for ; Tue, 19 Jun 2018 16:08:08 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=xBqTMM6qLgQXCHEZ7Q2YvOWJRLln4B+awobnKMAhu5c=; b=aptHzVf/pJvPHzOZYZ+NuMBN4AWngow2yWDjN7CUJvm7c8SBsE85Co+amq5WzGH0+FjdWyb6zY/xUEhusSr6nRNOixZULtMR6EWUJ1zXnX2k9cjr2ojIKgwcY0hzoLAz7vGVeEZMvg/rn8TEc1TvXSQ3I3PkfGP/uYlF54KUfbU= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Anoob.Joseph@cavium.com; Received: from hyd1ajoseph-dt.caveonetworks.com (115.113.156.2) by BN7PR07MB4897.namprd07.prod.outlook.com (2603:10b6:406:ef::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.863.16; Tue, 19 Jun 2018 14:08:03 +0000 From: Anoob Joseph To: Bruce Richardson Cc: Pablo de Lara , "Jacob, Jerin" , "Athreya, Narayana Prasad" , "dev@dpdk.org" , Hemant Agrawal , Nikhil Rao , Sunil Kumar Kori , gage.eads@intel.com, harry.van.haaren@intel.com, narender.vangati@intel.com, "Bhagavatula, Pavan" , Thomas Monjalon , Akhil Goyal References: <1528477766-15788-1-git-send-email-anoob.joseph@caviumnetworks.com> <1528976946-14396-1-git-send-email-anoob.joseph@caviumnetworks.com> <20180619100939.GA23348@bricha3-MOBL.ger.corp.intel.com> Message-ID: <827adfdd-602f-a70c-fe9f-2c821ceaafb3@caviumnetworks.com> Date: Tue, 19 Jun 2018 19:37:48 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <20180619100939.GA23348@bricha3-MOBL.ger.corp.intel.com> Content-Language: en-US X-Originating-IP: [115.113.156.2] X-ClientProxiedBy: BM1PR0101CA0049.INDPRD01.PROD.OUTLOOK.COM (2603:1096:b00:19::11) To BN7PR07MB4897.namprd07.prod.outlook.com (2603:10b6:406:ef::26) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: cecee199-35d3-4cf7-eac5-08d5d5ee14e7 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(4534165)(7168020)(4627221)(201703031133081)(201702281549075)(5600026)(711020)(2017052603328)(7153060)(7193020); SRVR:BN7PR07MB4897; X-Microsoft-Exchange-Diagnostics: 1; BN7PR07MB4897; 3:1CLStw3AW94WaEMGoT/WQ3paO30a+L97U3PbiaaKoVIlqx8Mq0YLOxK+Cve1qsLm6GQBr8fBpras4N8f83TdBH3xH2W2oIWM4bxZ5tltiiP8cRzY6kbbNzKdYkAANSzs4X/JoZuPaSs++N9yrtRLEJpQ6HO/EOpsCFIM5XkAhpWfMaHIlQK25VznVEEpP12GLa6Y6UKM1sPa4gxQ+gNEiBQrj25GJzPaztI54IRPBIhae4uSdkuZrPgF/l1hGRUU; 25:2U902F0v45P5bVCsXzdJK30fsc5p3OYxrZa/Eh5FHVTBCRwBrVf2U7qqd9aFivnvZsf6bon5J+XBbYaYRJwCQK4fG2NxlkIRAZilIJYdZ9aH/5o/UMy3NGaCcQACHbh0yC1r/rf6wT1opnOz6zNjqox8/BjD/dH8Wjoh+/zYStFct/cTEubHzAXRVaZH/8qEOs8JzkAimtg5ReK28hG9Ng32cS4m9TpLZ+nzdajNiYjz48QfiNTnR32NOR6NYz8wKMBAAoqMaUISuEKWf5DUrZUm9Bt2vl2G6RRR9nsbV4oaNEj9btCg9mEK7NAFmBCSsB2RDU5uqEYWhIhR3eCT6g==; 31:1wb3MqY6Hkt793QXFqNtGOEP+6wzdtPrGSqTPBFR2PY5P5Tc2fEomNp8rUZVc2hdMWT6li9NhSKLyaHmwAGFDT2TydsyPLVzPrGCoX2BtkT80MaZB2df8OFXakJ7Havs+bEwvsjoLxNDvWJ6PDs9AxuWPRGyIvpjicIywtycvHbU2i3AIdr4J85mLRE1h684SbpF/r3O4RbYvH8488LMmKOdRc9Ar7ldi9Y+ilJejMY= X-MS-TrafficTypeDiagnostic: BN7PR07MB4897: X-Microsoft-Exchange-Diagnostics: 1; BN7PR07MB4897; 20:iG1OaAUSpt4cLqGESMFSYjY8C9/E0c0X11T/3vH9DHYZ+ehOb36BaFYWakIddyDSwZ09eRgIuAgF6A4XHGvvKkdvvCxMRut5A3mr5S87AGUF4yZ68t57j0qXGciqQLr5GbhMnHLKWB89DDqczCsWWIw5QFkl+ih/O0QYSiSJZXWBErJ4TcZ+PmNWt7zn5esCtSd9RdF3M3VTH2TSRUOarw49FTH0gJ+gdxrttP4b/NZryGMsi8dIQ4zYfGhilKdO6QuzKb5HIzgfjHlhE9T/Y4XOz4z+DbR+5KQ+yDpmeNUBDAtWlNQxn+Bd8Web/wo5TXd/Yxctbk9s1M1LpZfjs/RCyb6sFkieGtwveMOxjbYAmbvIEHfqM6EnYPSGOjYmOVl5dzZNEGmwnOE8W7YgnRFtnHOK+LGvqHUUtt8NDtKMFx5zF1Xll/vgny2RAlnYs81CoCYLfuMsVS8ipkmKlXAH3Fb0jkZjqoJEX/Jf7FgB7+dTudLZXr7Slid/jsMlwIx9qH6VBzvaiJMKah7xKCGNWQUw4H11D0qcQc5qLzFG7gSFODk740eB/RPibM6VwsBRiqVo1NgNIJkBEpbQKHLdgeUBG798qf94FArMEh0= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(278428928389397)(185117386973197); X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(10201501046)(93006095)(3231254)(944501410)(52105095)(3002001)(149027)(150027)(6041310)(20161123564045)(20161123558120)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(6072148)(201708071742011)(7699016); SRVR:BN7PR07MB4897; BCL:0; PCL:0; RULEID:; SRVR:BN7PR07MB4897; X-Microsoft-Exchange-Diagnostics: 1; BN7PR07MB4897; 4:kqMvMWBF5qv5jTfrHy5i+6IV+fZbMJal8xtMVm5Trq6xTT+jU9HONop8HpQLvRtQ3hHxrZA6dppXuuyoAm2kxJRJ2m9WYJ1dNE03FykFO4kK+QmErYI7PT2JDWVTtermUC0aOXR6bwIBhoJY+onvBBKY9kFj8NQs4MBhpTEVUZKePhPYaDJeML00+tZN25hZjZbPKDgRZOAHGImH5nUjcOkCQkpUqe4nQhtHQ13XdXpBsh9AUQ99PYmvRnJIiG5YolI3iRtXwm1x0yB0qh5bH2pAYgegQFGn1oO954J0cr17/4ZaxA8GWSO4FP/X3WyVbSy9d6X9Bfl94GQmriil+6lbpfmytBZH1AfnHCMTSZc= X-Forefront-PRVS: 07083FF734 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(396003)(39380400002)(39850400004)(346002)(376002)(366004)(51914003)(199004)(189003)(106356001)(68736007)(7416002)(42882007)(65826007)(31686004)(105586002)(2906002)(8936002)(386003)(6916009)(6506007)(53546011)(76176011)(5660300001)(53416004)(8656006)(3846002)(6666003)(65956001)(8676002)(65806001)(6116002)(69596002)(19627405001)(478600001)(7736002)(270700001)(64126003)(84326002)(66066001)(97736004)(25786009)(31696002)(6512007)(72206003)(16586007)(4326008)(6486002)(229853002)(81166006)(236005)(37036004)(186003)(606006)(26005)(52116002)(55236004)(16526019)(93886005)(956004)(476003)(2616005)(966005)(53936002)(486006)(54896002)(58126008)(446003)(316002)(36756003)(11346002)(6246003)(6306002)(54906003)(81156014); DIR:OUT; SFP:1101; SCL:1; SRVR:BN7PR07MB4897; H:hyd1ajoseph-dt.caveonetworks.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BN7PR07MB4897; 23:38dxUw/VkRiljIwuyFYIGh4iHoLwg5grDxSAxavs/?= =?us-ascii?Q?yZ6wEUNfrZr3q6JbIzzkpYYAzYaxbEmlYVxq/zKZnr29jbfF94f6YjRS0O72?= =?us-ascii?Q?Kd3n4HGHw/L7bYO9ylX8VFreNaRKGpZ6Vowyp7aQBgKfZ89q03d+2bl6UaNP?= =?us-ascii?Q?1gtVvmGqEhnyRq21lH3mQmuIAEeMW/CE+dwWtrTEVrxig220WTE75JLzM5pD?= =?us-ascii?Q?W7uhAJcAF9k6IKVIDAjWDCG1YJHrG762b59MA4c42YOV897VoKb35lbkyykc?= =?us-ascii?Q?ZKQ0lM4wxSIfgnVLcRDu9cawqGI4sDqK1z0EEY0F0UTv1Z/KuzyAaz34tR1R?= =?us-ascii?Q?pxTznNNMXIfmNBX4ZUEUU23/s0FoQusw9gaCaRArdOVLJNQqdQdTnOgkdP1j?= =?us-ascii?Q?QZF8+dqaUgSfgRiYNAROnCFN6/KGwwytcCiSbqe77zoenZBNFrQ1BmwGUVlh?= =?us-ascii?Q?0D1lYBC6PU4wrpjKFqXSytjLCYUlHcP+wj4J2YZ+3zY9TMalN/avtF1nPPbu?= =?us-ascii?Q?X0OvMvpBUaf2obBVRZS18iR5JUjSXdRhz8aW1uD4GkaTtSd2Yu2BJxxYR4BS?= =?us-ascii?Q?/gO9hN30pYOVlkJ0cBKV+IWBpZ86Va2tqK4XcrlqDW6g3lJgKHVbz49pQI3e?= =?us-ascii?Q?vsCojeEvggEesR0hCPnCdx+6dofyQvNfbkNKUjdwnE3WERXVU2b+oWnvT3oY?= =?us-ascii?Q?AWcBFbc0c4S5s3xAJME8JEbKkdFt7qz3Na4WqBSQ2/89QRmO/8Rmog0EcBA5?= =?us-ascii?Q?nbIjPNqyrccJ0ACAYA9jQDUlbxAhayWa7u8/seUeZbYXFPcrAlZ2oP0Bfc12?= =?us-ascii?Q?Tps5xEnZWi6x2YySn523Dal8lIx93buN42jVaP0/G8fXtgs/o9X54HLTa1aA?= =?us-ascii?Q?X+pIPW5+npAEwaBqYl8M+pev4mBWFX3zrvOx8/lWCLuj68w7R5h5j2RuwhsN?= =?us-ascii?Q?rgVicnd28xRjSscrmHuTyJ/osO2WcisPGbrWoJqpk5GBBBV40nVHInMegBbY?= =?us-ascii?Q?gmo+8D198tCSOoDVV+JLZHW+n6WeB/MzQvoEYV384z1V2Br/7uriynak8Dwl?= =?us-ascii?Q?4baygQqcGtMeZ6FN56BGEECsOc/Pc5ltcUvMn5ojbxePeCXR9ahV1yUMJaRg?= =?us-ascii?Q?NnRlB6Se60m9ucTNjLlAQqyeFB6GG24GhaVxHpgluFbs3TXNNoff7cA8J/u1?= =?us-ascii?Q?0sUuWBZfJjKhFDprzZoYN+2L9QBt6dSGqEi5pI9y87iOznBcbyHtfUcj1HRt?= =?us-ascii?Q?q0Wgnd/GFM7v8JN+00ABo6vcdYccKiZoIHDyZlyY2JrkMVmi2waudmWZxKC9?= =?us-ascii?Q?AqbWtpTItWM7w9MWnUOCIGMe4QL4DAjpU/nXPgsV4DLM0YahTVgG9Xbuwlym?= =?us-ascii?Q?2BmoOkKlVu/mLMbrhvtvUlcs1yLyXCmjJZPCD87ggJreZyAixiWUdNv5zG98?= =?us-ascii?Q?B3z60x9/nVA+NzdnvGQaWL0j8QPx34RFJPD/E7VN2hgjwHlAYW3vZqfuDG0X?= =?us-ascii?Q?UBFz3si+zBg3DyjnPYR92eiFgfXN8R/aO8ucPYB7KpFE6VyVG3VQKxs9XfeC?= =?us-ascii?Q?Te9jc/RX4ymKd/LbY7cSKgYL3T9iqzc5Br+S17/CmUEdHANscSGTXRSLAdUi?= =?us-ascii?Q?mfcM0RPhxiLmqawuqLBtP8VjtIE7JsGucVi9KeWcR8=3D?= X-Microsoft-Antispam-Message-Info: aB6wPwlZHPOy/1TZIqM0vvVPPAgttGvd3d9sIQcIwSzW6c1x2dLfwX8jMpXZsThqfjkYCRuDvngCq+uSeVjrpkdcwyiS9efTBRH6K7f+dHgnImTIrEZusiOhiJuiMS3q2dhhwcUb0hHoP4DRlwDhqLgXeDMryjxJZcUTq+nsyoNnkK9BEcTJDjYLRJq/5TKT08xDYjvZzbzTEi/Ug0wOG4ven00DD2DDsG2bbmXhae5Km0rrXbQJgNLSzSGEsOGEvmTGrJSC2XYKoPmbUkEEtO1ZSXAB2WHy3aXiw0ObqTcGFi+Y21LJnnjobofDvNp6jtak77/ziyx8MWMU9zd4kQ== X-Microsoft-Exchange-Diagnostics: 1; BN7PR07MB4897; 6:qS3dUMfkPlO9o08CKZ2f4EvQ24d8x0Dpwc+Y0UdviNDXIkP0qaITuFGrflhXgNFcg6dClBGkKmx/GSHeLni5xaTxlkWS6ujRXCmmkp9xRBg56dLbFZRM1juocy51P+AQkAU8D6n65l3RO9Z4VA+EBoRGmIPM2Qes97Z8jOV96caK+1yxD2Zq14m2/jQgQr5aquzFDv0UibTJwCTEOUKtT4oIyLtxjU+dI4vCJosz6a74Q2YtYbIxXtVXCJsL4bWgCyyPprwqNGMS3tHFIRjwlVfE1MBimKNmUeCWQwkVMgYFsyEE6+aOizfz4JO+/eFNoOOvMgsvoWDZl0snjlSjd8pdWjQrccjDx8nZl741yjpJCLTo6f4ag57F3mb/pSMlGqjZMVGDXjYG2gKvxJHSZeDrk/6GPfQz8f55SZUaEvUTtvfVbx/mNCq8fvPGoyJ8aXJ6w/kuziIcwkwYIQhZRw==; 5:aneTAS0qvTZvQjn9ljzbF8dlb1IHu/OuybcOEuBgMz4cmC2U4f9/SFhxiBXHFvbiwQfnFgrqNa33aR7i6bG9P5SpYKv6A5m9oq4ayiLyUyBghCseSkdDJqSDl+2OOUkGR94Rg4k/cXW14Sf3AL1gFWUuwtT2UJvSUsSCwcWeRjE=; 24:g3a8Scm4M9Co0GsgxOc1zit17qgsmwJLoKljYBFI7pI7cjlOw5pJsno7GjcpsCWc7kJTZoKCaIbuWkyBNVr4JC4hKIJrFk+cjGHVC+BghzU= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; BN7PR07MB4897; 7:PRoZ2rG2zasef48tuCXX/FQkJL64KxJYtDQZHvD6u9efobn+3LvsQcepCO2Nr0zppUJM9T7KPUA0eO/Y+vjhjpHQWyVEW2+fvZm6oBINu3F907VLBpBaHaOjLhYq6Li2OQAeqiixHgXjOPcvEDLXsSF3CiKzNW3BgmtoWFyZvJnbnwXWlvn2+l14Ype4JbnlscPlhe9bL6DwIR7myJTEXMZ13rJSyguZs3wrbNM7005cHueRtfiyX5DSvKa4CqnD X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Jun 2018 14:08:03.6175 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: cecee199-35d3-4cf7-eac5-08d5d5ee14e7 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN7PR07MB4897 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH v1 00/15] preparing l2fwd for eventmode additions X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Jun 2018 14:08:09 -0000 Hi Bruce, Thanks for the feedback. Please see inline. + Hemant, Nikhil, Sunil, Gage, Harry, Narender, Pavan, Thomas, Akhil On 19/06/18 15:39, Bruce Richardson wrote: > On Tue, Jun 19, 2018 at 03:34:29PM +0530, Anoob Joseph wrote: >> Hi Bruce, Pablo, >> >> Any comments on this series? >> >> Thanks, >> Anoob >> >> On 14/06/18 17:18, Anoob Joseph wrote: >>> This patchset modularizes l2fwd application to prepare it for eventmode >>> additions. This patchset doesn't change the code flow or logic, except >>> for few minor improvements. Some of the newly added functions are used >>> in just one place, but is added for efficient usage with eventmode. >>> >>> v1: >>> * Fix all checkpatch reported issues >>> > My main concern here is how much this eventmode addition is going to > complicate the l2fwd example. l2fwd has always been a pretty basic example > app to get users started on the basics of DPDK use, and I'm not sure how > much we want to move away from that. Is this eventmode-l2fwd better being a > separate app, to allow l2fwd to be kept as simple as it can be? > > Looking for more thoughts from others here, since it's a community decision > as to the scope of the examples. > > /Bruce The eventmode helper abstracts most of the changes required by the application to run in eventmode. This was taken up following the comments on a patch submitted by Sunil(sunil.kori@nxp.com). http://patches.dpdk.org/patch/37955/ With eventmode helper, an application can be moved to eventmode with minimal changes. For l2fwd, the key patch which enables eventmode is, http://patches.dpdk.org/patch/40920/ [The aforementioned patch is dependent on this patch series] The bulk of the code in this patch(40920) is adding multiple event mode worker functions.The existing init code and poll mode worker is barely touched. Multiple workers were introduced because a single event mode worker would not have made the best use of the varying capabilities of event devices. Single event mode worker could've demonstrated how minimal the changes can be. But the ability to register multiple workers, fine tuned for varying capabilities, is a good feature to have since it will enable applications to utilize the full potential of the hardware. Eventmode helper patch series: http://patches.dpdk.org/project/dpdk/list/?series=61 The rules that were followed while drafting eventmode helper were very simple, 1. Move any code common to multiple applications to eventmode helper 2. Expose all capabilities of the devices involved (event & eth devs) 3. Minimize changes to the existing code For l2fwd we can opt for a new eventmode-l2fwd app, but this might not work for more complicated apps like l3fwd & ipsec-secgw. L2fwd app will stay the same even with the eventmode additions. It will still be a quick-start, easy-to-use app. In addition to demonstrating DPDK, it will also be able to demonstrate how easily an app can be made to run in eventmode, using the helper functions. With more event adapters getting added (tx adapter, crypto adapter, timer adapter etc), the helper will prove useful in abstracting the complex configuration options exposed by adapters. Similar changes would be required in other example apps, and the additions in l2fwd is to finalize on the approach. The current patch series just re-factors the code with couple of patches fixing preexisting checkpatch issues. The rest of the changes are split into individual patches for ease of review and testing. Hence the large number of patches. Thanks, Anoob