From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dispatch1-us1.ppe-hosted.com (dispatch1-us1.ppe-hosted.com [67.231.154.164]) by dpdk.org (Postfix) with ESMTP id F2C565F71 for ; Wed, 14 Mar 2018 16:57:43 +0100 (CET) X-Virus-Scanned: Proofpoint Essentials engine Received: from webmail.solarflare.com (webmail.solarflare.com [12.187.104.26]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1-us4.ppe-hosted.com (Proofpoint Essentials ESMTP Server) with ESMTPS id 76DBDB40094; Wed, 14 Mar 2018 15:57:42 +0000 (UTC) Received: from [192.168.38.17] (84.52.114.114) by ocex03.SolarFlarecom.com (10.20.40.36) with Microsoft SMTP Server (TLS) id 15.0.1044.25; Wed, 14 Mar 2018 08:57:38 -0700 To: santosh , CC: Olivier MATZ , Jerin Jacob , Hemant Agrawal , Shreyansh Jain References: <1516713372-10572-1-git-send-email-arybchenko@solarflare.com> <1520696382-16400-1-git-send-email-arybchenko@solarflare.com> <317a928a-f8de-13da-74bd-28fa0b5234b9@caviumnetworks.com> From: Andrew Rybchenko Message-ID: <4dae318c-ce21-185a-a6eb-3f193aae2ae9@solarflare.com> Date: Wed, 14 Mar 2018 18:57:34 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <317a928a-f8de-13da-74bd-28fa0b5234b9@caviumnetworks.com> Content-Language: en-US X-Originating-IP: [84.52.114.114] X-ClientProxiedBy: ocex03.SolarFlarecom.com (10.20.40.36) To ocex03.SolarFlarecom.com (10.20.40.36) X-MDID: 1521043063-6LN3KafvliB2 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8bit X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH v1 0/9] mempool: prepare to add bucket 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, 14 Mar 2018 15:57:44 -0000 Hi Santosh, On 03/14/2018 06:49 PM, santosh wrote: > Hi Andrew, > > > On Saturday 10 March 2018 09:09 PM, Andrew Rybchenko wrote: > > [...] > >> RFCv1 -> RFCv2: >> - add driver ops to calculate required memory size and populate >> mempool objects, remove extra flags which were required before >> to control it >> - transition of octeontx and dpaa drivers to the new callbacks >> - change info API to get information from driver required to >> API user to know contiguous block size >> - remove get_capabilities (not required any more and may be >> substituted with more in info get API) >> - remove register_memory_area since it is substituted with >> populate callback which can do more >> - use SPDX tags >> - avoid all objects affinity to single lcore >> - fix bucket get_count >> - deprecate XMEM API >> - avoid introduction of a new function to flush cache >> - fix NO_CACHE_ALIGN case in bucket mempool > I'm evaluating your series in octeontx platform. > Noticed a build break for dpaa platform: > CC dpaa_mempool.o > /home/ubuntu/83xx/dpdk/drivers/mempool/dpaa/dpaa_mempool.c: In function ‘dpaa_populate’: > /home/ubuntu/83xx/dpdk/drivers/mempool/dpaa/dpaa_mempool.c:291:41: error: passing argument 1 of ‘rte_mempool_op_populate_default’ discards ‘const’ qualifier from pointer target type [-Werror=discarded-qualifiers] > return rte_mempool_op_populate_default(mp, max_objs, vaddr, paddr, len, > ^ > In file included from /home/ubuntu/83xx/dpdk/drivers/mempool/dpaa/dpaa_mempool.h:15:0, > from /home/ubuntu/83xx/dpdk/drivers/mempool/dpaa/dpaa_mempool.c:28: > /home/ubuntu/83xx/dpdk/build/include/rte_mempool.h:490:5: note: expected ‘struct rte_mempool *’ but argument is of type ‘const struct rte_mempool *’ > int rte_mempool_op_populate_default(struct rte_mempool *mp, > ^ > /home/ubuntu/83xx/dpdk/drivers/mempool/dpaa/dpaa_mempool.c: At top level: > /home/ubuntu/83xx/dpdk/drivers/mempool/dpaa/dpaa_mempool.c:303:14: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types] > .populate = dpaa_populate, > ^ > /home/ubuntu/83xx/dpdk/drivers/mempool/dpaa/dpaa_mempool.c:303:14: note: (near initialization for ‘dpaa_mpool_ops.populate’) > cc1: all warnings being treated as errors Yes, my bad, const should be simply removed to match prototype (and mempool is actually modified since it is populated). Will fix. Many thank, Andrew.