From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hemant Agrawal Subject: Re: [PATCH 0/2] Multiple Pktmbuf mempool support Date: Fri, 22 Sep 2017 12:43:36 +0530 Message-ID: <08774e28-430e-4680-587a-909a5f44c5d6@nxp.com> References: <1499170968-23016-1-git-send-email-hemant.agrawal@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Cc: , To: , Return-path: Received: from NAM01-BN3-obe.outbound.protection.outlook.com (mail-bn3nam01on0060.outbound.protection.outlook.com [104.47.33.60]) by dpdk.org (Postfix) with ESMTP id EECE07CCD for ; Fri, 22 Sep 2017 09:13:41 +0200 (CEST) In-Reply-To: <1499170968-23016-1-git-send-email-hemant.agrawal@nxp.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Olivier, Any opinion on this patchset? Regards, Hemant On 7/4/2017 5:52 PM, Hemant Agrawal wrote: > This patch is in addition to the patch series[1] submitted by > Santosh to allow application to set mempool handle. > > The existing pktmbuf pool create api only support the internal use > of "CONFIG_RTE_MBUF_DEFAULT_MEMPOOL_OPS", which assumes that the HW > can only support one type of mempool for packet mbuf. > > There are multiple additional requirements. > > 1. The platform independent image detects the underlying bus, > based on the bus and resource detected, it will dynamically select > the default mempool. This need not to have the application knowlege. > e.g. DPAA2 and DPAA are two different NXP platforms, based on the > underlying platform the default ops for mbuf can be dpaa or dpaa2. > Application should work seemlessly whether it is running on dpaa or dpaa2. > > 2.Platform support more than one type of mempool for pktmbuf, > depend on the availability of resource, the driver can decide one > of the mempool for the current packet mbuf request. > > 3. In case of where application is providing the mempool, as proposed > in [1], the check preference logic will be bypassed and application > config will take priority. > > [1]Allow application set mempool handle > http://dpdk.org/ml/archives/dev/2017-June/067022.html > > Hemant Agrawal (2): > mempool: check the support for the given mempool > mbuf: add support for preferred mempool list > > config/common_base | 2 ++ > lib/librte_mbuf/rte_mbuf.c | 28 +++++++++++++++++++++++----- > lib/librte_mempool/rte_mempool.h | 24 ++++++++++++++++++++++++ > lib/librte_mempool/rte_mempool_ops.c | 32 ++++++++++++++++++++++++++++++++ > 4 files changed, 81 insertions(+), 5 deletions(-) >