From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Gonzalez Monroy, Sergio" Subject: Re: [PATCH v6 1/9] eal: move librte_malloc to eal/common Date: Fri, 03 Jul 2015 09:16:53 +0100 Message-ID: <559644F5.8050506@intel.com> References: <1433586732-28217-1-git-send-email-sergio.gonzalez.monroy@intel.com> <1435332578-28415-1-git-send-email-sergio.gonzalez.monroy@intel.com> <1435332578-28415-2-git-send-email-sergio.gonzalez.monroy@intel.com> <2707498.ThPVHMap7h@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Thomas Monjalon Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 294FFE62 for ; Fri, 3 Jul 2015 10:16:56 +0200 (CEST) In-Reply-To: <2707498.ThPVHMap7h@xps13> 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 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? Sergio