common/qat: fix undefined macro
Checks
Commit Message
When using RTE_ENABLE_ASSERT and debug mode, an undefined
macro error appeared for ICP_QAT_FW_SYM_COMM_ADDR_SGL.
This was not being defined, but is now added to the header file.
Bugzilla ID: 1395
Fixes: e9271821e489 ("common/qat: support GEN LCE device")
Signed-off-by: Ciara Power <ciara.power@intel.com>
---
Cc: nishikanta.nayak@intel.com
---
drivers/common/qat/qat_adf/icp_qat_fw_la.h | 1 +
1 file changed, 1 insertion(+)
Comments
> -----Original Message-----
> From: Power, Ciara <ciara.power@intel.com>
> Sent: Tuesday, March 5, 2024 5:26 PM
> To: dev@dpdk.org
> Cc: gakhil@marvell.com; Power, Ciara <ciara.power@intel.com>; Nayak,
> Nishikanta <nishikanta.nayak@intel.com>; Ji, Kai <kai.ji@intel.com>
> Subject: [PATCH] common/qat: fix undefined macro
>
> When using RTE_ENABLE_ASSERT and debug mode, an undefined macro
> error appeared for ICP_QAT_FW_SYM_COMM_ADDR_SGL.
> This was not being defined, but is now added to the header file.
>
> Bugzilla ID: 1395
> Fixes: e9271821e489 ("common/qat: support GEN LCE device")
>
> Signed-off-by: Ciara Power <ciara.power@intel.com>
>
> ---
> Cc: nishikanta.nayak@intel.com
> ---
> drivers/common/qat/qat_adf/icp_qat_fw_la.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/common/qat/qat_adf/icp_qat_fw_la.h
> b/drivers/common/qat/qat_adf/icp_qat_fw_la.h
> index 67fc25c919..fe32b66c50 100644
> --- a/drivers/common/qat/qat_adf/icp_qat_fw_la.h
> +++ b/drivers/common/qat/qat_adf/icp_qat_fw_la.h
> @@ -111,6 +111,7 @@ struct icp_qat_fw_la_bulk_req { #define
> ICP_QAT_FW_SYM_IV_IN_DESC_VALID 1 #define
> ICP_QAT_FW_SYM_DIRECTION_BITPOS 15 #define
> ICP_QAT_FW_SYM_DIRECTION_MASK 0x1
> +#define ICP_QAT_FW_SYM_COMM_ADDR_SGL 1
>
> /* In GEN_LCE AEAD AES GCM Algorithm has ID 0 */ #define
> QAT_LA_CRYPTO_AEAD_AES_GCM_GEN_LCE 0
> --
> 2.25.1
Acked-by: Nishikant Nayak <nishikanta.nayak@intel.com>
> -----Original Message-----
> From: Ciara Power <ciara.power@intel.com>
> Sent: Tuesday, March 5, 2024 1:56 PM
> To: dev@dpdk.org
> Cc: gakhil@marvell.com; Ciara Power <ciara.power@intel.com>;
> nishikanta.nayak@intel.com; Kai Ji <kai.ji@intel.com>
> Subject: [PATCH] common/qat: fix undefined macro
>
> When using RTE_ENABLE_ASSERT and debug mode, an undefined
> macro error appeared for ICP_QAT_FW_SYM_COMM_ADDR_SGL.
> This was not being defined, but is now added to the header file.
>
> Bugzilla ID: 1395
> Fixes: e9271821e489 ("common/qat: support GEN LCE device")
>
> Signed-off-by: Ciara Power <ciara.power@intel.com>
>
> ---
Can confirm it resolves the build failure, thanks!
Tested-by: Ali Alnubani <alialnu@nvidia.com>
05/03/2024 13:15, Ali Alnubani:
> > -----Original Message-----
> > From: Ciara Power <ciara.power@intel.com>
> > Sent: Tuesday, March 5, 2024 1:56 PM
> > To: dev@dpdk.org
> > Cc: gakhil@marvell.com; Ciara Power <ciara.power@intel.com>;
> > nishikanta.nayak@intel.com; Kai Ji <kai.ji@intel.com>
> > Subject: [PATCH] common/qat: fix undefined macro
> >
> > When using RTE_ENABLE_ASSERT and debug mode, an undefined
> > macro error appeared for ICP_QAT_FW_SYM_COMM_ADDR_SGL.
> > This was not being defined, but is now added to the header file.
> >
> > Bugzilla ID: 1395
> > Fixes: e9271821e489 ("common/qat: support GEN LCE device")
> >
> > Signed-off-by: Ciara Power <ciara.power@intel.com>
> >
> > ---
>
> Can confirm it resolves the build failure, thanks!
>
> Tested-by: Ali Alnubani <alialnu@nvidia.com>
Reported-by: Ali Alnubani <alialnu@nvidia.com>
Applied, thanks.
@@ -111,6 +111,7 @@ struct icp_qat_fw_la_bulk_req {
#define ICP_QAT_FW_SYM_IV_IN_DESC_VALID 1
#define ICP_QAT_FW_SYM_DIRECTION_BITPOS 15
#define ICP_QAT_FW_SYM_DIRECTION_MASK 0x1
+#define ICP_QAT_FW_SYM_COMM_ADDR_SGL 1
/* In GEN_LCE AEAD AES GCM Algorithm has ID 0 */
#define QAT_LA_CRYPTO_AEAD_AES_GCM_GEN_LCE 0