* [dpdk-dev] [PATCH v3] cryptodev: rework mod exp and mod inv comments
@ 2019-02-06 10:34 Arek Kusztal
2019-02-06 10:43 ` Trahe, Fiona
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Arek Kusztal @ 2019-02-06 10:34 UTC (permalink / raw)
To: dev
Cc: akhil.goyal, fiona.trahe, shally.verma, sunila.sahu,
ashish.gupta, umesh.kartha, Arek Kusztal
This patch changes modular exponentiation and modular multiplicative
inverse API comments to make it more precise.
Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>
---
v2:
- grammar fixes
v3:
- remove information about positive integer
lib/librte_cryptodev/rte_crypto_asym.h | 43 ++++++++++++++++++++++------------
1 file changed, 28 insertions(+), 15 deletions(-)
diff --git a/lib/librte_cryptodev/rte_crypto_asym.h b/lib/librte_cryptodev/rte_crypto_asym.h
index 5e185b2..0a50cd5 100644
--- a/lib/librte_cryptodev/rte_crypto_asym.h
+++ b/lib/librte_cryptodev/rte_crypto_asym.h
@@ -72,8 +72,8 @@ enum rte_crypto_asym_xform_type {
* Refer to rte_crypto_asym_op_type
*/
RTE_CRYPTO_ASYM_XFORM_MODINV,
- /**< Modular Inverse
- * Perform Modulus inverse b^(-1) mod n
+ /**< Modular Multiplicative Inverse
+ * Perform Modular Multiplicative Inverse b^(-1) mod n
*/
RTE_CRYPTO_ASYM_XFORM_MODEX,
/**< Modular Exponentiation
@@ -233,29 +233,38 @@ struct rte_crypto_rsa_xform {
struct rte_crypto_modex_xform {
rte_crypto_param modulus;
/**< modulus
- * Prime modulus of the modexp transform operation in octet-string
- * network byte order format.
+ * Pointer to the modulus data for modexp transform operation
+ * in octet-string network byte order format
+ *
+ * In case this number is equal to zero the driver shall set
+ * the crypto op status field to RTE_CRYPTO_OP_STATUS_ERROR
*/
rte_crypto_param exponent;
/**< exponent
- * Private exponent of the modexp transform operation in
- * octet-string network byte order format.
+ * Exponent of the modexp transform operation in
+ * octet-string network byte order format
*/
};
/**
- * Asymmetric modular inverse transform operation
+ * Asymmetric modular multiplicative inverse transform operation
*
- * Structure describing modulus inverse xform params
+ * Structure describing modular multiplicative inverse transform
*
*/
struct rte_crypto_modinv_xform {
rte_crypto_param modulus;
/**<
- * Pointer to the prime modulus data for modular
- * inverse operation in octet-string network byte
- * order format.
+ * Pointer to the modulus data for modular multiplicative inverse
+ * operation in octet-string network byte order format
+ *
+ * In case this number is equal to zero the driver shall set
+ * the crypto op status field to RTE_CRYPTO_OP_STATUS_ERROR
+ *
+ * This number shall be relatively prime to base
+ * in corresponding Modular Multiplicative Inverse
+ * rte_crypto_mod_op_param
*/
};
@@ -317,14 +326,18 @@ struct rte_crypto_dsa_xform {
/**
* Operations params for modular operations:
- * exponentiation and invert
+ * exponentiation and multiplicative inverse
*
*/
struct rte_crypto_mod_op_param {
rte_crypto_param base;
/**<
- * Pointer to base of modular exponentiation/inversion data in
- * Octet-string network byte order format.
+ * Pointer to base of modular exponentiation/multiplicative
+ * inverse data in octet-string network byte order format
+ *
+ * In case Multiplicative Inverse is used this number shall
+ * be relatively prime to modulus in corresponding Modular
+ * Multiplicative Inverse rte_crypto_modinv_xform
*/
};
@@ -348,7 +361,7 @@ struct rte_crypto_asym_xform {
/**< Modular Exponentiation xform parameters */
struct rte_crypto_modinv_xform modinv;
- /**< Modulus Inverse xform parameters */
+ /**< Modular Multiplicative Inverse xform parameters */
struct rte_crypto_dh_xform dh;
/**< DH xform parameters */
--
2.1.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dpdk-dev] [PATCH v3] cryptodev: rework mod exp and mod inv comments
2019-02-06 10:34 [dpdk-dev] [PATCH v3] cryptodev: rework mod exp and mod inv comments Arek Kusztal
@ 2019-02-06 10:43 ` Trahe, Fiona
2019-02-06 10:50 ` Shally Verma
2019-03-06 16:34 ` Akhil Goyal
2 siblings, 0 replies; 4+ messages in thread
From: Trahe, Fiona @ 2019-02-06 10:43 UTC (permalink / raw)
To: Kusztal, ArkadiuszX, dev
Cc: akhil.goyal, shally.verma, sunila.sahu, ashish.gupta, umesh.kartha
> -----Original Message-----
> From: Kusztal, ArkadiuszX
> Sent: Wednesday, February 6, 2019 10:34 AM
> To: dev@dpdk.org
> Cc: akhil.goyal@nxp.com; Trahe, Fiona <fiona.trahe@intel.com>; shally.verma@caviumnetworks.com;
> sunila.sahu@caviumnetworks.com; ashish.gupta@caviumnetworks.com;
> umesh.kartha@caviumnetworks.com; Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com>
> Subject: [PATCH v3] cryptodev: rework mod exp and mod inv comments
>
> This patch changes modular exponentiation and modular multiplicative
> inverse API comments to make it more precise.
>
> Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>
Acked-by: Fiona Trahe <fiona.trahe@intel.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dpdk-dev] [PATCH v3] cryptodev: rework mod exp and mod inv comments
2019-02-06 10:34 [dpdk-dev] [PATCH v3] cryptodev: rework mod exp and mod inv comments Arek Kusztal
2019-02-06 10:43 ` Trahe, Fiona
@ 2019-02-06 10:50 ` Shally Verma
2019-03-06 16:34 ` Akhil Goyal
2 siblings, 0 replies; 4+ messages in thread
From: Shally Verma @ 2019-02-06 10:50 UTC (permalink / raw)
To: Arek Kusztal, dev
Cc: akhil.goyal, fiona.trahe, shally.verma, sunila.sahu,
ashish.gupta, umesh.kartha
>-----Original Message-----
>From: Arek Kusztal <arkadiuszx.kusztal@intel.com>
>Sent: 06 February 2019 16:04
>To: dev@dpdk.org
>Cc: akhil.goyal@nxp.com; fiona.trahe@intel.com; shally.verma@caviumnetworks.com; sunila.sahu@caviumnetworks.com;
>ashish.gupta@caviumnetworks.com; umesh.kartha@caviumnetworks.com; Arek Kusztal <arkadiuszx.kusztal@intel.com>
>Subject: [PATCH v3] cryptodev: rework mod exp and mod inv comments
>
>External Email
>
>This patch changes modular exponentiation and modular multiplicative
>inverse API comments to make it more precise.
>
>Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>
>---
>v2:
>- grammar fixes
>
>v3:
>- remove information about positive integer
>
> lib/librte_cryptodev/rte_crypto_asym.h | 43 ++++++++++++++++++++++------------
> 1 file changed, 28 insertions(+), 15 deletions(-)
>
Acked-by: Shally Verma <shally.verma@marvell.com>
Thanks
Shally
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [dpdk-dev] [PATCH v3] cryptodev: rework mod exp and mod inv comments
2019-02-06 10:34 [dpdk-dev] [PATCH v3] cryptodev: rework mod exp and mod inv comments Arek Kusztal
2019-02-06 10:43 ` Trahe, Fiona
2019-02-06 10:50 ` Shally Verma
@ 2019-03-06 16:34 ` Akhil Goyal
2 siblings, 0 replies; 4+ messages in thread
From: Akhil Goyal @ 2019-03-06 16:34 UTC (permalink / raw)
To: Arek Kusztal, dev
Cc: fiona.trahe, shally.verma, sunila.sahu, ashish.gupta, umesh.kartha
On 2/6/2019 4:04 PM, Arek Kusztal wrote:
> This patch changes modular exponentiation and modular multiplicative
> inverse API comments to make it more precise.
>
> Signed-off-by: Arek Kusztal <arkadiuszx.kusztal@intel.com>
> ---
> v2:
> - grammar fixes
>
> v3:
> - remove information about positive integer
>
> lib/librte_cryptodev/rte_crypto_asym.h | 43 ++++++++++++++++++++++------------
> 1 file changed, 28 insertions(+), 15 deletions(-)
>
Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
Applied to dpdk-next-crypto
Thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-03-06 16:34 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-06 10:34 [dpdk-dev] [PATCH v3] cryptodev: rework mod exp and mod inv comments Arek Kusztal
2019-02-06 10:43 ` Trahe, Fiona
2019-02-06 10:50 ` Shally Verma
2019-03-06 16:34 ` Akhil Goyal
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).