]> git.droids-corp.org - dpdk.git/commitdiff
crypto/cnxk: use struct sizes for CTX writes
authorAnoob Joseph <anoobj@marvell.com>
Fri, 17 Dec 2021 09:19:56 +0000 (14:49 +0530)
committerAkhil Goyal <gakhil@marvell.com>
Fri, 21 Jan 2022 08:40:01 +0000 (09:40 +0100)
CTX writes only require the lengths are 8B aligned. Use the struct size
directly.

Signed-off-by: Anoob Joseph <anoobj@marvell.com>
Acked-by: Akhil Goyal <gakhil@marvell.com>
drivers/crypto/cnxk/cn10k_ipsec.c

index b4acbaca3190d2aad117e5c6fc30d7364e2b66b8..0832b53e2e4df725c2d4adb6baf46006b32a9f8c 100644 (file)
@@ -52,14 +52,12 @@ cn10k_ipsec_outb_sa_create(struct roc_cpt *roc_cpt, struct roc_cpt_lf *lf,
        out_sa = &sa->out_sa;
 
        /* Allocate memory to be used as dptr for CPT ucode WRITE_SA op */
-       sa_dptr = plt_zmalloc(ROC_NIX_INL_OT_IPSEC_OUTB_HW_SZ, 0);
+       sa_dptr = plt_zmalloc(sizeof(struct roc_ot_ipsec_outb_sa), 8);
        if (sa_dptr == NULL) {
                plt_err("Couldn't allocate memory for SA dptr");
                return -ENOMEM;
        }
 
-       memset(sa_dptr, 0, sizeof(struct roc_ot_ipsec_outb_sa));
-
        /* Translate security parameters to SA */
        ret = cnxk_ot_ipsec_outb_sa_fill(sa_dptr, ipsec_xfrm, crypto_xfrm);
        if (ret) {
@@ -133,7 +131,7 @@ cn10k_ipsec_outb_sa_create(struct roc_cpt *roc_cpt, struct roc_cpt_lf *lf,
 
        /* Write session using microcode opcode */
        ret = roc_cpt_ctx_write(lf, sa_dptr, out_sa,
-                               ROC_NIX_INL_OT_IPSEC_OUTB_HW_SZ);
+                               sizeof(struct roc_ot_ipsec_outb_sa));
        if (ret) {
                plt_err("Could not write outbound session to hardware");
                goto sa_dptr_free;
@@ -169,14 +167,12 @@ cn10k_ipsec_inb_sa_create(struct roc_cpt *roc_cpt, struct roc_cpt_lf *lf,
        in_sa = &sa->in_sa;
 
        /* Allocate memory to be used as dptr for CPT ucode WRITE_SA op */
-       sa_dptr = plt_zmalloc(ROC_NIX_INL_OT_IPSEC_INB_HW_SZ, 0);
+       sa_dptr = plt_zmalloc(sizeof(struct roc_ot_ipsec_inb_sa), 8);
        if (sa_dptr == NULL) {
                plt_err("Couldn't allocate memory for SA dptr");
                return -ENOMEM;
        }
 
-       memset(sa_dptr, 0, sizeof(struct roc_ot_ipsec_inb_sa));
-
        /* Translate security parameters to SA */
        ret = cnxk_ot_ipsec_inb_sa_fill(sa_dptr, ipsec_xfrm, crypto_xfrm);
        if (ret) {
@@ -225,7 +221,7 @@ cn10k_ipsec_inb_sa_create(struct roc_cpt *roc_cpt, struct roc_cpt_lf *lf,
 
        /* Write session using microcode opcode */
        ret = roc_cpt_ctx_write(lf, sa_dptr, in_sa,
-                               ROC_NIX_INL_OT_IPSEC_INB_HW_SZ);
+                               sizeof(struct roc_ot_ipsec_inb_sa));
        if (ret) {
                plt_err("Could not write inbound session to hardware");
                goto sa_dptr_free;