From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH v8 0/9] Dynamic memzones Date: Tue, 13 Oct 2015 17:12:00 -0700 Message-ID: <20151013171200.0a994215@xeon-e3> References: <1435917343-23032-1-git-send-email-sergio.gonzalez.monroy@intel.com> <1436864233-24746-1-git-send-email-sergio.gonzalez.monroy@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Sergio Gonzalez Monroy , Thomas Monjalon Return-path: Received: from mail-pa0-f53.google.com (mail-pa0-f53.google.com [209.85.220.53]) by dpdk.org (Postfix) with ESMTP id 9AC208D9D for ; Wed, 14 Oct 2015 02:11:50 +0200 (CEST) Received: by pabws5 with SMTP id ws5so4680145pab.1 for ; Tue, 13 Oct 2015 17:11:49 -0700 (PDT) In-Reply-To: <1436864233-24746-1-git-send-email-sergio.gonzalez.monroy@intel.com> 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 Tue, 14 Jul 2015 09:57:04 +0100 Sergio Gonzalez Monroy wrote: > Current implemetation allows reserving/creating memzones but not the opposite > (unreserve/free). This affects mempools and other memzone based objects. > > From my point of view, implementing free functionality for memzones would look > like malloc over memsegs. > Thus, this approach moves malloc inside eal (which in turn removes a circular > dependency), where malloc heaps are composed of memsegs. > We keep both malloc and memzone APIs as they are, but memzones allocate its > memory by calling malloc_heap_alloc. > Some extra functionality is required in malloc to allow for boundary constrained > memory requests. > In summary, currently malloc is based on memzones, and with this approach > memzones are based on malloc. > > v8: > - Rebase against current HEAD to factor for changes made by new Tile-Gx arch Following rules in kernel. You need to fix the 32 bit build and resubmit whole series. Thomas, this patchset should be marked "Changes requested" in patchwork.