From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0046.outbound.protection.outlook.com [104.47.42.46]) by dpdk.org (Postfix) with ESMTP id D084B2F7D for ; Tue, 26 Sep 2017 00:18:36 +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=hL7BrUDiiIQZzrkF5j8qOrckzZ+0yC5yK4FYQgPoo7M=; b=H87VvdMKvRCo4KAucofc/Z54TI8iUSMbxEEi1xAGMbBHL9FSvOcHgrOFpTjD3OOz99+UMN1axg2ZXIjM8eO6+Rs3wc8uRXi6hLhOsiEA/Os9gv3huJPq/amA1U/wQByJM3XABbJBPvXSyiI6OAL+yjMkLO2sW6LXOycEABSV4a8= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Santosh.Shukla@cavium.com; Received: from [192.168.89.224] (31.216.236.195) by CY4PR07MB3093.namprd07.prod.outlook.com (10.172.115.7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Mon, 25 Sep 2017 22:18:31 +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> Cc: dev@dpdk.org, thomas@monjalon.net, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com From: santosh Message-ID: <8326c640-93f9-2662-56fb-26f50e42161c@caviumnetworks.com> Date: Mon, 25 Sep 2017 23:18:17 +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: <20170925114141.ynhho56eciqlyxqu@platinum> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [31.216.236.195] X-ClientProxiedBy: VI1PR0901CA0086.eurprd09.prod.outlook.com (10.173.68.12) To CY4PR07MB3093.namprd07.prod.outlook.com (10.172.115.7) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 443621b8-8695-408e-99ef-08d504635cb7 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:CY4PR07MB3093; X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3093; 3:NVPp1EDhHLSKdOguJ7K8zooYpt6fqbhF7Bqlf8/W5JQ0Gs9m3fOdFRfjtHwSc9tQFirnVyKfL63ca6XevJnH1+4ZMMzl5Dk8Py1/vNWEMdjtOLtfsL7hAYwRmr1OEP++4KUFBlgxFKrcMYP5f+SqiMM2IhBj4beph0f19Nh4BXEHFhqGZQLznMHKNHUUaK+soTmEz2r6SPST4Lf+1BUEx8cgKMnKgE06lq8Knuo/84D7Kf4vInc3mhdUgHNiS60x; 25:M9D+po97URzufu0h+rhQY66YOPuEiDSS72oA5EIri1ZmdEF9ylYpDcTXYbGOvkMRhLd0B0g0K+y5TMg0Z/NGNQ1QMaPCMn5dKfpkR6UdYDW4dn+9KSqdldZ9s6IQMg6/K+T4EHbKReXA+i0YvdBW44hLucKj0oMHLJWBDXf9UjuGk/q/mibwz+67JMXosqR2h6kHkRFfVHSv8lT9FhO+XC3s6zPCGOsrclYfYo6yDd4w3z2ejKgGTLSGPpBu4l5MkZ4bNEwIzK6Chq3HO/k3pRgBIBD4B+Y2UqVprfODcWMDGZtmL2ir9tVzeapBHT/jEL/A1rPCrBM5qTwp9GrUtA==; 31:96YDpyEdC1QYeVRLXuidcRZiPm8/cD5+zHFm5wVS8l7CsV06cbNzvhvAEQHJbkDTf6o2uT4IEaJZ9nmnfuE6Kss2oe7zVK7uADKLt6JwJW8buZswKK44NzKjbs03baKQKbIeFfOMIXE2UwD8OdaWet/NYiiVU7uc2OEuQ5uUW6F0fgEHb53yu8WiUgEZoGGMQqSvd+/pZu6zEpWnwBKOP7iE1C0L2DqISvDbklS5Q5g= X-MS-TrafficTypeDiagnostic: CY4PR07MB3093: X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3093; 20:hXn2N+HZ9fnftuiF131jrDZrV3JL85A7bBYL4lw4yqkBtbmqJqaDF66LeA8xy6GJ1xu6QVRTXmHlPoAfZUFNqCLtlP9ILs7mbETdYPOcE+BSezm4CIUTfMkQvfqQXC0LaZECaom8/WX0MVRDFW8GXq3Os1uw4vthCn+1EotT2Av/0Zch+buiTpWuLKvZTj0Y/QdDx3ySKjeGebx48ZkKOqNjKJaBz+gWXj6zG41DwFGzKVL0azkGE0PXoZ4FuoFlsJnhZlUAYvn/2r6W1c6rDOVbluYKe4iOAi4Jh7gC2LOEQwV0IqXHNqKiy3exsZBG/xqZIqRGThTjlxqavVlIO4EWPB2rWuo9Ga/9ZRWA1HkGFrJwi0HSJK/ZrAVSacaz0DIO6vQI//YyWiFKJi9aF9fRDmSUznL8WvCEdW//SQtrBD38/TYMMrRY1o1SR/glQZZA9EREuiE4rsP4XSGI/+5YV4xU8sLN2D53WrXT2NFsb/O43eenYGKCRppbHoBYdACGvraUFWRWw1j203SHv1ThTh6AKACM2ecB7nbII5mFEx3GfkTvhXHtb3UBIKFWppk/DZd7CiixyVG1gt7NnD2ZuKb/4rVBp5AECwN1pbQ=; 4:55WQyf4m/zP33ZZHLrUl9wEEHKnhMTJf7iBJvwDbBvm+85GricTPOxilp8+Wu7bQfFeozwA59M1Z+0Kl6HwgAGiGQJhDN0Ty4ajJIQyyaF6uHMCHyLmncVv0zLHNG/Sjn7qIq2+s1FVKYAD/weOPu4bkehQkSzpzytkgzowqYV0YQahRGrNG1SioAPoPLPAsTkeTvHsWSjJywd0vNiYUBWewt5m3XAZlRZpGti3NQ7l6USgHzPn14BcT53jf/VrF 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)(3002001)(100000703101)(100105400095)(93006095)(6041248)(20161123555025)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(20161123562025)(20161123560025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:CY4PR07MB3093; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:CY4PR07MB3093; X-Forefront-PRVS: 04410E544A X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(6049001)(376002)(346002)(199003)(501624003)(189002)(24454002)(377454003)(65806001)(16576012)(230700001)(65956001)(66066001)(189998001)(54356999)(76176999)(8656003)(33646002)(97736004)(68736007)(229853002)(47776003)(50986999)(3846002)(23746002)(65826007)(83506001)(6486002)(90366009)(77096006)(36756003)(6666003)(6116002)(6916009)(86152003)(58126008)(966005)(105586002)(2906002)(106356001)(72206003)(31696002)(6306002)(42882006)(53936002)(2950100002)(117156002)(305945005)(478600001)(81156014)(101416001)(25786009)(16526017)(7736002)(4326008)(316002)(53376002)(50466002)(6246003)(81166006)(31686004)(5660300001)(93886005)(64126003)(8676002)(8936002); DIR:OUT; SFP:1101; SCL:1; SRVR:CY4PR07MB3093; H:[192.168.89.224]; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX: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; CY4PR07MB3093; 23:C2XvY/KA0QfvX5lRu8Suh+geKoFWmlWWxHKfx?= =?Windows-1252?Q?mZZlQK2yATb6U0HHxaBS2r+Zmz4Sd8w9tj4QFGAeq5D41yklkpyFTmRi?= =?Windows-1252?Q?fhgU6HBSGvJlPDA5SexHKyFreOctG4RxG4NVswf/LeTIDxPvLpS8QejC?= =?Windows-1252?Q?V9M0rIEWOdwPB0HNxN1d9TOWH3pJ+cTmuOSNwduB/8kr8CuF/FOeCOEY?= =?Windows-1252?Q?LQaNWJUNSl4L08VByuj8ZXqr/VgagpBtW8PXNeJGdqLuVYwijDoAWtFe?= =?Windows-1252?Q?B6f3PPFy+cCI8j3x6BRqS5jBQp+qUf5Xjj89vcm531VmagzhXXIl8UEp?= =?Windows-1252?Q?nLmNXogJ8CtYirp8OfON7NMsRrOasNIysQHmr+NBJkxRBW1quqS3JmiQ?= =?Windows-1252?Q?K4/fEQ9kDNsIQjDHzR8dCQunadV91jeE2XVljZj/JGjIWUfnsXcZa6LP?= =?Windows-1252?Q?6Nli5VRCAcbrvLemFNHwj6lNkoUvHqNbXYkAfbX1ViWpqDEBV4XhoOFt?= =?Windows-1252?Q?L0QR1cfs9lF63Wuuv+L+PiPO5cRdCVeIN9hxIFmCcScdREFwSER8dH1M?= =?Windows-1252?Q?zqaHFttF7c72jkpsv+owP3oQNQEXPPNTrfKQzKwF0Bo02nWCae8ma3dR?= =?Windows-1252?Q?NlOR2pvdRq6xahwR0eDhvRRzQweT/aExXR0U/uiYdZUUF14SpVtZ+6dO?= =?Windows-1252?Q?cThHpDuGUitAHhWmXcDcYeR2L+G7mtrHrYv5PpaPpn07gFVdpq3pul5b?= =?Windows-1252?Q?80r1QtupUOvsn5YePuffUOh/uzsZ/mPb7BVBQY+5GebRV5Iq0NHJKWoK?= =?Windows-1252?Q?tXhyvnbmxZDD0yNE6TAXotxAWk5JcQEK9bbbDjcJkW15npfgMgw4aktg?= =?Windows-1252?Q?9JK8RkHM5BnBUy5hNi7uOg50YDpLbKRjHnDXLd7eJ4gLn0o2p87tsGPF?= =?Windows-1252?Q?kmAyU6zA3iiRkeCHOFSN2qnPQ8H1w6+7B3rhMXvyGfOflJwiqM5VFfQ2?= =?Windows-1252?Q?+rBFC6RHBdSyQ9ITYp9cmXYj6O18FVDTLSYTUaDzuoXbPknS4gKvZBJ/?= =?Windows-1252?Q?L0OGE9B9gc7Ab0kkAi0PYS0h0PWdhSR9XF37DQKi7dABzU4bcN/2RGbP?= =?Windows-1252?Q?kKJkA/d15FOywkUXNMwfHZ6CkkePW7C+aV8YdYWiHMuvrTo1lnvbfqA9?= =?Windows-1252?Q?5bpV2tuz9q5tMuYwOrmaBRJCh6s0rsGhRd6NQxVh+jhHQ4sYnK6MWD9X?= =?Windows-1252?Q?6jiPstq3iFMEU/zlFwmRqlHgSkWzNiAq/9IE+xW52OCbRCRqJ5CdUMD6?= =?Windows-1252?Q?Z56IHTvC3EqK20SH5noBjKIAN7cvO/PE/EcBpkkGsSnRm19dnhOGCzyA?= =?Windows-1252?Q?BahdxbyEYsXS6zmKSqUAmnqz7pVH2XuE/WRZJKHwMfHqjPw4YBh1Pq38?= =?Windows-1252?Q?z8BPUTLZH7y17fs42u2MWBvT2zOH5C9i1Qb7HzZGo0vLGgx30S2nKmtE?= =?Windows-1252?Q?QDzaC1ni6jL0jXFSdnjc2vCNdIrQ+jGas2OoUP4Zl6ilr89pVtubXtiy?= =?Windows-1252?Q?8AGRrSxhPTDC6AcWrVuubzFCE0V+c8FqdV/DAtCwg3QW+fqBAEj6ZLDz?= =?Windows-1252?Q?nMhMol3P6q/VHbt/b32I3FJ6SkRLgEBjvalnyFHlmdJ?= X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3093; 6:7WvwKs3+9uGhqhrFklRdhNapQL0sVhROO78W6W/urO9GTgGkFf11nnTo7TPGJ2SRpVRVqZTzI5iEom7ercHaKCqz6Z02+nobJDhjBfM4XF/vBuxS/dTFpBOw68afTdb8qx2ctdPUsV6DGZHqMuVNkxceHZ5FDhmoLhVUJ4sCRrgh24ncv4D6IX8VdfbIW39Hz707CS1k74/eL/wrxp+U5vFp0i+cWap7Fz8DsRxA6G6c8rz0WvkwhJ2YscuC8hdkGMXarH7SrrQFCB8lCepxTHaZT3mNiLAkberNn4QliPNZUJ4mRNZfwsOAoYAkRqAkZ/dP7YHXFRETyHG7a5Pb+g==; 5:ygz+g+0oMfj0GozERxscVGV/E7dCb4pmQXYrfVjdHQH+OiHD6KshgbdXWXfGYEI/QhOH0XxeIy0GRVEcxTsZJnJxImOcqV/6qhP1vUpn9wkEgw6NbknEdl+q41XF/3CmEt6BT9g6bzhm4joPJoFq9Q==; 24:pem9jwPt7xaYXrOZvouZVNt/48vQHFKGyAN3tZaPKJv0dcksTgwxMzQ6YE3ZHacRwN5gEC/tiI5Ik4iN4wstLZmN26/spAyBqDAu2Aq2RjQ=; 7:jpjR8kCz2q7H77pG215QaKe13x8c93pqRJC6ht+M3JEHJ1RN3k9PNKwrw3ix1mXR3G/sv+DtHia6BYNBltyWDcNGC5Inso9Y3W2PEyAmQj/Ifzn6g6M8ka3ingt4rSNaioNO2N0wY32Zycn5RuPcBdnOwCpF3AINbdlAOFmmMXn57Zpdp/t+yDen5ZBUFHH+Ke92EvOEBfsxmUqfvCRCY2028yOH49t+WBegxfX1Tdg= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Sep 2017 22:18:31.6602 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR07MB3093 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: Mon, 25 Sep 2017 22:18:37 -0000 On Monday 25 September 2017 12:41 PM, Olivier MATZ wrote: > On Thu, Sep 07, 2017 at 09:00:42PM +0530, Santosh Shukla wrote: >> HW pool manager e.g. Octeontx SoC demands s/w to program start and end >> address of pool. Currently, there is no such api in external mempool. >> Introducing rte_mempool_ops_register_memory_area api which will let HW(pool >> manager) to know when common layer selects hugepage: >> For each hugepage - Notify its start/end address to HW pool manager. >> >> Signed-off-by: Santosh Shukla >> Signed-off-by: Jerin Jacob >> >> [...] >> >> +/** >> + * @internal wrapper for mempool_ops register_memory_area callback. >> + * API to notify the mempool handler if a new memory area is added to pool. >> + * > if -> when ok. >> + * 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. >> + * >> + * @param mp >> + * Pointer to the memory pool. >> + * @param vaddr >> + * Pointer to the buffer virtual address >> + * @param paddr >> + * Pointer to the buffer physical address >> + * @param len >> + * Pool size > Minor: missing dot at the end ok. >> + * @return >> + * - 0: Success; >> + * - ENOTSUP: doesn't support register_memory_area ops (valid error case). > Missing minus before ENOTSUP. > The dot should be a semicolon instead. > ok. Thanks.