From: Pablo de Lara Date: Sun, 2 Jul 2017 05:41:03 +0000 (+0100) Subject: cryptodev: replace enums with 1-byte variables X-Git-Tag: spdx-start~2697 X-Git-Url: http://git.droids-corp.org/?a=commitdiff_plain;h=4943cb4ad21d51e31ad45cec67d6853cab867e62;p=dpdk.git cryptodev: replace enums with 1-byte variables Instead of storing some crypto operation flags, such as operation status, as enumerations, store them as uint8_t, for memory efficiency. Also, reserve extra 5 bytes in the crypto operation, for future additions. Signed-off-by: Pablo de Lara Acked-by: Declan Doherty Acked-by: Akhil Goyal Acked-by: Fiona Trahe --- diff --git a/doc/guides/rel_notes/release_17_08.rst b/doc/guides/rel_notes/release_17_08.rst index 4e6a8bd5da..bfcc1103cb 100644 --- a/doc/guides/rel_notes/release_17_08.rst +++ b/doc/guides/rel_notes/release_17_08.rst @@ -96,6 +96,8 @@ New Features The crypto operation (``rte_crypto_op``) has been reorganized as follows: * Added field ``rte_crypto_op_sess_type``. + * Enumerations ``rte_crypto_op_status`` and ``rte_crypto_op_type`` + have been modified to be uint8_t values. Resolved Issues diff --git a/lib/librte_cryptodev/rte_crypto.h b/lib/librte_cryptodev/rte_crypto.h index b816b45580..dd17020909 100644 --- a/lib/librte_cryptodev/rte_crypto.h +++ b/lib/librte_cryptodev/rte_crypto.h @@ -102,19 +102,20 @@ enum rte_crypto_op_sess_type { * rte_cryptodev_enqueue_burst() / rte_cryptodev_dequeue_burst() . */ struct rte_crypto_op { - enum rte_crypto_op_type type; + uint8_t type; /**< operation type */ - - enum rte_crypto_op_status status; + uint8_t status; /**< * operation status - this is reset to * RTE_CRYPTO_OP_STATUS_NOT_PROCESSED on allocation from mempool and * will be set to RTE_CRYPTO_OP_STATUS_SUCCESS after crypto operation * is successfully processed by a crypto PMD */ - enum rte_crypto_op_sess_type sess_type; + uint8_t sess_type; /**< operation session type */ + uint8_t reserved[5]; + /**< Reserved bytes to fill 64 bits for future additions */ struct rte_mempool *mempool; /**< crypto operation mempool which operation is allocated from */