From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0079.outbound.protection.outlook.com [104.47.42.79]) by dpdk.org (Postfix) with ESMTP id 3FD72F94 for ; Fri, 22 Sep 2017 08:14:32 +0200 (CEST) Received: from MWHPR03CA0027.namprd03.prod.outlook.com (10.174.173.144) by CY1PR03MB2362.namprd03.prod.outlook.com (10.166.207.149) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.56.11; Fri, 22 Sep 2017 06:14:30 +0000 Received: from BN1AFFO11FD017.protection.gbl (2a01:111:f400:7c10::111) by MWHPR03CA0027.outlook.office365.com (2603:10b6:301:3b::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.56.8 via Frontend Transport; Fri, 22 Sep 2017 06:14:30 +0000 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=nxp.com; monjalon.net; dkim=none (message not signed) header.d=none;monjalon.net; dmarc=fail action=none header.from=nxp.com; Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not designate 192.88.168.50 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.168.50; helo=tx30smr01.am.freescale.net; Received: from tx30smr01.am.freescale.net (192.88.168.50) by BN1AFFO11FD017.mail.protection.outlook.com (10.58.52.77) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.20.35.14 via Frontend Transport; Fri, 22 Sep 2017 06:14:29 +0000 Received: from [10.232.14.39] ([10.232.14.39]) by tx30smr01.am.freescale.net (8.14.3/8.14.0) with ESMTP id v8M6EQ8W012766; Thu, 21 Sep 2017 23:14:27 -0700 To: Thomas Monjalon CC: , , References: <20170823141213.25476-1-shreyansh.jain@nxp.com> <20170909112132.13936-1-shreyansh.jain@nxp.com> <25531756.sobbYYJFMz@xps> From: Shreyansh Jain Message-ID: Date: Fri, 22 Sep 2017 11:55:23 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <25531756.sobbYYJFMz@xps> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-EOPAttributedMessage: 0 X-Matching-Connectors: 131505344699388712; (91ab9b29-cfa4-454e-5278-08d120cd25b8); () X-Forefront-Antispam-Report: CIP:192.88.168.50; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(7966004)(336005)(346002)(39860400002)(376002)(39380400002)(2980300002)(1110001)(1109001)(339900001)(24454002)(377454003)(189002)(199003)(81166006)(65826007)(105606002)(2906002)(31686004)(8936002)(106466001)(33646002)(50986999)(189998001)(54356999)(6246003)(498600001)(76176999)(65956001)(53936002)(65806001)(50466002)(356003)(47776003)(5660300001)(8676002)(23676002)(81156014)(305945005)(230700001)(97736004)(31696002)(36756003)(77096006)(64126003)(316002)(83506001)(85426001)(68736007)(54906003)(86362001)(6666003)(58126008)(2950100002)(229853002)(8656003)(104016004)(6916009)(4326008)(217873001); DIR:OUT; SFP:1101; SCL:1; SRVR:CY1PR03MB2362; H:tx30smr01.am.freescale.net; FPR:; SPF:Fail; PTR:InfoDomainNonexistent; MX:1; A:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BN1AFFO11FD017; 1:0fHy5P5zFbCB0fvTBf+zNSub9H7/qIfdX1tlAvxs7niy8tuIMMVCJJDrNtdAqcGagXM0D32xjlqeg5D7ve800hMgKdpNQglo/UvB2x4JEp/uaTkr/qUqngFkqB0VNBJl X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 7e635396-4379-44de-4820-08d501812f43 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(300000503095)(300135400095)(2017052603199)(201703131430075)(201703131517081)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:CY1PR03MB2362; X-Microsoft-Exchange-Diagnostics: 1; CY1PR03MB2362; 3:/0B/03AEJ/xGMDiNEI9csLdlgOaJH12fVYve2bzE/2EvGbXq4PnwxPltxQpPHwqqLbQNcdvSzIdDRc2RVlp0qpPFj/kCnN4KCCZQjevRyBW5lSQxtCM497TUNrpmWeF6gAKhzQJoNXTPFPZEHa2qbF3/gfVLcrxHxWt64IUh1tb/Ei7xkHDlkO34YS0uTNHlpfFqL3sOF1HumEMXR93ZzTbTZOIHEFw+xhF5gzSwcC+/whNlX/0Cza2hhe0gV6LTSYtcj40ZXET+uMTvL0Ci+UqeLJJxbdh85c3wTha6xBty591ZG1vc4rZ5LDcdrEYeDZJ5zNufpn9ezwpner4C6CKuHFvw372clV+ywaK3cmA=; 25:klDraaD/uQQpT5y41MSe8yFRu7k6fBXbP6u1shTA5CNNu0qe94MNWxMM9WfjgqIJGM/B+0mNuTRiREO6Fy4WWDjDX/ptlLuqgS4M7nHoGYFsXfs2wjrq3q25BL1FtjlrXMfCnSOVe2OhPszwy5OPPC9n1PZx+DYjeFaEt9pa2+jCJpWIZMDUr+yKoM5DUlJAGYoxcBiOwNni5DQ2YJuZ52aeYbGDJFdOMYsNwAJ3SbJkaInIs9ZqKqdu35BUol8+rjpLrHkljUYA+ess/ZnGOkYirYSZZAt85iGbeHh5jtgBlDrvpG7txOMmrcndpE42gyaJV29oYihhHAyegOmsjQ== X-MS-TrafficTypeDiagnostic: CY1PR03MB2362: X-Microsoft-Exchange-Diagnostics: 1; CY1PR03MB2362; 31:cEfblaXf0dPdmAwUgW1wkSMqdN9bG16g8NW4eYFsqiXQlAN1nivNaIXtgrp0VkzhDFJZbAwLPyY4Xv3uzY/9lJ4AZ5FS45GQZLHkSU8aRUXEhMYxzlXYJSuIDXTrCc7Li/Su3cXJK+YFIqAGaJXcIHNEq9v3ry0AAPlqI5ZSv/5/lHXibdKA+Oy7IK0jILqk923LXywt4qZVFdyZqZ1/fjIaR3cN74Bulo8MaoASVIw=; 4:4cZyi6vIhC87IRAuleRGGyO/qlN8iBPwhFk0GpaVSkryuRvagRCtToi3PmGv2DekUEhZqGswgvNCmqJ7jYrZaLvucFpGkeAT1NuIXvg86ZJisXuDO7CKtyXu856P/TMnGHZoww66zf68//QwBkv/mJKGrURDFPh7lqtSgUIIXuY+HX8RIJ4zbyP1yqDQhLmdwflBpBvkTnAL3RcxHRFDMv03CTIre8x2FqvOoPq+Mand64N03UYrs9CsSRSQ9hly X-Exchange-Antispam-Report-Test: UriScan:; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6095135)(2401047)(5005006)(8121501046)(100000703101)(100105400095)(10201501046)(3002001)(93006095)(93001095)(6055026)(6096035)(20161123563025)(201703131430075)(201703131441075)(201703131448075)(201703131433075)(201703161259150)(20161123556025)(20161123561025)(20161123559100)(20161123565025)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:CY1PR03MB2362; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(400006)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:CY1PR03MB2362; X-Forefront-PRVS: 0438F90F17 X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtDWTFQUjAzTUIyMzYyOzIzOk9aeittSTliRW9JaUVoeFRhNVJIMTdJbkVM?= =?utf-8?B?MEFKMk9GNWNpTzFiU05SbVZ0M29hTjErY3QzcnRYeGJGTStxTVVIcDVRbDY3?= =?utf-8?B?YndKb1FRUHM1WTc5azk5QXZiNVJNN3piYjVvakIveWxVbDFiTnNKNEwvczN0?= =?utf-8?B?SURYVXJiOWdVNGhuWC82NzZ5MlRyZjJRRFFEN2tIWEVDMWFXc29KT1BBV3Ez?= =?utf-8?B?R1JiUjBRQUpjT1NLTFRoQnF5bHIxdXhYTHNRUitHWURVQy9Ldyt5d0cydjIx?= =?utf-8?B?cGZxN3R0aDQ3VXdWcThCd25BUjNUeU5sZk5nbU1QUE1qUVpsTGM2dDhMR2wv?= =?utf-8?B?WWNVc1hCV3Y3R0NFV2J2K3ZuVDFNTExZSFlWemlBMjA2YkRSVnd2RWhXUjB6?= =?utf-8?B?WWZNMDBjZVd0Z05RYWhpQ3I4WjdNNUxWZkgrZ3VEU0w3UUg5SWlQbU8wclQ3?= =?utf-8?B?dUtZYzJyY0pLZTN6Yk5lSC9WbkJoNzRhaFlhK0p2TDg1RXB0MENnSThHbjJG?= =?utf-8?B?eVB5UHV3MGhUdWVoSndsbHMxd2JqNjgyb2tXQnlpdW93blBnRTZwd2Mzb1JI?= =?utf-8?B?NEdMVW9YcGlaNFJuRGhjcXIwcldydHJxdW1NdUtKRW1EcW8ySkxoL3F2VXdt?= =?utf-8?B?c3BsKzdRUUVKK2hJL3kyOUtxNTdmTDBEYi81a2J5cVpzUUxOWWtyTERWQUpv?= =?utf-8?B?a1RLeFdPMzZrT1FNT1AvTVpadUFvOU5HMEk4VWRVNk51VEl6bHRqQnQxREJS?= =?utf-8?B?ZzQwQ3lNK1BLWGdoTVB5RnlhTjdrMmVYMVBmVnhTR3dIRmcvNTFyMCtoOHlO?= =?utf-8?B?aURiSS9hYkZ3RGFReElQcWd1enJYL1V6TjVlaURwZjZkZkxpZ2xHbU4zM2pK?= =?utf-8?B?WGFqMnZndXVZUWZ3bmJTUWd4bGxYSzFsWTEweGNjbzZKUmZjSmhsemx3UjYz?= =?utf-8?B?Q1BIQmtVU20wY2JvTjZOaWpaQUhBRkZKUnAvYlJBbEVWRGdZbitJSDdtT0NQ?= =?utf-8?B?UHcwL0RKRCs2YjhQR3c3SGUrTTFMQWFpRjZZR3IrMHc5QmJ2QkwvbnZwMGhn?= =?utf-8?B?bW5CZnZLSWJFeGxHM1c2aHdUajI0R1AwY25sZ1BJc0RwMCt0OE1VTU5XWkZX?= =?utf-8?B?VlRwSWNneFdnSGpneGI5Ylk2WEo3Q2Z4bnArd3JBbVNRV1hpQ29rQkpYZTNU?= =?utf-8?B?T0Z4cGMvU2dkejBlTEVvVHlzdWhyVlk5cEFZMTF4WkV4U2hQbkpNcmRpSHp1?= =?utf-8?B?YUdZTVV0dDVlT2VEUXlSN28yZEdvZ0xSVGlLOGZXM3h3Y0gxajRBL015Y0Qr?= =?utf-8?B?ZDVBUkhIVWFacWFWdDdZNXNmTjNIY2JTWlcvLzdQUHJJakIyM2FWVnk4ZXBU?= =?utf-8?B?NWlDNVFqR2V2TFhnUzRENE5pNHRZcDF0SkFob2ZtZ2ZGSFJKb3VBNTFUaTRT?= =?utf-8?B?dFRYdEQxNEdoUUVXNjBDL2EvdWVHUnBLcVN4MW1CQmswV1VKVTZQSzB2NzVa?= =?utf-8?B?QTFtb1Zxc2ZxV1VreUV5eGpOQldhNHFoaUxDRGpaODFGVUhQQUdlaDZFN0w2?= =?utf-8?B?eEs5eDdCNmM5cGdIYVBvRU1JMWkvRFZzUEx4MVcvdi9aQVI1amthWXVrSmtW?= =?utf-8?B?dGFUUThJbWZMUno4NVQ0RFFqbGxQbnJpaVhXT3kxTTBTUDE4OVdmaWNZRlFq?= =?utf-8?B?KzlucEJPYlZTcjM5d2N1UzB4TG11cEpwRUtwcFdtc0kwY2N5LzlPbFdxTGlt?= =?utf-8?B?K0dlWEJSQkx0MStjOFBrTXZxRCtNTHAwbU9OY3cxOGordThpQW1POFdOTUpp?= =?utf-8?B?R2dwMk5JWWdjV2VkQ3pYRVMrVUU2aUJucWQrVFVTNFJ5RXRHY1VVclY0b1M5?= =?utf-8?B?ZXFaUWJIZXlqWnVNMVhtL0h1YTdnd1pYYS9mV3E4WVRBcXBXNWJTWWlpekZ0?= =?utf-8?B?eEFZbXlVRUNnPT0=?= X-Microsoft-Exchange-Diagnostics: 1; CY1PR03MB2362; 6:7wSyXLxYlAwMj2+UdrL4fWLkgEE7pf06Z9y5KXKJzSkVYzVRExDmDuDLWw19TJsL8uH218kjqDKeMrK3EyyUaYBTXBSfcD1+EadL17AyfTvgfb9XeLUiulxuoaYVJLYph8IsH48cv2mYl9rsv/6SIin+OrvKMuQ1uB2Bze3JjhvloONogi3OcHYqebH9tes+nSKz5be974Er/yn2lEEDswRPbcM9bIchlocnhBbAYgIf5yg9m3ilaWoj7jHNk+GG6Iql415xXRvh+LGO0ldLAHTIg4Mim4NGGHaR7Tg2kEF3QFEL8gRynnMDnlXmpYgs52FprZcMFDOpj9SF5yB8gQ==; 5:eTPdgT9IHlbZ0gWoMl0bas2QynA/wgNyOmzsIP3TkuBgll5lso09Vep+Uqh5z0+CYfm1kVzTegz/zw1+Hl+0guuWnkBV4bJmvdeZzQEJJqMp7EvpUXrUtyT9CPJ+EXQtWPDIPPbmdm/Ce97aaaAShA==; 24:lVij0BeZHCKdaW2RG3miV0q8b5bKPRRZb4qkeHOUBguxuts8nd2PJ3GRCZaNQYKxqH3b+WfP3sHAGGTSx+VwqU8ipoCQRwwEB4y2o6AYEvQ=; 7:D4ivEyyUouRN5EZAO9hU6vO/7m849D6pPQV8C53rly67+lngDR7h2JfPwsAETJ06+N/17tQc5tIOcyeBx5d/UtPjNqBL+pSP0yrLyXF/wKvAo497APce3QQsAi2G8v3WPuX1WeItcKdDXM8a4uNbuFwlpFdy7YdQjfUsnrFQuDyfjPYvsCTHO531EnLFrwisj21vdPxu1BQCYwoNP/lKn+HW0JuDFdbpt6XgVb+/4g4= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Sep 2017 06:14:29.6424 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.168.50]; Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY1PR03MB2362 Subject: Re: [dpdk-dev] [PATCH v4 00/41] Introduce NXP DPAA Bus, Mempool and PMD 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: Fri, 22 Sep 2017 06:14:32 -0000 Hi Thomas, Thanks for comments. I will reply soon to those on v3 as well. On Friday 22 September 2017 03:40 AM, Thomas Monjalon wrote: > 09/09/2017 13:20, Shreyansh Jain: >> DPAA, or Datapath Acceleration Architecture [R2], is a set of hardware >> components designed for high-speed network packet processing. This >> architecture provides the infrastructure to support simplified sharing of >> networking interfaces and accelerators by multiple CPU cores, and the >> accelerators themselves. >> >> This patchset introduces the following: >> 1. DPAA Bus (drivers/bus/dpaa) >> The core of DPAA bus is implemented using 3 main hardware blocks: QMan, >> or Queue Manager; BMan, or Buffer Manager and FMan, or Frame Manager. >> The patches introduce necessary layers to expose the DPAA hardware >> blocks for interfacing with RTE framework. > > I guess these are the same blocks as for DPAA2? > They are in drivers/bus/fslmc/ > Why introducing yet another bus driver? > The fslmc one was supposed to cover any Freescale (NXP (Qualcomm)) SoC. > >> 2. DPAA Mempool (drivers/mempool/dpaa) >> BMan, or Buffer Manager, block of DPAA features a hardware offloaded >> mempool. These patches add support for a driver to manage the BMan >> block. This driver allows for mempool creation, deletion, buffer >> acquire and release, as per the RTE APIs. >> >> 3. DPAA PMD (drivers/net/dpaa) >> The Poll Mode Driver for DPAA NIC Interfaces. >> >> Patch Layout >> ============ >> >> 01: Add DPAA SoC build configuration >> 02~16: Add DPAA Bus support and features, incrementally >> 17: Add Documentation >> 18~21: Add DPAA Mempool support >> 22~41: Add PMD and its various features, incrementally > > It is a very long series introducing 3 different subsystems. > I think everybody was scared about reviewing it. Well, then Ferruh is quite a brave man - I got loads of comments from him. :D > Why you did not split it? All the components are serially orders. So, whether I split it into three separate series, or clearly separated in a single series - it would be same thing. Isn't it? In fact, having three series, one dependent on other, looks more confusing to me. Personally, it would be difficult for me to review such patch series(s). Still, if you and Ferruh think this split helps, I am OK. But, I don't want it look as if a new request has been made which cannot be completed within 17.11 window.