diff mbox series

[09/25] crypto/cnxk: update max sec crypto caps

Message ID	1638859858-734-10-git-send-email-anoobj@marvell.com (mailing list archive)
State	Superseded, archived
Delegated to:	akhil goyal
Headers	From: Anoob Joseph <anoobj@marvell.com> To: Akhil Goyal <gakhil@marvell.com>, Jerin Jacob <jerinj@marvell.com> CC: Anoob Joseph <anoobj@marvell.com>, Archana Muniganti <marchana@marvell.com>, Tejasree Kondoj <ktejasree@marvell.com>, <dev@dpdk.org> Subject: [PATCH 09/25] crypto/cnxk: update max sec crypto caps Date: Tue, 7 Dec 2021 12:20:42 +0530 Message-ID: <1638859858-734-10-git-send-email-anoobj@marvell.com> In-Reply-To: <1638859858-734-1-git-send-email-anoobj@marvell.com> References: <1638859858-734-1-git-send-email-anoobj@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain Precedence: list Errors-To: dev-bounces@dpdk.org
Series	New features and improvements in cnxk crypto PMD \| [00/25] New features and improvements in cnxk crypto PMD [01/25] common/cnxk: define minor opcodes for MISC opcode [02/25] common/cnxk: add aes-xcbc key derive [03/25] common/cnxk: add bit fields for params [04/25] common/cnxk: fix reset of fields [05/25] common/cnxk: verify input args [06/25] crypto/cnxk: only enable queues that are allocated [07/25] crypto/cnxk: add lookaside IPsec AES-CBC-HMAC-SHA256 support [08/25] crypto/cnxk: clear session data before populating [09/25] crypto/cnxk: update max sec crypto caps [10/25] crypto/cnxk: write CPT CTX through microcode op [11/25] crypto/cnxk: support cnxk lookaside IPsec HMAC-SHA384/512 [12/25] crypto/cnxk: account for CPT CTX updates and flush delays [13/25] crypto/cnxk: use struct sizes for ctx writes [14/25] crypto/cnxk: add security session stats get [15/25] crypto/cnxk: add skip for unsupported cases [16/25] crypto/cnxk: add context reload for IV [17/25] crypto/cnxk: handle null chained ops [18/25] crypto/cnxk: fix inflight cnt calculation [19/25] crypto/cnxk: use atomics to access cpt res [20/25] crypto/cnxk: add more info on command timeout [21/25] crypto/cnxk: support lookaside IPsec AES-CTR [22/25] crypto/cnxk: fix extend tail calculation [23/25] crypto/cnxk: add aes xcbc and null cipher [24/25] crypto/cnxk: add copy and set DF [25/25] crypto/cnxk: add aes cmac

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Anoob Joseph Dec. 7, 2021, 6:50 a.m. UTC

  Update the macro to include newly added ciphers. Updated the functions
populating caps to throw error when max is exceeded.

Signed-off-by: Anoob Joseph <anoobj@marvell.com>
---
 drivers/crypto/cnxk/cnxk_cryptodev.h              | 2 +-
 drivers/crypto/cnxk/cnxk_cryptodev_capabilities.c | 8 ++++++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff mbox series

Patch

diff --git a/drivers/crypto/cnxk/cnxk_cryptodev.h b/drivers/crypto/cnxk/cnxk_cryptodev.h
index cfb9d29..2e0f467 100644
--- a/drivers/crypto/cnxk/cnxk_cryptodev.h
+++ b/drivers/crypto/cnxk/cnxk_cryptodev.h
@@ -11,7 +11,7 @@ 
 #include "roc_cpt.h"
 
 #define CNXK_CPT_MAX_CAPS	 34
-#define CNXK_SEC_CRYPTO_MAX_CAPS 4
+#define CNXK_SEC_CRYPTO_MAX_CAPS 6
 #define CNXK_SEC_MAX_CAPS	 5
 #define CNXK_AE_EC_ID_MAX	 8
 /**
diff --git a/drivers/crypto/cnxk/cnxk_cryptodev_capabilities.c b/drivers/crypto/cnxk/cnxk_cryptodev_capabilities.c
index 7d22626..8305341 100644
--- a/drivers/crypto/cnxk/cnxk_cryptodev_capabilities.c
+++ b/drivers/crypto/cnxk/cnxk_cryptodev_capabilities.c
@@ -943,8 +943,10 @@  static void
 sec_caps_add(struct rte_cryptodev_capabilities cnxk_caps[], int *cur_pos,
 	     const struct rte_cryptodev_capabilities *caps, int nb_caps)
 {
-	if (*cur_pos + nb_caps > CNXK_SEC_CRYPTO_MAX_CAPS)
+	if (*cur_pos + nb_caps > CNXK_SEC_CRYPTO_MAX_CAPS) {
+		rte_panic("Could not add sec crypto caps");
 		return;
+	}
 
 	memcpy(&cnxk_caps[*cur_pos], caps, nb_caps * sizeof(caps[0]));
 	*cur_pos += nb_caps;
@@ -957,8 +959,10 @@  cn10k_sec_crypto_caps_update(struct rte_cryptodev_capabilities cnxk_caps[],
 	const struct rte_cryptodev_capabilities *cap;
 	unsigned int i;
 
-	if ((CNXK_CPT_MAX_CAPS - *cur_pos) < 1)
+	if ((CNXK_SEC_CRYPTO_MAX_CAPS - *cur_pos) < 1) {
+		rte_panic("Could not add sec crypto caps");
 		return;
+	}
 
 	/* NULL auth */
 	for (i = 0; i < RTE_DIM(caps_null); i++) {