>-----Original Message-----
>From: Trahe, Fiona [mailto:fiona.tr...@intel.com]
>Sent: 03 July 2018 20:21
>To: Verma, Shally <shally.ve...@cavium.com>; De Lara Guarch, Pablo
><pablo.de.lara.gua...@intel.com>
>Cc: akhil.go...@nxp.com; dev@dpdk.org; Athreya, Narayana Prasad
><narayanaprasad.athr...@cavium.com>; Sahu, Sunila
><sunila.s...@cavium.com>; Gupta, Ashish <ashish.gu...@cavium.com>; Trahe,
>Fiona <fiona.tr...@intel.com>
>Subject: RE: [PATCH v3 5/6] crypto/openssl: add asym crypto support
>
>External Email
>
>Hi Shally,
>
>> -----Original Message-----
>> From: Shally Verma [mailto:shally.ve...@caviumnetworks.com]
>> Sent: Wednesday, May 16, 2018 7:05 AM
>> To: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>
>> Cc: Trahe, Fiona <fiona.tr...@intel.com>; akhil.go...@nxp.com; dev@dpdk.org;
>> pathr...@caviumnetworks.com; Sunila Sahu <sunila.s...@caviumnetworks.com>;
>> Ashish Gupta
>> <ashish.gu...@caviumnetworks.com>
>> Subject: [PATCH v3 5/6] crypto/openssl: add asym crypto support
>>
>> Add asymmetric crypto operation support in openssl PMD.
>> Current list of supported asym xforms:
>> * RSA
>> * DSA
>> * Deffie-hellman
>> * Modular Operations
>>
>> changes from v2:
>> - Update the pmd capability as per new capability structure
>>
>> changes from v1:
>> - resolve new line error in dod/guides/cryptodevs/openssl.rst
>>
>> Signed-off-by: Shally Verma <shally.ve...@caviumnetworks.com>
>> Signed-off-by: Sunila Sahu <sunila.s...@caviumnetworks.com>
>> Signed-off-by: Ashish Gupta <ashish.gu...@caviumnetworks.com>
>
>
>//snip//
>> +/* process rsa operations */
>> +static int process_openssl_rsa_op(struct rte_crypto_op *cop,
>> + struct openssl_asym_session *sess)
>> +{
>> + int ret = 0;
>> + struct rte_crypto_asym_op *op = cop->asym;
>> + RSA *rsa = sess->u.r.rsa;
>> + uint32_t pad = (op->rsa.pad);
>> +
>> + switch (pad) {
>> + case RTE_CRYPTO_RSA_PKCS1_V1_5_BT1:
>> + case RTE_CRYPTO_RSA_PKCS1_V1_5_BT2:
>> + pad = RSA_PKCS1_PADDING;
>> + break;
>> + case RTE_CRYPTO_RSA_PADDING_PSS:
>> + pad = RSA_PKCS1_PSS_PADDING;
>> + /* fall through */
>> + case RTE_CRYPTO_RSA_PADDING_OAEP:
>> + pad = RSA_PKCS1_OAEP_PADDING;
>> + /* fall through */
>> + default:
>> + pad = RSA_NO_PADDING;
>> + break;
>[Fiona] Maybe I'm missing something, but won't this fall through to
>RSA_NO_PADDING for OAEP and PSS cases?
>
>//snip//
Yes. It will because we were not claiming OAEP/PSS support in 1st cut. Also,
So, made them switched to no-padding.
However, openssl PMD asym changes , as such, is on hold until it is moved that
to 1.1.0. So, we will submit PMD and testapp as a separate patches from
Lib patch series.
