From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hemant Agrawal Subject: Re: [PATCH v1] NXP DPAA2 External Mempool Driver Date: Mon, 20 Mar 2017 15:38:13 +0530 Message-ID: References: <1489754838-1455-1-git-send-email-hemant.agrawal@nxp.com> <3431506.qvV2pjYT76@xps13> <20170317182233.27374d4e@platinum> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Cc: Thomas Monjalon , "dev@dpdk.org" , Shreyansh Jain , "ferruh.yigit@intel.com" To: Olivier Matz Return-path: Received: from NAM01-BN3-obe.outbound.protection.outlook.com (mail-bn3nam01on0083.outbound.protection.outlook.com [104.47.33.83]) by dpdk.org (Postfix) with ESMTP id 865191075 for ; Mon, 20 Mar 2017 11:08:24 +0100 (CET) In-Reply-To: <20170317182233.27374d4e@platinum> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 3/17/2017 10:52 PM, Olivier Matz wrote: > On Fri, 17 Mar 2017 17:12:59 +0000, Hemant Agrawal wrote: >>> -----Original Message----- >>> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] >>> Sent: Friday, March 17, 2017 7:13 PM >>> To: Hemant Agrawal >>> Cc: dev@dpdk.org; olivier.matz@6wind.com; Shreyansh Jain >>> ; ferruh.yigit@intel.com >>> Subject: Re: [PATCH v1] NXP DPAA2 External Mempool Driver >>> >>> 2017-03-17 18:17, Hemant Agrawal: >>>> (This patches has been split from DPAA2 PMD v8 series [1] as per >>>> comments received on ML [2].) >>>> >>>> This patch implements the HW mempool offload driver for packets >>>> buffers using the FSLMC provided DPBP object. >>>> >>>> The new pool is being configured using compile time option and pool >>>> name as "dpaa2". >>>> >>>> Dependency: >>>> This patchset is based on "NXP DPAA2 FSLMC Bus Driver" patchset [3]. >>>> >>>> References: >>>> [1] http://dpdk.org/ml/archives/dev/2017-March/059000.html >>>> [2] http://dpdk.org/ml/archives/dev/2017-March/059789.html >>>> [3] http://dpdk.org/ml/archives/dev/2017-March/060453.html >>> >>> I'm sorry, I think there is misunderstanding. >>> We were saying that you should move the existing mempool handlers first. >>> Have you submitted a patch for the ring and stack mempool handlers? >> [Hemant] >> >> I will do it. But that work is independent of NXP patches? Isn't it? > > I'd say the existing handlers have to be moved first, to avoid to > have a state where there are handlers at 2 different locations. > > So to me, it's just a chronological dependency. > We have just posted the ring and stack handler. http://dpdk.org/ml/archives/dev/2017-March/060647.html Also, if you are ok with NXP DPAA2 mempool driver, will you please ack it? > Regards, > Olivier >