From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCHv8 19/46] pool/dpaa2: add DPAA2 hardware offloaded mempool Date: Wed, 08 Mar 2017 16:39:28 +0100 Message-ID: <3284470.c3gGJi30Hp@xps13> References: <1487205586-6785-1-git-send-email-hemant.agrawal@nxp.com> <20170308100553.14ca39a7@glumotte.dev.6wind.com> <521bf2d1-0393-c4ac-422e-96134880502a@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: Olivier MATZ , dev@dpdk.org, bruce.richardson@intel.com, shreyansh.jain@nxp.com, john.mcnamara@intel.com, ferruh.yigit@intel.com, jerin.jacob@caviumnetworks.com To: Hemant Agrawal Return-path: Received: from mail-wm0-f49.google.com (mail-wm0-f49.google.com [74.125.82.49]) by dpdk.org (Postfix) with ESMTP id CB90C914 for ; Wed, 8 Mar 2017 16:39:29 +0100 (CET) Received: by mail-wm0-f49.google.com with SMTP id n11so34060746wma.0 for ; Wed, 08 Mar 2017 07:39:29 -0800 (PST) In-Reply-To: <521bf2d1-0393-c4ac-422e-96134880502a@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" 2017-03-08 18:22, Hemant Agrawal: > > On Fri, 3 Mar 2017 18:16:36 +0530, Hemant Agrawal > > wrote: > > I think the current mempool handlers should be moved first in a > > separate patch. Yes it should have been done earlier. > Are you seeing any benefit by making it a separate patch series? A separate patchset for moving mempool handlers will be easy to review and accept. If integrated in this series, it is kind of hidden and prevent the visibility and review it deserves. By the way the mempool move should be directly committed in the main repository, while this series targets next-net. > it will be difficult and tricky for us. The dpaa2_pool has a dependency > on mc bus patches. dpaa2_pmd has dependency on dpaa2_pool and mc buses. You will just have to rebase this series on top of the new one.