From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM02-SN1-obe.outbound.protection.outlook.com (mail-sn1nam02on0075.outbound.protection.outlook.com [104.47.36.75]) by dpdk.org (Postfix) with ESMTP id C61BC952 for ; Wed, 1 Mar 2017 13:26:54 +0100 (CET) Received: from CY1PR03CA0007.namprd03.prod.outlook.com (10.174.128.17) by DM5PR03MB2955.namprd03.prod.outlook.com (10.175.106.141) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.947.12; Wed, 1 Mar 2017 12:26:53 +0000 Received: from BN1BFFO11FD048.protection.gbl (2a01:111:f400:7c10::1:152) by CY1PR03CA0007.outlook.office365.com (2603:10b6:600::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.947.12 via Frontend Transport; Wed, 1 Mar 2017 12:26:52 +0000 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=nxp.com; 6wind.com; dkim=none (message not signed) header.d=none;6wind.com; 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 BN1BFFO11FD048.mail.protection.outlook.com (10.58.145.3) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.933.11 via Frontend Transport; Wed, 1 Mar 2017 12:26:52 +0000 Received: from [127.0.0.1] ([10.232.133.65]) by tx30smr01.am.freescale.net (8.14.3/8.14.0) with ESMTP id v21CQjMC028657; Wed, 1 Mar 2017 05:26:50 -0700 To: Thomas Monjalon , Shreyansh Jain References: <23fc7ab5-1d1b-183e-7997-6e078d49a499@intel.com> <1662518.dF45MxJRnv@xps13> CC: , Ferruh Yigit , From: Hemant Agrawal Message-ID: Date: Wed, 1 Mar 2017 17:56:45 +0530 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <1662518.dF45MxJRnv@xps13> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-EOPAttributedMessage: 0 X-Matching-Connectors: 131328448127108886; (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)(7916002)(336005)(39400400002)(39850400002)(39380400002)(39860400002)(39840400002)(39410400002)(39450400003)(2980300002)(1109001)(1110001)(339900001)(199003)(377454003)(377424004)(24454002)(189002)(9170700003)(626004)(305945005)(7246003)(7126002)(65826007)(2906002)(356003)(64126003)(229853002)(6636002)(105606002)(2950100002)(31686004)(6666003)(5660300001)(106466001)(53546006)(53376002)(6246003)(966004)(38730400002)(230700001)(4001350100001)(92566002)(8676002)(36756003)(81166006)(8936002)(53936002)(104016004)(50466002)(189998001)(23746002)(93886004)(120886001)(33646002)(85426001)(77096006)(54356999)(31696002)(76176999)(54906002)(86362001)(50986999)(4326008)(65956001)(65806001)(6306002)(83506001)(8656002)(47776003)(533714002); DIR:OUT; SFP:1101; SCL:1; SRVR:DM5PR03MB2955; H:tx30smr01.am.freescale.net; FPR:; SPF:Fail; MLV:ovrnspm; A:1; MX:1; PTR:InfoDomainNonexistent; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BN1BFFO11FD048; 1:++QwQNmLirj1BjaiJ/nRCSvSKq4toA6b8dmXfuo81UIF9RSpaPUf6TMa0Xjm8czWsvxAsiQlJXfQggOIf2RYEq2vCXBz+EusdgGCuE63q10hPeOBBRdlgiR4tU7igHYIGD849NEfMi0tka5Yon7SKu2HAZHhi0XT/7oddX3WXsbiXAc9a43Z1LWuDVyjUihjE49W18eYft2W9il0pnuyDXo8pGB9ODz0P0SIy0ryz+FfPu0Vg7AKM5OWelwNVoS8VaWELujrMinjzZ7OD/BTuvco3Kpod/jeaX/IVmK9X1MuwkzCJ2nkZ8XZs41BTk2xedsdvTtxHsVMkwbWhSdUoK1eGSdm+3tMHF6oyZ5tJe7F4uunFLEnVW0n1QUTczS9YhfSbA2yOfNetff6txA0JjfZ7rq2FSJr0dxrK+SeVrsO5b84WSPV0yLQJj4iWZR4F5IJ8FAX943sLYyzKAjI/CZ6cy+5WTtICpkZR1j97o8AsSp4BWJ68Pt/iqMCAa4z7AtXZfHfL0qnlyHIbypiI/BLCofgJb0AW+9Bv/LCb0PSyQau8LfLi0RKLxFyVucOzRuavmWZ8zHucH6tP445ZLIkVNwSVu8yz/4/YgCRj+U= X-MS-Office365-Filtering-Correlation-Id: 29382735-1881-416e-a5b8-08d4609e3df9 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:DM5PR03MB2955; X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2955; 3:1JYPfvaAo7BmkIqeJ9tub8+N4FZsbrEC8FIjiPQ8MUlWc4+jDxazEoNpVZihVvPCruL2SMkYRfsfU8GJYR3VmCYVwqRTlRpr3ev3nk3xgs7Y72/rwjWsegrsQT1gzPK7MAlHDfF1bqcNxWjrQJyhF2NMcb93vYFeAZi2SoadncVmb2O/RWY4+XIEjktAQFyqnR81eXgPXFojmufRXGxND90IbWBz8bvm+FskY0xzxGGjPGyg/vcsp5CyuPQlsbBGaM9B00PVSUqGXXqxTG0yoF80XRbbj8EWecW18YnQVjCM6/T10JJgG3YY7PgU9PVeFYsww6ULE/NnbL7sX05PCSxDFTrgLtdyBUK3xfOgA/FZgUAJFF5W/7/1Yl3EGZn5; 25:kelkf+xYRCslyIGe/D2MB+5NM4UaiHjtQkfl/jdY4ae8oEKNQfYHRtf2BWVGpx5ohraKKzVXBGs0u9nVCHyU+RnbiKpheShKJhrhirpdeIxR6R8VdObqhU5qxevPGtQf5+nKOlzYT1wXKz+FSZjZ+du837SdAOpTfrtEK6A8mLHf4uY/YEzuFW+7iisoAG8DrhGAwd1G/73AB+mq/SzNYwG7weyB9BZMwTLr8tJjzxyTJTF9JxHOFFVzfh0+PHENJ+8ktOyHIZQ3kvIqV0rnNxI8dfaKMrOPqkDVW1g0B0e/IIq7RL/jDxvrlmu+y4gMJTgD8xqUbTv5ecvvROWY+dmpR1M3nsbdtUQ42dw7OvCMriimA+TmX0vGt5O/+VzAM42z+JecdYe3fcxedaOwc5QpRwCohOGK6oZfKIQf3eg2/fnPEBZx6xwqMJzYLBqT8gztFdoOD9xOO3xeFSLltQ== X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2955; 31:cMOYO6f3q8IfweeVJb+8XYvrINfrtBzyfEMDJQqDq08/npZ2kWixCNPiR7ZtjM1DgVCxS8HJxjNRNP4t/LvzRqsxQvDLxUHs0urihZGxPzWITvRKBRni+kGpMFAqdQbAi+h3EXQnuZg7YnR0ZUwE8901ZeFXvRdbNy1mNtN6/qurV4aXviybJbWXTSiIbDGplUy3vUD+GjCULs9K+jE+FwumVoncU5/3mgD+pTwFDPOCdFd1kOQdLpSozQyCQ9dlM7Stz751PTKc4P7uDIFmqQ== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6095060)(601004)(2401047)(13015025)(13017025)(13024025)(13023025)(8121501046)(5005006)(13018025)(3002001)(10201501046)(6055026)(6096035)(20161123561025)(20161123559025)(20161123556025)(20161123563025)(20161123565025); SRVR:DM5PR03MB2955; BCL:0; PCL:0; RULEID:(400006); SRVR:DM5PR03MB2955; X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2955; 4:44Q7IjoReFCa5C9EMajchSB+fLlgJeZH/tQuU/h9yQ7CA+fXT24P/pCX5KDFwT1UDa8YGGZcu9q1P07VNIEn8W1SeDkOGbsGClDeZtPfzFVIViLbiG6WQ3EkRvzb44XaQwU5EoFJvhTn8Hbp6fOFd5kFKVx+pF5qpD2rbT08nYXj/z0CuCKfWkW2RRq7dWkuEK9FMF8u8sa3isTvI3SuSd9WQqLHB6LfVixcXWQdHdgOZE+/+zlUx4ht+f9YqBJcfIRbMPhOoJbku7kmYGb18QC0clchAz3HrME5am9mXJVKuYnvJJewE2q3URLa6NIfHmsEuwi2q6m/+zGJ/+1+zDw/o0e6FWAGOVnIs0o/uaQSbN8xoeJ/tmFVqFVlyG2tVmaEnnad+1AGtS+wMi/YvknNh+xJ5JuWN/RNAgiGBwJZ/x78PAcxlJ60MWvqmZmyGv9LRlWAMPpR9n38gbZ2ZVvZycYnkenkl9FFyFlPHam0IqQOt/J4w+cnIRhJwcSSRQ9bqCKhBHWyWjb4wjwe8vXUW4wW0Envp3SMoaQnVPg5RM1jpRQunJEe0Jszipma1oPgPKByA/smrhhaiWBHTLFt8v+RxfB+DNnhfnss+e68fja6uBxABoqbwPWkrJ2E3aa4gtEe32+lkn2U/WFzuPatBocpjbDp7CX2QQdZc3+oAJK5vG6S5NSbS611Lm9QDIpCvMDgMmi5H4QRLTqBUg== X-Forefront-PRVS: 0233768B38 X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1; DM5PR03MB2955; 23:tOJsCo2bVKvbf9G3JgO8x4HT0HVHf4WFlYokp?= =?Windows-1252?Q?zhFrKy7MGNVW+g+b3xEQj0XEvw6k8KLC6AeZAICK0Ufn7z4Ytgp8NbmJ?= =?Windows-1252?Q?j4n87+LeLXVfHSt90BA+mbVekWftCBpquFsueWwpSc6e90fmOO8fCpeM?= =?Windows-1252?Q?WxzwyBh1twnRbfjsU2cPeOAAvtN9E8qeretiW9YICRwIxdMdZ1H2vpWV?= =?Windows-1252?Q?KaLdFD4ghACJP6OZ1cXHhDQZQY4A8ZdbXa4lX5xX2T7glMdO1GHTtFUV?= =?Windows-1252?Q?a0HZs8hTVSC1GcEmZQqZOpuIEtyNqMKP7HIz4YclKJuDr8GxkmxiUX2m?= =?Windows-1252?Q?W9o/i06LJ9NU7Cfet9pfUmboCQNfslToV7yHdW5fJAAcoWeFqdz033v0?= =?Windows-1252?Q?VAWDU8URxCO0MwZnHDcz5FjvENPdnfl972ORI6shjmlJeZl/3hMjKFMq?= =?Windows-1252?Q?NDMoVwuzouYYmDIbAsSlNlKcfNN+p8hCy8anLGKJuZVA8Bzs8HDsvsE0?= =?Windows-1252?Q?LuB9WeP5nj73DdCno1flYYFGYDlAfpkPEU54xf+S+Dx7V37Ssq6T5985?= =?Windows-1252?Q?7AOQxwcGCHRIY/7OMWpr3NDvkSdJBJj5kdpbRukaJuK8t7qVdj90B7t/?= =?Windows-1252?Q?PSLfU65yliqlyOEplx47o/RcITIicUpfHRlJEjQ/d4+BfSoxigVWzxn3?= =?Windows-1252?Q?8IcQAXiqLcqKui16W/KIuf/0x4qHkkewq3w+rks+kcVLSDIgkLYvuxnp?= =?Windows-1252?Q?p4KQDz/D7QPJjLm2hQ+xyVgX/HVyb0GbBkn3wScymT7NjA72X4lUPkIG?= =?Windows-1252?Q?yKNecaqjYXca/2Tc1/kebnSuHnXwdDB8wVumZeSO51AH2I16CBNFRGR3?= =?Windows-1252?Q?hnkXaozAOKxpuzkRnCSxDhG/bLeLRcrEEncUKS2mwc2w8UbyQRc+RoHb?= =?Windows-1252?Q?aWiHiej6dj0KlCodBlk3fwdSbezNWxfvLIfqUkZukhQMdwJLfoNGr2lw?= =?Windows-1252?Q?jsgCsvZlGX8l9ES50xKZwPKwYNObb2OBzM4NhDPEEO/WyJJmfhbf0URj?= =?Windows-1252?Q?iWCDORWO7w3mISNeQgsmrtSf6jUOWzJYHG/HPVCZIZ6S/FH092BpZFRD?= =?Windows-1252?Q?+DkXoQzzB1aZ/e2/CDw0Va7eOmz2gXNNMVQaeUpOmhJW+mcIbCRNV/pg?= =?Windows-1252?Q?NKn0fj1DSjRfiTskWEBQsAShtdsrtoiN3sVHH6Q/5Wak1SEodm3COx8k?= =?Windows-1252?Q?ZpV/GoCqueH5kFyZ3e9NDfKzPTRv0OI0Pn2EcS8ZlDxWbGaeiwBJxC/I?= =?Windows-1252?Q?FVFsbXv9wRm/ujsHuTLCVU1E2b+0hai300dFu9cM0fWMsMRNRwii+YEl?= =?Windows-1252?Q?ZQbuPotwJ7qjm5bpgsFefNMuVqmUe6DsqdVz5Qkmq+Wo727xbF7+XwpU?= =?Windows-1252?Q?PjL0JxC4H1Nqrz+PLQCRgCAspM84AeiFAMRoworCnoaG465TbRDbLNRa?= =?Windows-1252?Q?uD/DxqFJ2KN1mgex9PplONMi/TKiejCtMrGA9dyuJxKO+5KSpTBF88se?= =?Windows-1252?Q?jyZE3V6G0c9EN0JaOH7432tkG1Wfjt2umxF1ymONZyYOxOw+PSv0OUl+?= =?Windows-1252?Q?uzyxtB3CQBWXrDJWBxwD3M/io4D4+y5NbS5gQWboWJOVoOayyNoTom1M?= =?Windows-1252?Q?8xSdd6mYDm13e4IQ29u5DKgCCFuSygZiv4tkCBslhQ0gfimY0zOM+qAq?= =?Windows-1252?Q?IAaoK91ehJ5LmXbaPLIVrAL0+8IzIzjGcUMdquIFd2KcrVMHWJ01MM7J?= =?Windows-1252?Q?x6Uq7EYWsTx+Wa3qiz6dJ6PH/YZSq0z/OWPTsIzs/Q7ANw=3D?= X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2955; 6:SzIN1Jrqif+53oYlcbWFUCjoP1mP5DRvasDsV1RoKKGQkWiQ5+qDulXUCWMhYRXFvYJH7RvhEsqsC2ijtoIARjtgyMvMIiBEL+C8+iNR5mA5hbXEKiELGJpuErDLx3SxVQHojku/JLBCGvJzC+nilovwuanlMf6/3f/qJ2F5k4+ErgKRQdG3cwQZxtU2s3Qjz4qhJCBQDvbhycd9tHsPZsOetWdlBL8u8wWpzP/j6AHAYe/upRcSD0nWqp0kvUhuPPbX1MUVM6+0AQmxiAMJfnaK4J9IQ+Ze8KYR1kuW7GUoy9kFb4kMo34jKyV/v4Nsb9kHjM/xEK+Tw1C2AnPuoD0sSd6UR2A4+4B9O+6EAniTQzjVU5sMZriXT54bmR8SfkZkxkkyFzYgzYFfvyKrEMt2TH64Iy7oDRVrFxKhAJk=; 5:PM3NTyt40qE1R9db5fpfarwnMadc+6LIB8GqNfWwo784NCiEKOEEsogKwGKcwT6aJUnt8ZIL6c+sXF24ikLkOMHUCIVr/vlKQPhQEBc4GlwLilf3wlbIPT2N3813FDA/CH46lI+e5Zrf7Cuwe/TUjVM5SfbeX2JwZp8sD1BJPz8MJ8FJXZlubStREH+rXrCy; 24:2SvdbRR4L8mwHOKmzS6i1jxAQup7K59HTAN5leshRH0d5ktjJeY2aK4y/Eb8XDdz9mr7yyyBMC55yqulGKhenjCCWAJJar3CwFA3bZFGoLA= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2955; 7:58nyvmhNtMKdHrJhQE4Ht9lgGfAOdJ+ODU172hdsgsT7wHnGxAlSVK8wMUq6nuu/6U+G4nMifpCmOVFiMOlq7r2bfWPFcEXaRbdUFaD/3HSQJ3z6/MRgVY15iyhers+8iTqU7TGOacRVckq2BpodInXmhI5Uqs3M673lIiDXEYk9DxRobd2Qg5ng3pwmrXcsRrts8ob4mJS89Y9mfhNNU4YtEN4RHrtV0/IBcfDDKZzQcnQx+fAMLbZs3lt8bUwsgnnOfBdW4WZxyqxFLN9yYjMK0+UhmaTS6wJgtcrM25wN5jdmRJ3mtjJhoTE5HZzYca5GAqzn9yph6TgRTaePgw== X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Mar 2017 12:26:52.5236 (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: DM5PR03MB2955 Subject: Re: [dpdk-dev] [PATCHv7 03/47] common/dpaa2: adding qbman driver 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: Wed, 01 Mar 2017 12:26:55 -0000 On 3/1/2017 4:30 PM, Thomas Monjalon wrote: > 2017-02-28 05:27, Shreyansh Jain: >> From: Ferruh Yigit >>> On 2/27/2017 10:01 AM, Shreyansh Jain wrote: >>>> On Friday 24 February 2017 03:28 PM, Ferruh Yigit wrote: >>>>> We can go with option (1) now, since these are not real APIs to user >>>>> application, it can be possible to change them if better solution found. >>>>> >>>>> Do you think is it good idea to have different naming syntax for those >>>>> libraries to clarify they are for PMD internal usage? >>>>> >>>> >>>> Indeed. Current name is librte_common_dpaa2_*. >>>> Do you think librte_drvlib_dpaa2 or librte_drvlib_dpaa2_pmd is better? >>> >>> common vs drvlib may not be different for who don't know about these >>> libraries, what about using "internal" or "private" kind of keyword? >> >> I am ok with librte_pvtlib_dpaa2_pmd or librte_pvtlib_dpaa2. Sounds fine? >> ('internal' is too long and its abbreviation 'int' doesn't make it easier >> to read. :D ) > > There is already "lib" in "librte". What about librte_internal_dpaa2_ prefix? > Ok. We were just trying to avoid that long name. :) > Internal is really the best word as you are requesting DPDK to host > libraries used only for your drivers. > I thought you agreed to host them outside of the DPDK repository. > What is your reason to keep pushing in DPDK? > I don't think the last discussion closed like this. You suggested [1] this as an option during handing of issue with rte.lib.mk causing the parallel build dependency of shared lib to fail for drivers/common. Ferruh has already fixed that[2]. The drivers/common is an integral part of PMD - very similar to a *base* directory of any existing DPDK PMDs. The only difference is that *NXP's base* is common across multiple drivers (bus, pool, net, crypto, eventdev etc) - all of them interface with SoC using this 'driver interface'. We named it as "driver common lib" - we could not find a better way to name it. Probably, a hw driver interface or dpaa2_base is a better explanation. Imagine a PMD which resides within DPDK (dpaa2 pmd) which only has high level interfaces with DPDK API and cannot talk to a real hardware (SoC) because it lacks the basic driver interface. That would really not make that code a 'PMD'. Probably it would be better if we close what is the scope of a PMD before this discussion is to go ahead. And, why would there be a need to store this 'lib' externally. Following is what Neil replied to your suggestion [3]. And we fully agree with it. "Please do not go with suggestion two, the more libraries get hosted outside of the project, the less likely any sort of test/build/ongoing maintenance from the community can be expected. If you're going to go with solution (2), then you may as well host the entire PMD outside of the DPDK project, and that more undesirable." [1] http://dpdk.org/ml/archives/dev/2017-January/056243.html [2] http://dpdk.org/ml/archives/dev/2017-January/056321.html [3] http://dpdk.org/ml/archives/dev/2017-January/056292.html