From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olivier MATZ Subject: Re: [PATCH 00/36] mempool: rework memory allocation Date: Thu, 14 Apr 2016 16:01:06 +0200 Message-ID: <570FA2A2.3060604@6wind.com> References: <1457540381-20274-1-git-send-email-olivier.matz@6wind.com> <1460629199-32489-1-git-send-email-olivier.matz@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: "Richardson, Bruce" , "stephen@networkplumber.org" To: "Wiles, Keith" , "dev@dpdk.org" Return-path: Received: from proxy.6wind.com (host.76.145.23.62.rev.coltfrance.com [62.23.145.76]) by dpdk.org (Postfix) with ESMTP id 253D04B79 for ; Thu, 14 Apr 2016 16:01:17 +0200 (CEST) In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 04/14/2016 03:50 PM, Wiles, Keith wrote: >> This series is a rework of mempool. For those who don't want to read >> all the cover letter, here is a sumary: >> >> - it is not possible to allocate large mempools if there is not enough >> contiguous memory, this series solves this issue >> - introduce new APIs with less arguments: "create, populate, obj_init" >> - allow to free a mempool >> - split code in smaller functions, will ease the introduction of ext_handler >> - remove test-pmd anonymous mempool creation >> - remove most of dom0-specific mempool code >> - opens the door for a eal_memory rework: we probably don't need large >> contiguous memory area anymore, working with pages would work. >> >> This breaks the ABI as it was indicated in the deprecation for 16.04. >> The API stays almost the same, no modification is needed in examples app >> or in test-pmd. Only kni and mellanox drivers are slightly modified. >> >> This patch applies on top of 16.04 + v5 of Keith's patch: >> "mempool: reduce rte_mempool structure size" > > I have not digested this complete patch yet, but this one popped out at me as the External Memory Manager support is setting in the wings for 16.07 release. If this causes the EMM patch to be rewritten or updated that seems like a problem to me. Does this patch add the External Memory Manager support? > http://thread.gmane.org/gmane.comp.networking.dpdk.devel/32015/focus=35107 I've reworked the series you are referring to, and rebased it on top of this series. Please see: http://dpdk.org/ml/archives/dev/2016-April/037509.html Regards, Olivier