[EXT] [PATCH v2 4/4] crypto: reorganize endianness comments, add crypto uint
Akhil Goyal
gakhil at marvell.com
Thu Feb 10 22:08:03 CET 2022
Hi Fan,
> Hi Akhil,
>
> I assume everything in asym crypto is under experimental tag at the moment
> right?
> The goal is to have them updated and fixed before DPDK 22.11 so the
> experimental tag can be removed.
>
Asymmetric crypto APIs are marked as experimental, but the structures are not
explicitly marked experimental.
rte_crypto_asym_op is part of union in rte_crypto_op which is definitely not experimental.
So a change in asym_op will result in ABI issues in rte_crypto_op.
David/Ray: Can you review the patch 1/4 of this series from ABI compatibility point of view.
http://patches.dpdk.org/project/dpdk/patch/20220207113555.8431-2-arkadiuszx.kusztal@intel.com/
IMO, as per current experimental tags, we cannot change parameters inside rte_crypto_asym_op
and subsequently in struct rte_crypto_dsa_op_param. What do you suggest?
However, I remember, some exception was added to ignore ABI issues related to asymmetric
crypto. Could you please check why that exception is not working in this case?
Regards,
Akhil
More information about the dev
mailing list