[dpdk-dev] [PATCH 00/26] DPDK PMD for OCTEONTX NW device

[Ferruh Yigit]

On 9/11/2017 7:15 PM, Jerin Jacob wrote:
> -----Original Message-----
>> Date: Tue, 5 Sep 2017 18:43:37 +0100
>> From: Ferruh Yigit <ferruh.yigit at intel.com>
>> To: Jerin Jacob <jerin.jacob at caviumnetworks.com>, dev at dpdk.org
>> CC: santosh.shukla at 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.

There may not be compile time dependency, but in case any change
requested in dependent patches requires update in this patchset.

> 
> 
>>
>>>
>>> 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.
>