-----Original Message----- > Date: Tue, 5 Sep 2017 18:43:37 +0100 > From: Ferruh Yigit <ferruh.yi...@intel.com> > To: Jerin Jacob <jerin.ja...@caviumnetworks.com>, dev@dpdk.org > CC: santosh.shu...@caviumnetworks.com > Subject: Re: [dpdk-dev] [PATCH 00/26] DPDK PMD for OCTEONTX NW device > User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 > Thunderbird/52.3.0 > > On 8/31/2017 3:54 PM, Jerin Jacob wrote: > > This patch set provides the initial version of DPDK PMD for the > > integrated NIC device found in Cavium OCTEONTX SoC(CN83xx) family. > > > > Introduction to OCTEONTX ARM processors can be found > > here: http://www.cavium.com/OCTEON-TX_ARM_Processors.html > > > > OCTEONTX SoC family has two types of NIC controllers, > > 1) nicvf HW, Which is more like traditional NIC(no NPU HW accelerators) > > http://dpdk.org/doc/guides/nics/thunderx.html > > > > 2) NIC controller, based on NPU like HW accelerated coprocessors, like > > HW assisted mempool manager, HW assisted packet scheduler, etc > > > > This driver adds support for the second type device and which will be > > used as HW implementation driver ethdev-eventdev Rx adapter [1] > > > > Implemented features and PMD documentation added > > in doc/guides/nics/features/octeontx.ini and doc/guides/nics/octeontx.rst > > respectively in this patch set. > > > > This driver has the dependency on: > > - IOVA infrastructure [2]. > > - Dynamically configure mempool handle (ie.. --mbuf-pool-ops eal arg) [3]. > > - Infrastructure to support octeontx HW mempool manager [4]. > > - Cavium Octeontx external mempool driver [5] > > Git source repo for all above dependency patches located at [6]. > > Hi Jerin,
Hi Ferruh, > > At first glance PMD looks good to me, but currently there are many > dependencies and I am not able to apply the set. I will wait a little > more for dependent patches to settle down and will re-visit this again. OK. The dependencies are more of runtime in nature not all of them are compile time. > > > > > The series has three check patch issues: > > 1) 1 Error on Complex Macro (which I believe it cannot be fixed) # Patch 2 > > 2) 1 Error on spacing while using gcc inline assembly(it is false positive) > > # patch 2 > > 3) Added Co-authored-by: signature to reflect the patch authorship status > > instead > > of "Signed-off-by". It can be changed or the contributing guide can be > > updated based on the community feedback in v2. > > > > [1] http://dpdk.org/dev/patchwork/patch/24176/ > > Are these APIs used in this patchset, I looked for them but not able to > find ... No.
