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

[santosh]

Hi Ferruh,


On Wednesday 04 October 2017 12:31 AM, Ferruh Yigit wrote:
> 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.

Octeontx PMD driver has no dependency in general (on other octeontx patch series), 
except one external mempool api.. coming in from series [1], 
and that api will remain unchanged as its octeon specific. 
So having said that, You can start reviewing and sharing
feedback to us.

Note: octeontx external mempool pmd [1] has direct dependency on [2],[3].[4]
and those dependency series akced and planned to get merged into -rc1 release. 

Thanks.

[1]http://dpdk.org/ml/archives/dev/2017-August/073898.html
[2]http://dpdk.org/ml/archives/dev/2017-October/077050.html
[3]http://dpdk.org/ml/archives/dev/2017-September/076597.html
[4]http://dpdk.org/ml/archives/dev/2017-October/077053.html