From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM01-BY2-obe.outbound.protection.outlook.com (mail-by2nam01on0076.outbound.protection.outlook.com [104.47.34.76]) by dpdk.org (Postfix) with ESMTP id 547661B1CE for ; Fri, 29 Sep 2017 10:25:40 +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=JA7x6vFyy3qXZ/K+j+ELPSROiGPYzByiKA/ASFTstdU=; b=Yt4Njhnk7t6as3ZqZ/scSBf7gBXqTzP2GM0D4AK4ET1iiwjs67LESwOjyg/0uTDpvSdZaIF8vRdVeeChyjxbzL1MByOAhnHojw+0/ITf/Ny/ovk88+a5Njg7mwfQUQrHF34KLzvzpWT5TB47/IJWCiA9/Xri4zQZu/HZYSOxx/U= Received: from [192.168.88.32] (31.216.236.195) by MWHPR07MB3102.namprd07.prod.outlook.com (10.172.95.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Fri, 29 Sep 2017 08:25:37 +0000 To: Olivier MATZ References: <20170906112834.32378-1-santosh.shukla@caviumnetworks.com> <20170907153042.30890-1-santosh.shukla@caviumnetworks.com> <20170907153042.30890-9-santosh.shukla@caviumnetworks.com> <20170925114141.ynhho56eciqlyxqu@platinum> <8326c640-93f9-2662-56fb-26f50e42161c@caviumnetworks.com> <66635b0c-2a96-a1cd-4f99-89fea50faf11@caviumnetworks.com> <20170929082049.hrbfrysfjudht3ip@platinum> Cc: dev@dpdk.org, thomas@monjalon.net, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com From: santosh Message-ID: <63375a8d-1799-1547-3c72-417f4af9604c@caviumnetworks.com> Date: Fri, 29 Sep 2017 09:25:27 +0100 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: <20170929082049.hrbfrysfjudht3ip@platinum> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [31.216.236.195] X-ClientProxiedBy: DB6PR0801CA0047.eurprd08.prod.outlook.com (10.169.219.15) To MWHPR07MB3102.namprd07.prod.outlook.com (10.172.95.8) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: d6e1b1f3-83bb-4c4a-9ccf-08d50713aa16 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:MWHPR07MB3102; X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3102; 3:3XosI2dlgjPcKPW0MVKbKf+aZYH/NmrtQJgQ6nIGT8aTMLyIlhNXx7/fbS4YVqKID+12I8FnjbPHcCCFDUwVp8VBgafpJLcSC5A/Dr+21KF0OdSVAb/CLpb2GZazTa8dJgUhSLwM5h5VubhNaO8uKQOZSkVF6G7Yffreg7IVp+ApwKkfQjHakh651/p3e3ZsSYEaHRT1UK0qcO0qwFtXe5NVD+RlBaQQ0c+B4O2Rw1C9nyg6fLK0XWiOgeo7nYXQ; 25:LwJJKFekIpKm8f6LS73KWrATB/2qvvgDZ5LdrDFFZ0IQ3XQAEOXheX28R1gphmJKlDvaPIs6haXMJ/ZO0AD5s3YwLHNrNxPvijHewJR4/++zYi2IABzBCBXBGeocgyVIDuazCsXS3wq7LC1J78q6dXfqUGGIke7O7NLhyN0jUisxXTUgpNCNF1VkanwExsCh8WC5w82FTYJrgMn0lGvpCUVcW9FscJeVyAOFNRdqbD7h9EWH7tNsYUYQejXnJgUl2iZDt8VRpcxxta7OUd9bYxE4Hn9penufLCmHOnpcNtx4BP1ho7neaDDVGgBAbsf0R5YxLHU49YNrHxi351nurg==; 31:zkyLiDvbXYsymh4XV3bXgccXIV4KziGq4iY9s9sSGQAS3No/rqp5ak6jd0EMSeD4oFw+DHPTcvZt7JNG2FTAU/Eb8JV/favAC5tltjZc2d3oyScQiTlOxEgPw/HvMcDJCVgegwrqhoId/Rbg0fjU8RqmGT17/g5R4hI3gAeLOd5AjxRZn2UgHrgRUbLRP4fQXyOtGqJbdzhvKRuOuQPCS/KfZQUBRi7pa/hn/1X/VUw= X-MS-TrafficTypeDiagnostic: MWHPR07MB3102: X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3102; 20:HJ70UN3rdcjVpK3+actKjd1ZXvtlN7rDyGfkTMrS3B+Bl94qpMpJoDa098IzjS/dqs7rfCgAA/WHhE7SAUxuGcbk5RW7JjDH9v5Kik1hCRTFMPTNkk5hoNikZXNPUixA+zeLoZnN3xxfSiK07+Ugz8k5UrLKXwlm/P1Z738pBJVyknqtIhuHP/piyR7EfqGBxi6/9oVUGnUbKfH1RSv9Pi90YsHdRoyXwXHKAVlEBkYbpsl+oJMlyjhLN2dy6N/F1giBPj+7+7y0HQ5bRw0DsTo6JLAvtx4OqsyG1XXbv6sri6kESYWxbPWE7UdrpiEOMUXYuq7Axy9RUaqvmJvNfgwTXY6NTWBHOl0Qd8G+67Wp22LPc7EvW2Ba1hkCmMtEl4i1Qt0X93fM8hyU99lOkw5grD0Aaxug1DfWjRlfAXetRXaMlgYoL5wYfHw9mcyPuKL/X9h7/C6KyvalTBv4mrumTLj7GPnQgk0Rt5+5dNAlnDwWYqKNuCEh0WIJIsspiiUb0rsYfeKpfdeIje5MC+jfkXlPLSWCZbPb8bbE+1Qz9FvMdfJ8nX2bW1CTSZw3bh7Cec3qz8sJsHz90aPE75HxwY3WozV/RJa1u3UdjBE=; 4:+Gjt6FdqP7IWnViAm3agKenQaV8uYLajxyZCWXwg0gbqhTUeDDevSuIPwI7I+14AhjH5RUwXKcHmKDEFas4hieQmZghIyVU8MpFjMKR+14gxrO/ucJM0tOm3JOmvgicYVOd89B11muI67v8ywp9IabuNP8C85B5pSqQhOjszBeYD+Zfotixt6ILdvDzFODYkfR+VpG68RTmAP7hUT2vTgv3uDJyCy1oSNCRJi+UIUxxVJyqWieoaefmeoeXOYqI/ 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)(2401047)(5005006)(8121501046)(10201501046)(93006095)(3002001)(100000703101)(100105400095)(6041248)(20161123560025)(20161123562025)(20161123555025)(20161123558100)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:MWHPR07MB3102; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:MWHPR07MB3102; X-Forefront-PRVS: 0445A82F82 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6049001)(6009001)(376002)(346002)(189002)(199003)(377454003)(501624003)(24454002)(105586002)(64126003)(6666003)(2906002)(90366009)(189998001)(305945005)(77096006)(7736002)(31686004)(6486002)(106356001)(81166006)(65826007)(33646002)(229853002)(42882006)(230700001)(5660300001)(8936002)(31696002)(6916009)(8656003)(50466002)(6116002)(2950100002)(3846002)(117156002)(101416001)(50986999)(53936002)(76176999)(81156014)(6246003)(47776003)(83506001)(25786009)(68736007)(53376002)(65956001)(65806001)(66066001)(93886005)(8676002)(54356999)(36756003)(16526017)(478600001)(6306002)(4326008)(16576012)(966005)(72206003)(23746002)(86152003)(58126008)(97736004)(316002); DIR:OUT; SFP:1101; SCL:1; SRVR:MWHPR07MB3102; H:[192.168.88.32]; 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) Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Santosh.Shukla@cavium.com; X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1; MWHPR07MB3102; 23:ywlOMaWgL6dNga48I5uL6L+72APveEAnCBlLN?= =?Windows-1252?Q?uqRce9IfwgvvmTDof9tgbx/hpfUm3B7yt6enU3xF6ePlAYw1WL3CAvIc?= =?Windows-1252?Q?0+NscYrO44on5JogVMEjvbKKyiZf9wFsUUjJvg5s18XPXq+1jnhS03N4?= =?Windows-1252?Q?v4tNyh9BgnWQwd//tNTxUj/Zc1Knl1cGgotcOY4vSVAttMugHoK10pTr?= =?Windows-1252?Q?Qd0IAZ92pLVj28/Nf2PjBQUNFbNFtyVhjvKkQe4zH5VwjccQAIoc7zk8?= =?Windows-1252?Q?4RDWERLnHqxS6QiEn6YVtK92ayc+WO50vv8G3Pc/cshzxMfH3fwHZAOt?= =?Windows-1252?Q?RUE9iORis5hvwNTzA5zVcNJ89MbqiS+4ZgghxBXzFAGJuhVfWXPQp/GN?= =?Windows-1252?Q?Fjnfl8gYn5mN3dD0DLW825kdr6/KxDuj7e7dXweWCjnfE5Y+42MFxfpY?= =?Windows-1252?Q?BURHKKgag5fyMQzr8HS61JjUg/MxRui36PQuuGT2GvbYC45Xy+HSmuZv?= =?Windows-1252?Q?Mll+xH09ATSG8wQp9SVdjUJLTxOoTgmNXsnRZaYdaHRFKdj996Cessu4?= =?Windows-1252?Q?6poTg58XGArrw4H4VIXBPt0bFHTSeK6iu+zyXJki8aSvXCv9G8TKV6gB?= =?Windows-1252?Q?Kyz0jsAPEAvm2pziF0YvSx4JxPXI11Ycn0O+o8RplrGIOkIwu3EPoJut?= =?Windows-1252?Q?e7TSh7+fMNUvZw+lW+2rrbm8AD3JqnODmebgW4cx9e7SFJqkGHbd8D7s?= =?Windows-1252?Q?pql9EetVlSGQvg7sg/49f/Pbq41sD0/knOJLD7CnsyoDel0J4c1UOL2S?= =?Windows-1252?Q?RouVPmUILn1qsj4zvbeMn5OdlbIWwffwUOTOgqYuVwsRuqPsvWvqjdcB?= =?Windows-1252?Q?tN4L4a835kddBX6BqDf7QFhoJfXjYrYvagPhwo1dwFDWFXT4/O2CMtEk?= =?Windows-1252?Q?bmkcFwNBOxRUcwgHUdwqyKm1BN+dg96zlpOXz0kyeLXu+tGqtwcEvsAf?= =?Windows-1252?Q?OAbowoIZEd78oC6x2igiFkN0QtKVIZ8gnn3Kv1xW3zE0cwGrkxr2QYFD?= =?Windows-1252?Q?Wwi2YowTYdt7aEZuavEMOmk6WmWXbrzniYivm0gvKKcRhtTmpEvPXVxV?= =?Windows-1252?Q?/FTc+v133jqHKw+pHT2EYic/Vju9aXsyAaEhTd6vK2L4HGxhMgI045Sp?= =?Windows-1252?Q?ubLKldCrsw51GgxJ1vuoXDY/7AghrCQnPRyL2QzdZqJ2tIxMP1RmM0yL?= =?Windows-1252?Q?tvo2TRMZDkcYfpVtMeA+TPhz4Mi8uWjcsfMWnVqbjUa/JkMxhAWu+oRM?= =?Windows-1252?Q?WX/jz8+OvOSnfCqFst7chYM23pYm4/WdJ9vOw6csZDPqqON7dq3oBxAY?= =?Windows-1252?Q?07PFPZXtnO8GcLV6k21tu91KcgK9n/z3hB23GIGV4RqjNtk+wGcHfq6s?= =?Windows-1252?Q?QKq3EHnBDooXS9yV4ynTomnw+afqgEwfEovbg7tmdQi+rtnVDzvsyA0M?= =?Windows-1252?Q?pFYmu0d7VgJSCInmEijnmv02fqt03hJ4XvJLbj49aaeyuWJpx3BTs+Nl?= =?Windows-1252?Q?aJOzie7cmgYGNTwFCWyGsScfV1EYgrbK1OEZtFM6FKfddBt/herx19Nd?= =?Windows-1252?Q?0P+tBpiPHil8dYmGw7FbvbodjR8wuJ7b7k2X8/kW51o?= X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3102; 6:jquzu6LPSHikPKedzT4p86DJRg9CBGSPfObqdLRZTs5+X9hmwBRyrtZFWw6IZxVAHfIaSI07DHwkflBQWojs5o17iEeu2mJCP7Hkyy1G3UP1MJmF6IniNtT4/fpJov/sHmhd3Cx2L4WcaGfwg57fTKTG5wJnxtIGH19zKqERQwjRICE1XEoKo/uCpywNNsC+qUHMJOJsd6WkIrr7r1WFfptjvt/Y1c1x71AWWoAHYdFHq9s+6b+pxksBvs8l5Zp/l6FgADd5CHy1C7mhq+6kq58m+yft+8RKDgpwANTxRks9VzhlCngSmtRqXpmxCDChRCzv7H1O4ndMRE30xVeSsQ==; 5:W9Hb3TglhdNNpj38rerTrDoXqJeupVVyy+SrGDu/Hv7tu8Wh3nqEqc4aYGsHQQy3hWqT/C7cR64GHoipl5SPn1ZykORAhGQDUMJ89fdF6cL9ZyA7sncCCrKE656a5J2aNqiJZJvnp39LlWcSdL32BBcoUDYg0cNasdnzTq+My7o=; 24:OZg1k5iyrjvXvr2oVgM18wqLaBUvVXZFlEVILUnBqrib9emvhnymt/j3OMepCMFUD+6TTZpiqUp3Mz1QOqdPsBdDo8vdoNB5aSibA41dCfM=; 7:udbwGVDqiE3w63NjRi0kNXEv54lUyxnD3TGTy+UgnPLJjff52oR/GR6ubhx+ZilntBNzZ4GQlgZRm0vTqy8nXH014slknBkOF4OIXdqouEKPR5wjQoM4uWx5yTlAtScsqfTib3MwqzWDMebFP0WQ6vWyPTUhixOVnzrNWFa87em9WO+ZfDwJxLxg27uhTHy1h97GIPJWquOtkDypY9QGs5B76jRP52XUteu+4M4Shzw= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Sep 2017 08:25:37.1706 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR07MB3102 Subject: Re: [dpdk-dev] [PATCH v6 8/8] mempool: notify memory area to pool 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, 29 Sep 2017 08:25:40 -0000 Hi Olivier, On Friday 29 September 2017 09:20 AM, Olivier MATZ wrote: > On Fri, Sep 29, 2017 at 05:53:43AM +0100, santosh wrote: >> Hi Olivier, >> >> >> On Monday 25 September 2017 11:18 PM, santosh wrote: >>> On Monday 25 September 2017 12:41 PM, Olivier MATZ wrote: >>>> On Thu, Sep 07, 2017 at 09:00:42PM +0530, Santosh Shukla wrote: >>>>> + * Mempool handler usually get notified once for the case of mempool get full >>>>> + * range of memory area. However, if several memory areas exist then mempool >>>>> + * handler gets notified each time. >>>> Not sure I understand this last paragraph. >>> Refer v5 history [1] for same. >>> >>> [1] http://dpdk.org/dev/patchwork/patch/28419/ >>> >>> there will be a case where mempool handler may have more than one memory example, no-hugepage case. >>> In that case _register_memory_area() ops will be called for more than once. >>> >>> In v5, you suggested to mention this case explicitly in api description. >>> >>> If your not clear with write up then could you propose one and also are you fine >>> with [8/8] patch beside above note? planning to send v7 by tomorrow, appreciate if you answer question. >> Ping? >> >> IMO, remove above description keep it like: >> "API to notify the mempool handler if a new memory area is added to pool." Is it ok with you? Can you pl. confirm, I need to send v7 and we want this series in -rc1, its blocking octeontx mempool and nw driver.. delayed review blocking progress. > The proposed description is ok. > I have no other comment for the rest of the patch. > Ok, will send v7 with above api description. Thanks.