From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM04-BN3-obe.outbound.protection.outlook.com (mail-eopbgr680044.outbound.protection.outlook.com [40.107.68.44]) by dpdk.org (Postfix) with ESMTP id B64291B09D for ; Mon, 17 Sep 2018 16:50:58 +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=cOMnRUa4pVjVBFfF2OpoldBxgNAHf+NT3A9u1xrbi28=; b=fecPic6fffrKYZr0QgUPKtBrToCaUOSQfyxonIIrvUCCt7D8/k4AdKBxuWSY/aJWjI8+txjTzOUWdKjqkPlEKfNhz013lYhwVYjFzPNKFsQyaPuZ1Ve6Tfv65Uh9lMFXuAItwCEDzMecrtNHklIkwddiLKyGu3IBPHIrA6jT4AQ= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Anoob.Joseph@cavium.com; Received: from [192.168.1.9] (223.230.22.245) by DM6PR07MB4906.namprd07.prod.outlook.com (2603:10b6:5:a3::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1143.18; Mon, 17 Sep 2018 14:49:31 +0000 To: Thomas Monjalon Cc: dev@dpdk.org, Akhil Goyal , Anoob Joseph , Pablo de Lara , Ankur Dwivedi , Jerin Jacob , Narayana Prasad , Murthy NSSR , Nithin Dabilpuram , Ragothaman Jayaraman , Srisivasubramanian S , Tejasree Kondoj References: <1528476325-15585-1-git-send-email-anoob.joseph@caviumnetworks.com> <06b2d39f-5d9a-d1e2-f3bb-5238e40e6406@nxp.com> <23e3b8a8-32f9-cd91-bef9-c03eff0d4e46@caviumnetworks.com> <4535314.c5AgpL1HhT@xps> From: "Joseph, Anoob" Message-ID: <394b9be9-fa59-b5fc-08ec-2736cd1bdfcb@caviumnetworks.com> Date: Mon, 17 Sep 2018 20:20:24 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <4535314.c5AgpL1HhT@xps> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US X-Originating-IP: [223.230.22.245] X-ClientProxiedBy: BM1PR0101CA0059.INDPRD01.PROD.OUTLOOK.COM (2603:1096:b00:19::21) To DM6PR07MB4906.namprd07.prod.outlook.com (2603:10b6:5:a3::11) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 37854b17-1b3d-4cfd-6678-08d61cacc9b2 X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989137)(5600074)(711020)(2017052603328)(7153060)(7193020); SRVR:DM6PR07MB4906; X-Microsoft-Exchange-Diagnostics: 1; DM6PR07MB4906; 3:SNV8/fCPjy8LhKA6m4VHcrQkCem3Wre4BkgiWicdA+OcclCm/5lhX8WPHI9eOvYDWNgTv8G/FijvD9qgOKSZKp0cXdJzRA/RfnjWyGIDuZxvtv/qVA+9/Gr/SmG0FtxHjJbBSSB7O9gexphUNnbDYH/vy9pQ+yuR5adLToiZHTVmaHAusfQZVpL5JexD3VVJ8/sceyG6ueQaDjfyzs9Aoy+R5iXTFs6/F6oAN3aUMbsJaM7uBLrL8n069X3UP1Q+; 25:1h1xinDQ14ZGxuU+qn2Mw/dvyYHaA9f7jee6M21BCjwuDkYGJcXQqsgK7rKhmaQOLBBL5rNzTvsOnkXD9ke6N5XekMQ6VgwwpuVCKqamw6NZ74YYPUUblQVkh4fgWO5+hsxyISxjyi6nyV7OEYfsEPtdSU9SKIq2E7u09l2xhjJRIRZufSxLkl6zgwFixaPWJeTGjcHAFSXg+UNdkDifMetHoYsda5zYNw9z/i5JEX3OKGyAgIbMTAZrfX+K2KWcuHeBDrUUwjuoWCMbpb2/DhI/RsVFmeLYGA0ieGpbl6duhu6IE8JijtbUPgwJzr+c52rnbDAsejpUi3GnIE+W2Q==; 31:+yZKflnAwiAaE03BlgDGMZXxVALdXu7W1/tLWInD9hVye8og5CU4nqZsv64VJ6P8HeUcmY0fCoB8X6Xv2cvUC6i3YW73MWqcjdV1ScepLG9wVEH9bSpv3ig2SnH/tvbmwHw49BTMNiEHMJOPB2/630cK9UmnlFYh/tbPceSUFlaJeY8OmrT8DGjWRbwqCr2ei4MP2RzychCW9ZjDOAsK4w+vhrEemR9F/yYjfkgrI6E= X-MS-TrafficTypeDiagnostic: DM6PR07MB4906: X-Microsoft-Exchange-Diagnostics: 1; DM6PR07MB4906; 20:WlFZEe11zmmuLH2w+kETNDcsD/BzI9N8AUcTebAzD6rjSH90NfpI+aGVAxvDLhpwJCZ9RFk7So0Ne7pVKuupeji4/oef6PslA+FXeJaKDCpSH7JZb+lkJ5d224dJZBDmxIrObbM0BfnVlMWPe7bdOzraV56NOW6M3xwcW/bgptUCYh39j4hafYhQrm4ilzIS7PG64fgomFkfO4CYRzwGmJ/eeCzq6Wh9844Qd00mQYVtAnuoQxREfKFtDCoBhsu2bUoCtH+jYzxSLpkzXsvyEOWGV4HnDYfwWbrj8/TbPNuTvQ1sDQFOKqH7rbQ7P1nr8y+/V1NjHTRTEqfBFKas/6hnt9F+O+LxD8XfoljNXec5nzsyfbx/hCjHMoOdTG8BEoVbmOklLA9CavGq8tqSz6R14gcX8rsbbtQKFpB1QrBXAmuBp0pnPni+hRCxB/S7f8H5maWRzF4k75DvQViaB7lhCQRpswcIv1paMce6WZBPcN3DdqgyBvPu6kJ7qTUpQzn2U1kJXZFRKt26EKo+gR/xYJImALP2v25tDPvs77fVSJv9n4c57oTxUi7zYByZiYNNi37r8HfXRjxmDNeTlja2dPTPnUVc4uvYlHKJFvs=; 4:Iz2zNWzrRRRmPBBtrt+ywcqxrFvEuLZ9Ibg1Q75PxbnMC2oOyN0yCOSJ3rZxOhwCDVBRHKBXrVZf6gqmEE+/AfPNwF4At3YKn62DrB696LHdQ7gTTY8usHb+v8sAwEeZQ5/5JdNzAKRalRc6lhn9Qv3nGspiSFhKT+rojjm6FD4xhk6NGVKwaDB2yt8CHE3WzS1KHIvNPWS7J1xaX9wEXMahnKGv55+sGzIOM6KAEhv0Ty9xL023AFrtmOJNRivqemXhyllxxNMWi03mUbFxnQ== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3231355)(944501410)(52105095)(3002001)(10201501046)(93006095)(149027)(150027)(6041310)(20161123564045)(20161123558120)(20161123560045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123562045)(201708071742011)(7699050); SRVR:DM6PR07MB4906; BCL:0; PCL:0; RULEID:; SRVR:DM6PR07MB4906; X-Forefront-PRVS: 0798146F16 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6049001)(136003)(396003)(366004)(376002)(346002)(39860400002)(199004)(51444003)(189003)(58126008)(52146003)(54906003)(26005)(53546011)(486006)(386003)(67846002)(52116002)(23676004)(2486003)(76176011)(446003)(68736007)(64126003)(97736004)(2906002)(11346002)(42882007)(53936002)(93886005)(66066001)(31696002)(316002)(16576012)(65956001)(77096007)(956004)(2616005)(229853002)(476003)(50466002)(47776003)(6486002)(65806001)(16526019)(6916009)(36756003)(6116002)(3846002)(31686004)(6246003)(107886003)(106356001)(5660300001)(478600001)(72206003)(7736002)(305945005)(6666003)(8936002)(81166006)(81156014)(8676002)(117156002)(65826007)(3260700006)(4326008)(25786009)(105586002)(230700001); DIR:OUT; SFP:1101; SCL:1; SRVR:DM6PR07MB4906; H:[192.168.1.9]; 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: =?utf-8?B?MTtETTZQUjA3TUI0OTA2OzIzOlJRd3R1YVoxR0xlYkFld2ZoOEwzL01tYjFD?= =?utf-8?B?ZlFqRG5tcEJ4cEpFemN1NlJ6MkZxWUQzWnphdlhqZGxyRUtBSEJzYkEwb25y?= =?utf-8?B?bi81Vm1aWElzOUlKUTNTUnNTUUJBcllua29md0dKeFdXZmxLOStnd3VZUmhG?= =?utf-8?B?WUI4cm55TlRxNFc2b242T1czQW5FZ3haTUNsOTh5NTVDa00zUExQWmN5aWxw?= =?utf-8?B?K0krTUVuQ21Nd2hyaHBTOEd1bkIzS090Q0M1YjFMeGpqRjNnWkJaNzM3cHFp?= =?utf-8?B?ZjNMbGpIWnJMZ0plUnlITWlvTmE0cGg1Uk1LWmR2Um43OVF2RGhELytaR2U5?= =?utf-8?B?TFdnTGpGR3ExUjNkYVNDUkIzSjJaQUJRVXJyQzdLWmpMZzBvam1xQ0VYNnJm?= =?utf-8?B?NGdoalpOcm1VbmdVY0lpRVBaZWJjTXNuOThJcmpHQmNTUzdUM2hPMi9VY0RK?= =?utf-8?B?aTlVd1VMK2MwblcvaVByZWlOVzk5ODJkdTF3Mkh0aDRPeUJ4QkJ4eWI4MGVC?= =?utf-8?B?MjVEQ0haOE1DQ0tERHZ1MjlQSmJXalpyOVkyTExBL0VIdlFrM1RYQWtXYnRw?= =?utf-8?B?ZE01ekxUZ0t6blRYYW8zSkl5cENmNzNPYWU3U0tiTDlVY2RuOXhKNWtES2dW?= =?utf-8?B?cGJWaUpucE13blhoRDZKUVU1VjBleTZabmRqcVVJZTRtaTY3dXAvUDJZMDRO?= =?utf-8?B?RFRzZUg0ek0yU21hQUEySk1sVklqR1FiVUkzcG1ya1JqR2pEUnZ2OWhpZDRP?= =?utf-8?B?cForb3kyS1YrV0EreG9jenRib2VDSldmbWV0NG1WM3JjUm9MUHRzQTVBdHpi?= =?utf-8?B?MUYvMEh5eHFIZitXMmVOQ3M5eU1icVA4ZFNZSlExNkRVNVFkRXV1NVBpelVp?= =?utf-8?B?SVppWEljaEU2UUJqV29CTVZsaWI4SHc1aFk5QkFMd2phT3FTMHBSQ1R1ZjVE?= =?utf-8?B?bHU5MjN3cVdDTlNpSjdDZTB3cTJndUtuOVdJZW1tUTV2QzJ5SUdOSWZBR1dE?= =?utf-8?B?OXVrZUliYVBoL1NuWnN1WHlESmxVcjh2MmM3b3NNaXFnRFJMdFR4aWx2N2NF?= =?utf-8?B?UDVUS20xTzJXNk9MSllSdkRNRTEwRDRJVzBreDllQzJaZ2NkallIa3pVbFpp?= =?utf-8?B?akpEeXVVK2c2Q0d0M0JVOEdrck16dmkxNjEyOEhFQnE5V3hjUytMd2pVWWdE?= =?utf-8?B?MXRRQWhKT0RiazBueFpiS0VvN04vTTl0ckZtcTBrYkl3WEZHTlZzTnJGeTRN?= =?utf-8?B?YUo0bi9qL3E4UnJ3dS84WFBtT2syczhzdWJkZWZzVXVHSG1OTHNzVWZOeU1n?= =?utf-8?B?a3krdzFWMGtUOVF2K092cWZxNzdiMVZuYXpGTnFtcEVnbGdJdHE0TjdzSEJP?= =?utf-8?B?MFgydVFIMUN0TnJVV3FrWEhxUE1DZUlCWjVNOFVlVzI1M2tXb0NQcExNN2lH?= =?utf-8?B?V2dJeWZjMjd3U3orZ1VIVUxqeU4xSU9ETFU4WFFHV2dpcnlVckRoQzU1UUNy?= =?utf-8?B?ZW12Qm42aEttVlBmYlpnOW1nT0NNVWVJMEJMTlI4S2F0Y2NRUWROL01MdWQx?= =?utf-8?B?OGxyM1pSRlVVUWxGZjNVR0h0UGdPUzY1WDJJM0ZmdURZKzI2b2VleEhXbmht?= =?utf-8?B?dWVxWVV6cGgxazFFdkpqV2JnUm5wdi8zYUJNZGZUdXVGZTViTWFLdTY1SElQ?= =?utf-8?B?TEFkbTdONnBNbFJEZ25OUHc2SzBWVzJEUWpIMXl0b0luMlk0MTFaV0UwbWxx?= =?utf-8?B?aldCUXdVaEdTNTVvQ0t2QkJJSWoxOE9xQXlLclh0aE92MS82ZmU1dTQ1MDB0?= =?utf-8?B?Y0k3WTFBY2ZIYVdRdmQ3T1IrSlVDdEhOWUg2Y3dDNnFRZVFPTjMyVlgzOFph?= =?utf-8?B?TWZldDFZdDhYRlBmaEJqa1lxL0cyeXppK01HZUI4Zk13dXpmQ0pPMDNEdkx3?= =?utf-8?B?TmVYWDBnalpoaU5nMWZ2STg5azJ4WDRtQWFhODR4M21WZ3lNV0F5YmRjaWhw?= =?utf-8?B?T2pNQmtJZjVLeUVuWjg5TG9yWjI2N05PRUF1VHpoY2UyS2dkZmpPZHBGak8z?= =?utf-8?B?a0hoRjlUK1pUSTRNQ2JVcHBqVDVTdWNGRDIwZk91cmZtRHhrejJPU0s3K0FX?= =?utf-8?Q?fv2HWXHjHxBotjqLehsA87M=3D?= X-Microsoft-Antispam-Message-Info: cGx9jPiQ7PDCl425zf+0KpwmKY/OF3wNZGjW8FhrxjxSw+W2C0aVCwEbzRL+IDADv9CPVF1rw86YbeSxWvwL1Hb3n7JhVHtC3r9YTehZ+Uq2T22zfYkG6u7ZPW7994H4Q1ZJgWVawCmFhVXVqYdQQAPuVUoewgEde2KDwTXyeDwmhlEFkco5qczH5n2CLvRy+xkdzf/wfpVhkJEx90JrB4mPywWQLHdn/jYX10cbbHG+U0oni47cGP8QW70zdeAzBQv/YdvO/fOf8T+PW8b59j/cSfH/rByIxdDpH5NWH2R1WwQEoTyVbhUCQ7/HwdCbHxGccYkjCrlW0gCfOYXmT2uYajMclgIWCHtnSB/O3Lk= X-Microsoft-Exchange-Diagnostics: 1; DM6PR07MB4906; 6:AnxV5EcDgpIeiXvMyLlm53X1e6fz8VezpADDQ30FZX+2bdfoFjAq/kL21RYu5QR0Ou5D6Qudoe9tObKUxyN+MYgZU6RCu629qp3VVt1KUXEXpudKL/9+UDGkTvQQWAiPQTda1yqA4RXqP25o/6MmrIGToqxAh24uTGhHggCF4rTzAaM2OG0o2xX1FBFCn+S350OhwCir6KY00WtOcQ4MZHvkwtWWJeuFhz+n/AszkJFXd2oihsHzrwP2DdFXpMfbFWOSN1BKLkUcPbBT2Za2TMg0q6cJ29v8Zr9MNtwSCwRAx2LuzlBsFPHCZwJHJWMxFgg/YEJGTIqx5ZmNAihr1uVSM5oBtCL4K78X2ABisJ38yRmbzcP2LEJWWfTJ1Aam84VEX5x5Ewr/Z0YclKyCScQ0IJCTeQ0UCSQoNS6Fk83l/RWb48PRI17PJ2izhtex/+9CU2T05l8zzjmCYXqdIA==; 5:+f7pniHprZZS+DEUrwuL3TtBnUzK4j1Xz9wNVjNPO9+A/QvCooHhRw3XmF6ECX2wp6RnrPwOyZ7u+dekV5wcPGvdwpKcEZgfhpvCrgCd1vP7zsWIvw2f9GT6xcHxYTBlGkO4FYRPrwW1mv2u6B5GGOj4u06tH3+a/I3EPVw9BQY=; 7:BB0vLINyndTRiGHPNRl+TBuaudZjL7INvEbjbLk5IQiKNA+f6NSqz4PYsXzHp2NH4uM7+P/NUYl0cAkE1nPZP5PNzazlsvoErcUYdeY/SqAqiW6HwBcnINSEZrKDn3SMoaXkVUsAAMtPIaoAy3DGSisqK3cwRK88wSINswgNjFHXsuMpXzu0HmW1QG7ltbkyfuwff7uY6t6QalOMCwu21F6kIzl7WyKV06aO/HoCZcVrFF/V3AQBzPLSNNnCsn8W SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Sep 2018 14:49:31.8579 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 37854b17-1b3d-4cfd-6678-08d61cacc9b2 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR07MB4906 Subject: Re: [dpdk-dev] [PATCH v2 01/33] config: add Cavium OcteonTX crypto PMD skeleton 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: Mon, 17 Sep 2018 14:50:59 -0000 Hi Thomas, On 17-09-2018 20:06, Thomas Monjalon wrote: > External Email > > 17/09/2018 16:13, Joseph, Anoob: >> On 17-09-2018 17:50, Akhil Goyal wrote: >>> On 9/17/2018 5:12 PM, Joseph, Anoob wrote: >>>> On 17-09-2018 16:07, Akhil Goyal wrote: >>>>>>> I think it would be better to squash the makefile related changes in >>>>>>> the 3/33 patch as the code >>>>>>> is actually added in that and here the code is not getting compiled >>>>>>> here. >>>>>> So the changes in the following files has to be moved to patch 3/33? >>>>>> drivers/crypto/Makefile >>>>>> drivers/crypto/meson.build >>>>>> drivers/crypto/octeontx/Makefile >>>>>> drivers/crypto/octeontx/meson.build >>>>>> mk/rte.app.mk >>>>>> I think this patch will just have MAINTAINER edit (even that might be >>>>>> required to be moved to 3/33?) & changes to config/common_base, after >>>>>> that. Is that fine? >>>>> In my opinion, you do not need this patch as separate one. >>>>> config/common_base can also be added in the 3/33. >>>> In that case 02/33 patch would become the first patch right? The same >>>> problem would be there too, I guess. The macros added in that patch gets >>>> used only in 03/33 patch. Is that fine? >>> I think that would be fine. Better to have a 03/33 patch before 02/33 >>> if it doesn't have dependencies. >> 03/33 patch is dependent on 02/33 patch. Shall I proceed with merging >> 01/33 to 03/33 and make 02/33 the first patch? > You should start by introducing common code directory. > Then you introduce the PMD directory with minimal code in it. Understood. > Question: what means "cpt"? > Why it is not an "octeontx" common directory? CPT is the cryptographic engine in OcteonTX family. This dir will have the common code that will be shared between octeontx and other processors with similar engine. The directory "common/octeontx" is for common code shared by different components of the same chip(octeontx). Here it's the code for same engine across multiple platforms. Anoob