Hi Pablo/Declan,

On 8/4/2017 8:55 PM, De Lara Guarch, Pablo wrote:


-----Original Message-----
From: Akhil Goyal [mailto:[email protected]]
Sent: Thursday, August 3, 2017 4:32 PM
To: [email protected]; Doherty, Declan <[email protected]>;
[email protected]; Nicolau, Radu <[email protected]>;
[email protected]; [email protected];
[email protected]; De Lara Guarch, Pablo
<[email protected]>
Cc: Akhil Goyal <[email protected]>
Subject: [PATCH] doc: announce ABI change for cryptodev and ethdev

Support for security operations is planned to be added in ethdev and
cryptodev for the 17.11 release.

For this following changes are required.
- rte_cryptodev and rte_eth_dev structures need to be added new
parameter rte_security_ops which extend support for security ops to the
corresponding driver.
- rte_cryptodev_info and rte_ethd_dev_info need to be added with
rte_security_capabilities to identify the capabilities of the corresponding
driver.

Signed-off-by: Akhil Goyal <[email protected]>

Not sure if this needed to be split into two patches, as this affects two 
libraries.
At least, from cryptodev side:

Acked-by: Pablo de Lara <[email protected]>


We would be needing one more ABI change, Can I send it now. I discovered it after I sent this patch.

In the struct rte_crypto_sym_op, we would need to add a pointer to a security session in the union of session and xform.

Also, Do I need to split this patch into two for crypto and eth?

Regards,
Akhil

Reply via email to