[PATCH] doc: announce cryptodev changes to offload RSA in VirtIO

[Kusztal, ArkadiuszX]

Hi Gowrishankar,

> -----Original Message-----
> From: Gowrishankar Muthukrishnan <gmuthukrishn at marvell.com>
> Sent: Monday, July 22, 2024 4:56 PM
> To: dev at dpdk.org; Anoob Joseph <anoobj at marvell.com>; Richardson, Bruce
> <bruce.richardson at intel.com>; ciara.power at intel.com; jerinj at marvell.com;
> fanzhang.oss at gmail.com; Kusztal, ArkadiuszX <arkadiuszx.kusztal at intel.com>;
> Ji, Kai <kai.ji at intel.com>; jack.bond-preston at foss.arm.com; Marchand, David
> <david.marchand at redhat.com>; hemant.agrawal at nxp.com; De Lara Guarch,
> Pablo <pablo.de.lara.guarch at intel.com>; Trahe, Fiona
> <fiona.trahe at intel.com>; Doherty, Declan <declan.doherty at intel.com>;
> matan at nvidia.com; ruifeng.wang at arm.com; Gujjar, Abhinandan S
> <abhinandan.gujjar at intel.com>; maxime.coquelin at redhat.com;
> chenbox at nvidia.com; sunilprakashrao.uttarwar at amd.com;
> andrew.boyer at amd.com; ajit.khaparde at broadcom.com;
> raveendra.padasalagi at broadcom.com; vikas.gupta at broadcom.com;
> zhangfei.gao at linaro.org; g.singh at nxp.com; jianjay.zhou at huawei.com; Daly,
> Lee <lee.daly at intel.com>
> Cc: Gowrishankar Muthukrishnan <gmuthukrishn at marvell.com>
> Subject: [PATCH] doc: announce cryptodev changes to offload RSA in VirtIO
> 
> Announce cryptodev changes to offload RSA asymmetric operation in VirtIO
> PMD.
> 
> Signed-off-by: Gowrishankar Muthukrishnan <gmuthukrishn at marvell.com>
> --
> RFC:
>   https://patches.dpdk.org/project/dpdk/patch/20230928095300.1353-2-
> gmuthukrishn at marvell.com/
>   https://patches.dpdk.org/project/dpdk/patch/20230928095300.1353-3-
> gmuthukrishn at marvell.com/
> ---
>  doc/guides/rel_notes/deprecation.rst | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> index 6948641ff6..26fec84aba 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -147,3 +147,14 @@ Deprecation Notices
>    will be deprecated and subsequently removed in DPDK 24.11 release.
>    Before this, the new port library API (functions rte_swx_port_*)
>    will gradually transition from experimental to stable status.
> +
> +* cryptodev: The struct rte_crypto_rsa_padding will be moved from
> +  rte_crypto_rsa_op_param struct to rte_crypto_rsa_xform struct,
> +  breaking ABI. The new location is recommended to comply with
> +  virtio-crypto specification. Applications and drivers using
> +  this struct will be updated.
> +

The problem here, I see is that there is one private key but multiple combinations of padding.
Therefore, for every padding variation, we need to copy the same private key anew, duplicating it in memory.
The only reason for me to keep a session-like struct in asymmetric crypto was exactly this.

> +* cryptodev: The rte_crypto_rsa_xform struct member to hold private key
> +  in either exponent or quintuple format is changed from union to
> +struct
> +  data type. This change is to support ASN.1 syntax (RFC 3447 Appendix A.1.2).
> +  This change will not break existing applications.
This one I agree. RFC 8017 obsoletes RFC 3447.
> --
> 2.21.0