From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v6 1/9] eal: move librte_malloc to eal/common Date: Fri, 03 Jul 2015 11:02:28 +0200 Message-ID: <9388529.0Xl0Ea6jym@xps13> References: <1433586732-28217-1-git-send-email-sergio.gonzalez.monroy@intel.com> <2707498.ThPVHMap7h@xps13> <559644F5.8050506@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: "Gonzalez Monroy, Sergio" Return-path: Received: from mail-wi0-f172.google.com (mail-wi0-f172.google.com [209.85.212.172]) by dpdk.org (Postfix) with ESMTP id 966AC902 for ; Fri, 3 Jul 2015 11:03:38 +0200 (CEST) Received: by wibdq8 with SMTP id dq8so94540405wib.1 for ; Fri, 03 Jul 2015 02:03:38 -0700 (PDT) In-Reply-To: <559644F5.8050506@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" 2015-07-03 09:16, Gonzalez Monroy, Sergio: > On 02/07/2015 13:14, Thomas Monjalon wrote: > > 2015-06-26 16:29, Sergio Gonzalez Monroy: > >> --- a/MAINTAINERS > >> +++ b/MAINTAINERS > >> @@ -73,6 +73,7 @@ F: lib/librte_eal/common/* > >> F: lib/librte_eal/common/include/* > >> F: lib/librte_eal/common/include/generic/ > >> F: doc/guides/prog_guide/env_abstraction_layer.rst > >> +F: doc/guides/prog_guide/malloc_lib.rst > >> F: app/test/test_alarm.c > >> F: app/test/test_atomic.c > >> F: app/test/test_byteorder.c > >> @@ -97,6 +98,8 @@ F: app/test/test_spinlock.c > >> F: app/test/test_string_fns.c > >> F: app/test/test_tailq.c > >> F: app/test/test_version.c > >> +F: app/test/test_malloc.c > >> +F: app/test/test_func_reentrancy.c > > I think we should keep a separate maintainer section for memory allocator > > in EAL. I suggest this: > > > > Memory allocation > > M: Sergio Gonzalez Monroy > > F: lib/librte_eal/common/include/rte_mem* > > F: lib/librte_eal/common/include/rte_malloc.h > > F: lib/librte_eal/common/*malloc* > > F: lib/librte_eal/common/eal_common_mem* > > F: lib/librte_eal/common/eal_hugepages.h > > F: doc/guides/prog_guide/malloc_lib.rst > > F: app/test/test_malloc.c > > F: app/test/test_func_reentrancy.c > > > > > Fine with me. > Do you need a new version of the patches with that change? Yes please. Thanks for your involvement.