From mboxrd@z Thu Jan 1 00:00:00 1970 From: Panu Matilainen Subject: Re: [PATCH v3 3/4] mempool: allow rte_pktmbuf_pool_create switch between memool handlers Date: Wed, 9 Mar 2016 12:54:52 +0200 Message-ID: <56E000FC.2040204@redhat.com> References: <1455634095-4183-1-git-send-email-david.hunt@intel.com> <1457517037-71693-1-git-send-email-david.hunt@intel.com> <1457517037-71693-4-git-send-email-david.hunt@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit To: David Hunt , dev@dpdk.org Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id EFE0E2BA9 for ; Wed, 9 Mar 2016 11:54:54 +0100 (CET) In-Reply-To: <1457517037-71693-4-git-send-email-david.hunt@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 03/09/2016 11:50 AM, David Hunt wrote: > If the user wants to have rte_pktmbuf_pool_create() use an external mempool > handler, they define RTE_MEMPOOL_HANDLER_NAME to be the name of the > mempool handler they wish to use, and change RTE_MEMPOOL_HANDLER_EXT to 'y' > > Signed-off-by: David Hunt > --- > config/common_base | 2 ++ > lib/librte_mbuf/rte_mbuf.c | 8 ++++++++ > 2 files changed, 10 insertions(+) > > diff --git a/config/common_base b/config/common_base > index 1af28c8..9d70cf4 100644 > --- a/config/common_base > +++ b/config/common_base > @@ -350,6 +350,8 @@ CONFIG_RTE_RING_PAUSE_REP_COUNT=0 > CONFIG_RTE_LIBRTE_MEMPOOL=y > CONFIG_RTE_MEMPOOL_CACHE_MAX_SIZE=512 > CONFIG_RTE_LIBRTE_MEMPOOL_DEBUG=n > +CONFIG_RTE_MEMPOOL_HANDLER_EXT=n > +CONFIG_RTE_MEMPOOL_HANDLER_NAME="custom_handler" > > # > # Compile librte_mbuf > diff --git a/lib/librte_mbuf/rte_mbuf.c b/lib/librte_mbuf/rte_mbuf.c > index c18b438..42b0cd1 100644 > --- a/lib/librte_mbuf/rte_mbuf.c > +++ b/lib/librte_mbuf/rte_mbuf.c > @@ -167,10 +167,18 @@ rte_pktmbuf_pool_create(const char *name, unsigned n, > mbp_priv.mbuf_data_room_size = data_room_size; > mbp_priv.mbuf_priv_size = priv_size; > > +#ifdef RTE_MEMPOOL_HANDLER_EXT > + return rte_mempool_create_ext(name, n, elt_size, > + cache_size, sizeof(struct rte_pktmbuf_pool_private), > + rte_pktmbuf_pool_init, &mbp_priv, rte_pktmbuf_init, NULL, > + socket_id, 0, > + RTE_MEMPOOL_HANDLER_NAME); > +#else > return rte_mempool_create(name, n, elt_size, > cache_size, sizeof(struct rte_pktmbuf_pool_private), > rte_pktmbuf_pool_init, &mbp_priv, rte_pktmbuf_init, NULL, > socket_id, 0); > +#endif > } > > /* do some sanity checks on a mbuf: panic if it fails */ > This kind of thing really has to be run-time configurable, not a library build-time option. - Panu -