cryptodev: remove duplicated device name length
authorPablo de Lara <pablo.de.lara.guarch@intel.com>
Wed, 13 Dec 2017 15:54:08 +0000 (15:54 +0000)
committerPablo de Lara <pablo.de.lara.guarch@intel.com>
Sat, 20 Jan 2018 15:10:53 +0000 (16:10 +0100)
Two macros were defined in cryptodev, to serve the same
purpose: RTE_CRYPTODEV_NAME_LEN (in the config file) and
RTE_CRYPTODEV_NAME_MAX_LEN (in the rte_cryptodev.h file).
Since the second one is part of the external API,
the first one has been removed, avoiding duplications.

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Acked-by: Tomasz Duszynski <tdu@semihalf.com>
12 files changed:
app/test-crypto-perf/cperf_options.h
config/common_base
drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h
drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c
drivers/crypto/aesni_mb/rte_aesni_mb_pmd_private.h
drivers/crypto/armv8/rte_armv8_pmd_private.h
drivers/crypto/kasumi/rte_kasumi_pmd_private.h
drivers/crypto/mrvl/rte_mrvl_pmd_ops.c
drivers/crypto/null/null_crypto_pmd_private.h
drivers/crypto/openssl/rte_openssl_pmd_private.h
drivers/crypto/snow3g/rte_snow3g_pmd_private.h
drivers/crypto/zuc/rte_zuc_pmd_private.h

index b0c9f44..54a3ad5 100644 (file)
@@ -3,6 +3,7 @@
 #define _CPERF_OPTIONS_
 
 #include <rte_crypto.h>
+#include <rte_cryptodev.h>
 
 #define CPERF_PTEST_TYPE       ("ptest")
 #define CPERF_SILENT           ("silent")
@@ -104,7 +105,7 @@ struct cperf_options {
 
        uint16_t digest_sz;
 
-       char device_type[RTE_CRYPTODEV_NAME_LEN];
+       char device_type[RTE_CRYPTODEV_NAME_MAX_LEN];
        enum cperf_op_type op_type;
 
        char *test_file;
index 1443460..bbf535b 100644 (file)
@@ -468,7 +468,6 @@ CONFIG_RTE_LIBRTE_PMD_BBDEV_TURBO_SW=n
 CONFIG_RTE_LIBRTE_CRYPTODEV=y
 CONFIG_RTE_LIBRTE_CRYPTODEV_DEBUG=n
 CONFIG_RTE_CRYPTO_MAX_DEVS=64
-CONFIG_RTE_CRYPTODEV_NAME_LEN=64
 
 #
 # Compile PMD for ARMv8 Crypto device
index b121a1f..3d60583 100644 (file)
@@ -56,7 +56,7 @@ struct aesni_gcm_qp {
        /**< Session Mempool */
        uint16_t id;
        /**< Queue Pair Identifier */
-       char name[RTE_CRYPTODEV_NAME_LEN];
+       char name[RTE_CRYPTODEV_NAME_MAX_LEN];
        /**< Unique Queue Pair Name */
        uint8_t temp_digest[DIGEST_LENGTH_MAX];
        /**< Buffer used to store the digest generated
index 2de0f4e..9d685a0 100644 (file)
@@ -410,7 +410,7 @@ aesni_mb_pmd_qp_create_processed_ops_ring(struct aesni_mb_qp *qp,
                const char *str, unsigned int ring_size, int socket_id)
 {
        struct rte_ring *r;
-       char ring_name[RTE_CRYPTODEV_NAME_LEN];
+       char ring_name[RTE_CRYPTODEV_NAME_MAX_LEN];
 
        unsigned int n = snprintf(ring_name, sizeof(ring_name),
                                "%s_%s",
index 7d4250e..948e091 100644 (file)
@@ -130,7 +130,7 @@ struct aesni_mb_private {
 struct aesni_mb_qp {
        uint16_t id;
        /**< Queue Pair Identifier */
-       char name[RTE_CRYPTODEV_NAME_LEN];
+       char name[RTE_CRYPTODEV_NAME_MAX_LEN];
        /**< Unique Queue Pair Name */
        const struct aesni_mb_op_fns *op_fns;
        /**< Vector mode dependent pointer table of the multi-buffer APIs */
index 5debe33..b8966e9 100644 (file)
@@ -120,7 +120,7 @@ struct armv8_crypto_qp {
        /**< Session Mempool */
        struct rte_cryptodev_stats stats;
        /**< Queue pair statistics */
-       char name[RTE_CRYPTODEV_NAME_LEN];
+       char name[RTE_CRYPTODEV_NAME_MAX_LEN];
        /**< Unique Queue Pair Name */
        uint8_t temp_digest[DIGEST_LENGTH_MAX];
        /**< Buffer used to store the digest generated
index 97577dc..a397bee 100644 (file)
@@ -44,7 +44,7 @@ struct kasumi_private {
 struct kasumi_qp {
        uint16_t id;
        /**< Queue Pair Identifier */
-       char name[RTE_CRYPTODEV_NAME_LEN];
+       char name[RTE_CRYPTODEV_NAME_MAX_LEN];
        /**< Unique Queue Pair Name */
        struct rte_ring *processed_ops;
        /**< Ring for placing processed ops */
index 434cf85..a1de33a 100644 (file)
@@ -558,7 +558,7 @@ mrvl_crypto_pmd_qp_setup(struct rte_cryptodev *dev, uint16_t qp_id,
                int socket_id, struct rte_mempool *session_pool)
 {
        struct mrvl_crypto_qp *qp = NULL;
-       char match[RTE_CRYPTODEV_NAME_LEN];
+       char match[RTE_CRYPTODEV_NAME_MAX_LEN];
        unsigned int n;
 
        /* Allocate the queue pair data structure. */
index 9b37aab..0fd1336 100644 (file)
@@ -39,7 +39,7 @@ struct null_crypto_private {
 struct null_crypto_qp {
        uint16_t id;
        /**< Queue Pair Identifier */
-       char name[RTE_CRYPTODEV_NAME_LEN];
+       char name[RTE_CRYPTODEV_NAME_MAX_LEN];
        /**< Unique Queue Pair Name */
        struct rte_ring *processed_pkts;
        /**< Ring for placing process packets */
index 0f2d61e..bc8dc7c 100644 (file)
@@ -70,7 +70,7 @@ struct openssl_private {
 struct openssl_qp {
        uint16_t id;
        /**< Queue Pair Identifier */
-       char name[RTE_CRYPTODEV_NAME_LEN];
+       char name[RTE_CRYPTODEV_NAME_MAX_LEN];
        /**< Unique Queue Pair Name */
        struct rte_ring *processed_ops;
        /**< Ring for placing process packets */
index c4f018f..eea900e 100644 (file)
@@ -44,7 +44,7 @@ struct snow3g_private {
 struct snow3g_qp {
        uint16_t id;
        /**< Queue Pair Identifier */
-       char name[RTE_CRYPTODEV_NAME_LEN];
+       char name[RTE_CRYPTODEV_NAME_MAX_LEN];
        /**< Unique Queue Pair Name */
        struct rte_ring *processed_ops;
        /**< Ring for placing processed ops */
index 5d1253b..b83c4a0 100644 (file)
@@ -45,7 +45,7 @@ struct zuc_private {
 struct zuc_qp {
        uint16_t id;
        /**< Queue Pair Identifier */
-       char name[RTE_CRYPTODEV_NAME_LEN];
+       char name[RTE_CRYPTODEV_NAME_MAX_LEN];
        /**< Unique Queue Pair Name */
        struct rte_ring *processed_ops;
        /**< Ring for placing processed ops */