Hi Pablo,
Thanks for the review. We had made some architectural modifications to
the PMD while addressing your comments. This is taking longer than
expected and so we might as well defer the v2 to the next release cycle.
Is that fine?
Do you suggest submitting the patches as and when it is ready or should
I wait till next release cycle?
Please, take into account the following patchset which will be applied tomorrow
(just waiting for a couple of acks):
http://patches.dpdk.org/user/todo/dpdk/?series=416
This series is integrated.
Thanks,
Anoob
On 10-07-2018 04:25, De Lara Guarch, Pablo wrote:
External Email
-----Original Message-----
From: De Lara Guarch, Pablo
Sent: Monday, June 18, 2018 3:30 PM
To: Anoob Joseph <[email protected]>; Akhil Goyal
<[email protected]>
Cc: Thomas Monjalon <[email protected]>; Ankur Dwivedi
<[email protected]>; Jerin Jacob
<[email protected]>; Murthy NSSR
<[email protected]>; Narayana Prasad
<[email protected]>; Nithin Dabilpuram
<[email protected]>; Ragothaman Jayaraman
<[email protected]>; Srisivasubramanian Srinivasan
<[email protected]>; [email protected]
Subject: RE: [dpdk-dev] [PATCH 00/16] Adding Cavium's crypto device(CPT)
driver
Hi Anoob,
-----Original Message-----
From: dev [mailto:[email protected]] On Behalf Of Anoob Joseph
Sent: Monday, June 18, 2018 2:02 PM
To: Akhil Goyal <[email protected]>; De Lara Guarch, Pablo
<[email protected]>
Cc: Thomas Monjalon <[email protected]>; Ankur Dwivedi
<[email protected]>; Jerin Jacob
<[email protected]>; Murthy NSSR
<[email protected]>; Narayana Prasad
<[email protected]>; Nithin Dabilpuram
<[email protected]>; Ragothaman Jayaraman
<[email protected]>; Srisivasubramanian Srinivasan
<[email protected]>; [email protected]
Subject: Re: [dpdk-dev] [PATCH 00/16] Adding Cavium's crypto
device(CPT) driver
Hi Akhil, Pablo,
Are there any top level comments on this?
Sorry for the delay. I didn't have the time to review this yet.
Will do that asap.
Hi Anoob,
I had some comments on this patchset.
Could you address these comments and send a v2?
Please, take into account the following patchset which will be applied tomorrow
(just waiting for a couple of acks):
http://patches.dpdk.org/user/todo/dpdk/?series=416
Thanks,
Pablo
