Acked-by: Kai Ji <kai...@intel.com>

________________________________
From: Radu Nicolau <radu.nico...@intel.com>
Sent: 12 June 2025 11:14
To: Ji, Kai <kai...@intel.com>; Kusztal, ArkadiuszX 
<arkadiuszx.kusz...@intel.com>
Cc: dev@dpdk.org <dev@dpdk.org>; Nicolau, Radu <radu.nico...@intel.com>; 
sta...@dpdk.org <sta...@dpdk.org>
Subject: [PATCH] crypto/qat: fix out-of-place chain/cipher/auth headers

Extend the original fix to gen3 and gen4 code.

Fixes: 317d05f3721c ("crypto/qat: fix out-of-place chain/cipher/auth headers")
Cc: arkadiuszx.kusz...@intel.com
Cc: sta...@dpdk.org

Signed-off-by: Radu Nicolau <radu.nico...@intel.com>
---
 drivers/crypto/qat/dev/qat_crypto_pmd_gen3.c | 13 ++++++++-----
 drivers/crypto/qat/dev/qat_crypto_pmd_gen4.c | 13 ++++++++-----
 2 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/drivers/crypto/qat/dev/qat_crypto_pmd_gen3.c 
b/drivers/crypto/qat/dev/qat_crypto_pmd_gen3.c
index 478e766b97..428430882d 100644
--- a/drivers/crypto/qat/dev/qat_crypto_pmd_gen3.c
+++ b/drivers/crypto/qat/dev/qat_crypto_pmd_gen3.c
@@ -643,16 +643,19 @@ qat_sym_dp_enqueue_aead_jobs_gen3(void *qp_data, uint8_t 
*drv_ctx,
         for (i = 0; i < n; i++) {
                 struct qat_sym_op_cookie *cookie =
                         qp->op_cookies[tail >> tx_queue->trailz];
+               int error = 0;

                 req  = (struct icp_qat_fw_la_bulk_req *)(
                         (uint8_t *)tx_queue->base_addr + tail);
                 rte_mov128((uint8_t *)req, (const uint8_t *)&(ctx->fw_req));

                 if (vec->dest_sgl) {
-                       data_len = qat_sym_build_req_set_data(req,
-                               user_data[i], cookie,
-                               vec->src_sgl[i].vec, vec->src_sgl[i].num,
-                               vec->dest_sgl[i].vec, vec->dest_sgl[i].num);
+                       data_len = qat_reqs_mid_set(&error, req, cookie, 
user_data[i],
+                                       &vec->src_sgl[i], &vec->dest_sgl[i], 
ofs);
+                       /* In oop there is no offset, src/dst addresses are 
moved
+                        * to avoid overwriting the dst header
+                        */
+                       ofs.ofs.cipher.head = 0;
                 } else {
                         data_len = qat_sym_build_req_set_data(req,
                                 user_data[i], cookie,
@@ -660,7 +663,7 @@ qat_sym_dp_enqueue_aead_jobs_gen3(void *qp_data, uint8_t 
*drv_ctx,
                                 vec->src_sgl[i].num, NULL, 0);
                 }

-               if (unlikely(data_len < 0))
+               if (unlikely(data_len < 0) || error)
                         break;

                 enqueue_one_aead_job_gen3(ctx, req, &vec->iv[i],
diff --git a/drivers/crypto/qat/dev/qat_crypto_pmd_gen4.c 
b/drivers/crypto/qat/dev/qat_crypto_pmd_gen4.c
index 6a5d6e78b9..638da1a173 100644
--- a/drivers/crypto/qat/dev/qat_crypto_pmd_gen4.c
+++ b/drivers/crypto/qat/dev/qat_crypto_pmd_gen4.c
@@ -418,16 +418,19 @@ qat_sym_dp_enqueue_aead_jobs_gen4(void *qp_data, uint8_t 
*drv_ctx,
         for (i = 0; i < n; i++) {
                 struct qat_sym_op_cookie *cookie =
                         qp->op_cookies[tail >> tx_queue->trailz];
+               int error = 0;

                 req  = (struct icp_qat_fw_la_bulk_req *)(
                         (uint8_t *)tx_queue->base_addr + tail);
                 rte_mov128((uint8_t *)req, (const uint8_t *)&(ctx->fw_req));

                 if (vec->dest_sgl) {
-                       data_len = qat_sym_build_req_set_data(req,
-                               user_data[i], cookie,
-                               vec->src_sgl[i].vec, vec->src_sgl[i].num,
-                               vec->dest_sgl[i].vec, vec->dest_sgl[i].num);
+                       data_len = qat_reqs_mid_set(&error, req, cookie, 
user_data[i],
+                                       &vec->src_sgl[i], &vec->dest_sgl[i], 
ofs);
+                       /* In oop there is no offset, src/dst addresses are 
moved
+                        * to avoid overwriting the dst header
+                        */
+                       ofs.ofs.cipher.head = 0;
                 } else {
                         data_len = qat_sym_build_req_set_data(req,
                                 user_data[i], cookie,
@@ -435,7 +438,7 @@ qat_sym_dp_enqueue_aead_jobs_gen4(void *qp_data, uint8_t 
*drv_ctx,
                                 vec->src_sgl[i].num, NULL, 0);
                 }

-               if (unlikely(data_len < 0))
+               if (unlikely(data_len < 0) || error)
                         break;

                 enqueue_one_aead_job_gen4(ctx, req, &vec->iv[i],
--
2.43.0

Reply via email to