[dpdk-dev] [PATCH v2 2/6] lib/cryptodev: add asym op support in cryptodev

Trahe, Fiona fiona.trahe at intel.com
Tue May 8 16:43:57 CEST 2018


Hi Shally,


> -----Original Message-----
> From: Shally Verma [mailto:shally.verma at caviumnetworks.com]
> Sent: Thursday, April 5, 2018 12:25 PM
> To: De Lara Guarch, Pablo <pablo.de.lara.guarch at intel.com>
> Cc: Doherty, Declan <declan.doherty at intel.com>; Trahe, Fiona <fiona.trahe at intel.com>;
> pathreya at caviumnetworks.com; ssahu at caviumnetworks.com; agupta at caviumnetworks.com;
> dev at dpdk.org; Sunila Sahu <sunila.sahu at caviumnetworks.com>; Ashish Gupta
> <ashish.gupta at caviumnetworks.com>
> Subject: [PATCH v2 2/6] lib/cryptodev: add asym op support in cryptodev
> 
>  struct rte_cryptodev_sym_session *
>  rte_cryptodev_sym_session_create(struct rte_mempool *mp)
>  {
>  	struct rte_cryptodev_sym_session *sess;
> 
>  	/* Allocate a session structure from the session pool */
> -	if (rte_mempool_get(mp, (void **)&sess)) {
> +	if (rte_mempool_get(mp, (void *)&sess)) {
[Fiona] Are you introducing a bug here?
The fn expects a void **
Also this will need rebasing - the latest code has an extra byte.



More information about the dev mailing list