From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM01-SN1-obe.outbound.protection.outlook.com (mail-sn1nam01on0043.outbound.protection.outlook.com [104.47.32.43]) by dpdk.org (Postfix) with ESMTP id B2E252BC9 for ; Fri, 4 Aug 2017 06:00:30 +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; bh=3JElQEACwgJ647znWWIpWgZUWVKF88yAo4HwVJUwC3M=; b=cNWJPI5aejemEneJySn8gpffkOGmaJHjsjBUsdqkWqua12t0XkUWqYiQM7hUDUC0ZIO5fJVnG/wzXM/ox9gYp3M33k4buLEPl2NSkz+AAhlSJLvRoMjFn8+0LA0NVgzdSohkxQhAGHOPCI7BqIw2RlOnEJVv1a31aV6WFCGpsI0= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Santosh.Shukla@cavium.com; Received: from [192.168.0.103] (103.76.57.163) by DM5PR07MB3098.namprd07.prod.outlook.com (10.172.85.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.1304.22; Fri, 4 Aug 2017 04:00:23 +0000 To: Thomas Monjalon References: <20170608110513.22548-1-santosh.shukla@caviumnetworks.com> <15653957.xZGtGGaVPI@xps> <3b6f7ebf-0b4b-59a7-d913-0097c045b818@caviumnetworks.com> <6030915.ZPqjvz0sR7@xps> Cc: Olivier Matz , dev@dpdk.org, bruce.richardson@intel.com, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com, shreyansh.jain@nxp.com, gaetan.rivet@6wind.com, sergio.gonzalez.monroy@intel.com, anatoly.burakov@intel.com, stephen@networkplumber.org, maxime.coquelin@redhat.com From: santosh Message-ID: <417047d2-6a9f-9bd1-9582-3b32275f20cc@caviumnetworks.com> Date: Fri, 4 Aug 2017 09:30:09 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <6030915.ZPqjvz0sR7@xps> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [103.76.57.163] X-ClientProxiedBy: HK2PR04CA0086.apcprd04.prod.outlook.com (10.170.154.158) To DM5PR07MB3098.namprd07.prod.outlook.com (10.172.85.8) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: bc62d666-b18c-45d5-616e-08d4daed57b2 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(300000503095)(300135400095)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:DM5PR07MB3098; X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3098; 3:p2IeW6I9Rm85Y92WxJU1EjSjwtdI/o6c4YJY6m9xp5t/j1+WQ7iww+fdTizUEZqgeRwSZl9JTNGEKvujkexNmRdTxG+xdLOkvnIlBzM3768TN0y1lENRExxr/N95OTOJ2NWKY/zuwyF3WlH8cGFj/r5Fd8tQSygsV/76+3pMdg5d/yu3hN9SWeyy0Q/Jrl8fc1nip6fsj9lzGD2L5UN7Bgo8asWLkN/SV4jDSn+jlDkNRpjXJQr6stCYEpiU1kbH; 25:5WW3lEQKDqz/7vOyjomi4iS1JOx29gQcUuNaNJR0goSuCgG+lglLHlIEsKLUGU+MsbLNuk8Gc2DLo+modGUcP0wspLHRwndnv9mVY3Xq33du7nWbvN/p1Agaekmv+LUoqAEz5TXsoQIMKbLHnYwKx68eM0FdaBa5r1GaxqqCP+CC/ko3y3r1LhIWE697ALKyzkTeIjLrrWUgrMIVx873xyU8ijlco7wZYzST1yo/JWNv4J9SBAkRd15rFEcOAoIAzKB64eDS3VV1CdUztTgDAgP47W/MWP61qH5MOPiS9VqnaQybmhUW8RjhjhCIrGAnDMllshyoRLzAhWIDTRqIaw==; 31:a9l7rLAUZ4zljXRRJlIXsy3X3TeTS4LrDwuKv91LWO/00zt7V4KXBKgAEBsuIOBfqOPJnzgw+2ad6KBGCOSdD9DHpQXpiHiAPaJ4L0yKqiSaI/WsQU9Co9i0BAbnp232zgqdJrFpq/9qPEZ//IA5Ph5uGYBR2OXrHwtax/qk7LI6WHRQddyyhCq7phCeQRFoDr7nQur8/T7JMrJAC2QAHH3SyzkQm2rMgT0VBTmUaiA= X-MS-TrafficTypeDiagnostic: DM5PR07MB3098: X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3098; 20:wrh+x1GL4LXLn6RLfhwHTlgzIsEgM2r6DysBUspUDHvGeM3Is9NifcVG/D4ilX9y75nimS5wwjYxozNoNH0KwZnEh+aubPU7B5SbHaw/cg5YezTjiXGzL7pv6/dEFybxQj3gWkL1mfOSd0+OXxzu1lEElqauqZxcwpg1ZjXQm0qunCd8I+R9h30Pup40Vb4V+Vx6O+IA9wAh1pnRWiiUaemsSPBz3q3wu2rAzR9YejcbMuPvU7DFVRo332jLO17TBLoR//yyg15/n5PcTcgmd7K5+ZoPLSXOblb6luOfQhKNIcOimHKX+pFPk2f8v16Y94O4UcZPvBJ/pIrbak15PPDOrRefRyJUJ4Wx1hMD8y4c8sNWY9f2mKJ9Tj8vwp8MK65vAaq4FFS0xAPHqDOA3QQy9QTg5eEcDhijk8wKsj/gBdBtbhPNegReNT3zb/273A3NqJnQ60CTnwQ9wucNRtXfVwxRvwqsppz30676fWFOBcOWQQn/UIy2J8Fn0zvKkNFrsiLiixJhcyrOerkHCf76P3llTe+wT3+omYbxdhRCoTGXN9ioVS6bCTb5R+x/rzEV2eGsCgVop/uFev/BsUc9n1gttN537nYNXlPXTE4=; 4:qaBqkEjQrHoM4EvJcmBwxzseN0wC2fH+dhh5iLm/loqyo886uT0kY+rOIsR7yxDrF8BOI5wFYx7ldRqDng6ztkJkM3lQJspHR2Bbbt1V6Yuyu36YZlcSwya5VHMMFX7u+y1bJTYIbxvMKTmweMjbkkDc5FBO5Tbo0gt3QLWwdHOX7/HIkyvdhLAO7z2TgRF+Vqd+IWNQpkeoNJukyYOap/Rb6T1BGZq+Wb2CLfiRBmt1MT8HvHEyO5SvT9dIqcqc 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)(6040450)(601004)(2401047)(5005006)(8121501046)(93006095)(100000703101)(100105400095)(3002001)(10201501046)(6041248)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(20161123564025)(20161123560025)(20161123555025)(6072148)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:DM5PR07MB3098; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:DM5PR07MB3098; X-Forefront-PRVS: 0389EDA07F X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(7370300001)(4630300001)(6009001)(6049001)(39450400003)(39400400002)(39410400002)(39850400002)(39840400002)(52314003)(189002)(24454002)(199003)(377454003)(72206003)(53936002)(189998001)(966005)(90366009)(5660300001)(81156014)(53376002)(110136004)(38730400002)(8676002)(81166006)(93886004)(6246003)(65826007)(7416002)(6306002)(6116002)(2906002)(50466002)(230700001)(3846002)(97736004)(4001350100001)(83506001)(8656003)(64126003)(86152003)(68736007)(31696002)(7350300001)(6666003)(23746002)(478600001)(117156002)(54356999)(76176999)(31686004)(101416001)(42882006)(6916009)(2950100002)(33646002)(47776003)(36756003)(4326008)(25786009)(50986999)(65806001)(6486002)(77096006)(229853002)(42186005)(66066001)(65956001)(305945005)(7736002)(106356001)(105586002)(533714002); DIR:OUT; SFP:1101; SCL:1; SRVR:DM5PR07MB3098; H:[192.168.0.103]; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1; DM5PR07MB3098; 23:954zXAfw9oZZgU0rdAqiaUEfhyNYOg5MAI9zh?= =?Windows-1252?Q?qhqGLFftCvXhQj28dq7gVvtmCfAtUESGkqqA1HzuVyrNe3uIQKE20C8C?= =?Windows-1252?Q?Ha7rgwciJn5AmDclf2rFB8evHERUIoPAvi10YvJ2pA7myXO3cQaqF5FA?= =?Windows-1252?Q?ZbuYQcMsMk7Rj7Cy1NKho0iIXrLQaZA15NGJqes8mDvj+K4iiIwzIuML?= =?Windows-1252?Q?mqny+AsVVsctdSCrjB8yYP7k3jM9CqvFGW6R8kKUBoAUEuZkthARUpb7?= =?Windows-1252?Q?JBINjIbgo+Mk1RBJ7iLgBYZiQPJRmo99jpno2n276iA4ENYnQADcKJ0k?= =?Windows-1252?Q?0ncy1+YusGB8vlJlCfOfZf209KXoN0pk2oMMyj+RWjId9EQDeNMBc3Pa?= =?Windows-1252?Q?9BQaKYHgbezvEkFN55pE6troaxQyc7vI/0QNBR4Kac0iXzoITsDIl5TU?= =?Windows-1252?Q?bbFXgvfkdqhr4UxSbciNwNxJLOVJkJdOoo/pZwiuAOGHv1HCmfWMD0CS?= =?Windows-1252?Q?Cc6VYsLUFtMrdRCrgyyWjF+sBnM4kjV2Mc5oqimFFvfMAfgvJT+n8K5k?= =?Windows-1252?Q?QXO3+YfQcg2ukxZwqsb9vDIaIiaWE/6MLkpe0V9+ojQM5JWLbKv55Nv2?= =?Windows-1252?Q?uX10LmXaXvzExybW6TW9Rd6Wh7c8ZQt/8OHFW64iSIywXG8qASYRtLdq?= =?Windows-1252?Q?qOTOMN50XvKEfYtYGZ8QmAK82y5i2E369R+zMwC92Xh2sUBRtZCIvolQ?= =?Windows-1252?Q?wKAHEpfWvCj+UEzKcLXf84VlOk/Apk0GLOY8rlxpkVj/Zz1tfZmL3Xpm?= =?Windows-1252?Q?KZiHdbb2jZQxm5SrQkr1zrRTgh/7FY6ow0kQbB1SwYOsWtlhdzbF+OaN?= =?Windows-1252?Q?bix9sx652xHB9l6oMoWYf7oxBuxY7YIzgPh01UhYspoJ1BfpEdoeknRi?= =?Windows-1252?Q?C/z1K9R6JO5SpYA+a0V6hLEdBBPob660GwC8G8wui2+ejN7pbbZAcB3n?= =?Windows-1252?Q?eeZd51cEV4O4OVgNPE0WbxoGRDDYXOKbyl7YKsyPseDdjFHE3LNbtYJH?= =?Windows-1252?Q?uKHO7f9MTw0o3vCrFV6klyP11VIRzUPbsQs/1rSNCTqpmhJTydNQQ2tK?= =?Windows-1252?Q?ES+BcKUUGOLXIWfETJ4kEPnvj4tXqkCdbJUOfOQOIwuGme5DIL7u2+n0?= =?Windows-1252?Q?jb8F3sP0/ovECgl9l/JqWBjlBB8UsS9rMqu/sXvhYdB1Fnu1AOYq3/Av?= =?Windows-1252?Q?y1ATzNHI90rnXv1hzgJc1kcXi9ByNEK55rLGziMa11/jVkez2bIQyjPp?= =?Windows-1252?Q?j4KiFJDOPMnyuFSfVkd9XfYpme9e0xI35ZcbTNcgZRNzLIaCATjN5FPD?= =?Windows-1252?Q?RyQeffzFvXj1ehLqNZRk/eA6EEDLbYkaflTQpLtVfwMjJMSQOuGiPAQ+?= =?Windows-1252?Q?TScDcsk2jrTW7aFljkfS6Zx5meHgju1wLLCzt7qUqXEP2Jo3338YO7PN?= =?Windows-1252?Q?TN6lC1lWYADE7SOpNzXjHR1GcO1oJqxcYaIsBgpUAYi8kEP6XlDkGN2+?= =?Windows-1252?Q?n2rLFZQ+3gfCuML/j1bI02kKXS1R1sD16O4T6LtuwOISR4MT4XcCzroj?= =?Windows-1252?Q?BMWWQH0hhUsP0/BA1o9D5+13wKEsT0dSg0uY0/5GDJizs7iJyRHsQw/W?= =?Windows-1252?Q?Vxq1+HzNu8QiL5KNGxvHCdMABkTl58verAu36DPJW4Xhfyw7uyNaf/N7?= =?Windows-1252?Q?shdnltjDh/gLlvkMRlf/MmdJlo+HDtKgzQEsyF1P1NjVvncldCWT+FYG?= =?Windows-1252?Q?Wpr7haozv2l5phsoxZOocXX/Uq3lzce5Rwyi6h26vHcY2nb0wzBJHJ5b?= =?Windows-1252?Q?u4n3WjrrwQQ/qrHapJwqozWq7jmSeeYiw=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1; DM5PR07MB3098; 6:7snfo2uxPsZLcLEqnfIW5eSySmW1NIuGcE4XzxNObuWZwM9WkQw7+alFW+f5ZY29gNqOOVvd24dymOu2pbSOFYdq2ObntIAIlKUwzw7U+GlM0LgYerwIEANzRmlYQ/gI6L11dYR5XI38t5qSnK9UdPGlEszgvhKpaILMeu8/UoWoOJdGHwDN/Uwrq+Z0wOrmWcDYjwNLImAgMmA7VNmTkTErSyNS02thd2vw5CizW7uAdrdqBaYvUMOvVfYX1X3fSih1ClIqbsQZRdMYq1diyZK9j1jX4YlCcXYSH/d0IbLF56ZrILoWP+QS1etP1LIAnwoTn9s7cBn5g/6mSoajSA==; 5:U1cyE/9m2E/oumPzYWU/KukIcSK/vIo1L/8gL/RrzlwSSc6+6vZP0Euyycr/i3X3nDH5+PmfRU3bXRtrBqAQbnrddO4Iocxjzrz2x2P9qCIdbTRVXGs+C9j5XSqQiVKItlvodikVvKDXzymiQsFSLQ==; 24:fNLvlKvTk/2tNsPfRpPvz7gOyLW1sXv+XxElP4ryBGg/dJFxbiZyAbGeXKwTT0eEqXiFYySp30XKOkApaTKX1WSLv13Z+GkUWR2k4pKz8U4=; 7:KQYyNSVVNVnlNwhIe6ShbAjpBrBean4hrx6lU9nz+/jc2R+MFwB5x07JYu6/IHhoGYf8uKFJdKPYsS4mJcPomAmArR36QEIKBQy+mJgYb7euubZs4tNNjetEWWygcXvUTl8HdFjyajH4OjLHv/rKvfrEhZVUFg8hWfgeKZH5n0Pw+x0pP4gz7CF0j4DvgMS8wvbelwB6GQ7f6rvXDMuchdUizi6MWLEvJR/+RWp3QCQ= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Aug 2017 04:00:23.5745 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR07MB3098 Subject: Re: [dpdk-dev] [PATCH v2 11/12] mempool: honor iova mode in virt2phy 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, 04 Aug 2017 04:00:31 -0000 Hi Thomas, On Monday 10 July 2017 08:07 PM, Thomas Monjalon wrote: > 10/07/2017 16:22, santosh: >> On Monday 10 July 2017 07:39 PM, Thomas Monjalon wrote: >> >>> 10/07/2017 15:56, santosh: >>>> On Monday 10 July 2017 07:21 PM, Thomas Monjalon wrote: >>>> >>>>> 10/07/2017 15:30, santosh: >>>>>> Hi Olivier, >>>>>> >>>>>> On Monday 10 July 2017 05:57 PM, Olivier Matz wrote: >>>>>>> I didn't check the patchset in detail, but in my understanding, >>>>>>> what we call physaddr in dpdk is actually a bus address. Shouldn't >>>>>>> we start to rename some of these fields and functions to avoid >>>>>>> confusion? >>>>>> Agree. >>>>>> While working on iova mode thing and reading these vir2phy api - >>>>>> confused me more. Actually it should be iova2va, va2iova or pa2iova,iova2pa.. >>>>>> where iova address is nothing but bus address Or we should refer to linux >>>>>> semantics. >>>>>> >>>>>> We thought of addressing semantics after this series, Not a priority in IMO. >>>>> I think it is a priority to start with semantics. >>>>> The work is too hard with wrong semantic otherwise. >>>> Sorry, I don;t agree with you. Semantic shouldn't lower the iova priority. >>>> iova framework is blocking SoC's. w/o iova framework : One has to live with >>>> hackish solution for their SoC. >>>> >>>> Semantic change in any-case could be pipelined. It shouldn't be like >>>> Semantics change gets priority and therefore it blocks other SoCs. >>> I am not saying it is blocking. >>> I just say that you have not started your work by the beginning, >>> and now it make reviews difficult (from what I understand). >>> You must make all the efforts to make your patches easier to >>> understand and accept. >> It's just about changing name for virt2phy api's.. But changing those function >> names require deprecation notice, Once iova patchset is merged then I'll >> take up responsibility for sending deprecation notice and change those api >> name in the next release. > This series is not going to be integrated in 17.08. > Anyway, you should probably send the deprecation notice now, > in order to change the semantic in 17.11. > Olivier was also talking about physaddr wording in EAL code. Per above discussion, we had sent out deprecation notice [1], and agreed to keep iova patch series on hold for 17.08 release. Now that v5[2] iova series is reviewed and ready for 17.11. So iova,v5 series shouldn't be blocked/delayed in case iova deprecation notice not merged to 17.08 release. [1] http://dpdk.org/dev/patchwork/patch/26771/ [2] http://dpdk.org/ml/archives/dev/2017-July/071809.html