The microcode expects zuc-256 key to be in reverse of what is
provided by dpdk test app. This patch swaps the zuc-256 key.

Signed-off-by: Ankur Dwivedi <adwiv...@marvell.com>
Reviewed-by: Jerin Jacob Kollanukkaran <jer...@marvell.com>
---
 drivers/common/cnxk/roc_se.c |  7 +++++--
 drivers/common/cnxk/roc_se.h | 22 ++++++++++++++++++++++
 2 files changed, 27 insertions(+), 2 deletions(-)

diff --git a/drivers/common/cnxk/roc_se.c b/drivers/common/cnxk/roc_se.c
index ffe537af30..3f0821e400 100644
--- a/drivers/common/cnxk/roc_se.c
+++ b/drivers/common/cnxk/roc_se.c
@@ -283,6 +283,8 @@ roc_se_auth_key_set(struct roc_se_ctx *se_ctx, 
roc_se_auth_type type,
                                return ret;
                        se_ctx->pdcp_alg_type = ROC_SE_PDCP_ALG_TYPE_ZUC;
                        memcpy(ci_key, key, key_len);
+                       if (key_len == 32)
+                               roc_se_zuc_bytes_swap(ci_key, key_len);
                        cpt_pdcp_update_zuc_const(zuc_const, key_len, mac_len);
                        se_ctx->fc_type = ROC_SE_PDCP;
                        se_ctx->zsk_flags = 0x1;
@@ -459,9 +461,10 @@ roc_se_ciph_key_set(struct roc_se_ctx *se_ctx, 
roc_se_cipher_type type,
                zs_ctx->zuc.otk_ctx.w0.s.alg_type = ROC_SE_PDCP_ALG_TYPE_ZUC;
                se_ctx->pdcp_alg_type = ROC_SE_PDCP_ALG_TYPE_ZUC;
                memcpy(ci_key, key, key_len);
-               if (key_len == 32)
+               if (key_len == 32) {
+                       roc_se_zuc_bytes_swap(ci_key, key_len);
                        memcpy(zuc_const, zuc_key256, 16);
-               else
+               } else
                        memcpy(zuc_const, zuc_key128, 32);
 
                se_ctx->zsk_flags = 0;
diff --git a/drivers/common/cnxk/roc_se.h b/drivers/common/cnxk/roc_se.h
index 145a1825bd..fde69ceb9e 100644
--- a/drivers/common/cnxk/roc_se.h
+++ b/drivers/common/cnxk/roc_se.h
@@ -301,6 +301,27 @@ struct roc_se_ctx {
        uint8_t *auth_key;
 };
 
+static inline void
+roc_se_zuc_bytes_swap(uint8_t *arr, int len)
+{
+       int start, end;
+       uint8_t tmp;
+
+       if (len <= 0)
+               return;
+
+       start = 0;
+       end = len - 1;
+
+       while (start < end) {
+               tmp = arr[start];
+               arr[start] = arr[end];
+               arr[end] = tmp;
+               start++;
+               end--;
+       }
+}
+
 int __roc_api roc_se_auth_key_set(struct roc_se_ctx *se_ctx,
                                  roc_se_auth_type type, const uint8_t *key,
                                  uint16_t key_len, uint16_t mac_len);
@@ -310,4 +331,5 @@ int __roc_api roc_se_ciph_key_set(struct roc_se_ctx *se_ctx,
                                  uint16_t key_len, uint8_t *salt);
 
 void __roc_api roc_se_ctx_swap(struct roc_se_ctx *se_ctx);
+
 #endif /* __ROC_SE_H__ */
-- 
2.28.0

Reply via email to