From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v2 00/10] Cavium Octeontx external mempool driver Date: Sat, 7 Oct 2017 05:26:06 +0100 Message-ID: <28848f95-ad92-6ae1-8003-de34e8e7e8aa@intel.com> References: <20170824132903.32057-1-santosh.shukla@caviumnetworks.com> <20170831063719.19273-1-santosh.shukla@caviumnetworks.com> <5a47127c-47a4-1966-cfec-937ea4e14eb0@caviumnetworks.com> <2007007.Y9KMiRGAUg@xps> <17f6ef1d-1375-961e-b4d0-175102bf5708@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: dev@dpdk.org, olivier.matz@6wind.com, jerin.jacob@caviumnetworks.com, john.mcnamara@intel.com, hemant.agrawal@nxp.com To: santosh , Thomas Monjalon Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 499E02C24 for ; Sat, 7 Oct 2017 06:26:07 +0200 (CEST) In-Reply-To: <17f6ef1d-1375-961e-b4d0-175102bf5708@caviumnetworks.com> Content-Language: en-US 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 10/7/2017 4:51 AM, santosh wrote: > > On Saturday 07 October 2017 02:25 AM, Thomas Monjalon wrote: >> 19/09/2017 10:29, santosh: >>> Ping? >> Pong >> >> It seems you forgot this series. > > This series review is pending and planned for -rc2 window. Hi Santosh, octeontx net pmd has dependency on this patch [1], pushing this to rc2 will push net pmd to rc2 as well. And I believe we should get new pmd in rc1 as much as possible. If you can send a new version of this patch at the beginning of the next week, both this and pmd one still can have chance to go in to rc1, can this be possible? [1] http://dpdk.org/ml/archives/dev/2017-October/077322.html > >> There is a compilation error and some doc changes requested. > > Yes, I have renamed few mempool api which was just merged in -rc1 then > will send out v3 series. > >> About commit titles, you might probably reword starting >> without "implement pool" which is redundant with "mempool/octeontx:". > > +1. > > Thanks. > >