[dpdk-dev] [PATCH v2 2/3] lib/librte_cryptodev: functions for new performance test application

De Lara Guarch, Pablo pablo.de.lara.guarch at intel.com
Fri Jan 6 13:21:52 CET 2017



> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Slawomir
> Mrozowicz
> Sent: Thursday, January 05, 2017 4:50 PM
> To: dev at dpdk.org
> Cc: Mrozowicz, SlawomirX; Doherty, Declan; Kerlin, Marcin
> Subject: [dpdk-dev] [PATCH v2 2/3] lib/librte_cryptodev: functions for new
> performance test application
> 
> This patch adds helper functions for new performance application.
> Application can be used to measute throughput and latency of
> cryptography operation performed by crypto device.
> 
> Signed-off-by: Declan Doherty <declan.doherty at intel.com>
> Signed-off-by: Slawomir Mrozowicz <slawomirx.mrozowicz at intel.com>
> Signed-off-by: Marcin Kerlin <marcinx.kerlin at intel.com>

Hi Slawomir,

You should change the title of this patch to something like: "cryptodev: add functions for...".
Also, maybe it would be better to say "add functions to retrieve device info",
since you are not saying much with the current title.

I have seen also that this is based on the SGL code, so this patch didn't apply cleanly.
Next time, specify it in the email the dependencies of the patch.

Thanks,
Pablo





More information about the dev mailing list