|SUCCESS| common/qat: add asym crypto disable option

[dpdklab at iol.unh.edu]

Test-Label: ai-code-review
Test-Status: SUCCESS
https://dpdk.org/patch/163121

_AI Code Review_

=== Patch Review: series.patch (via Claude) ===
Review date: 2026-04-16

I will review this patch against the DPDK coding standards and guidelines.

```
git sha: 20260416102543.133175-1-radu.nicolau at intel.com
Author: Radu Nicolau <radu.nicolau at intel.com>
Subject: common/qat: add asym crypto disable option

This patch adds the ability to disable the Asymmetric crypto QAT PMD
independently using the meson disable_drivers option. It relocates asym
crypto source files to a subdirectory and adds conditional compilation
guards.

> diff --git a/drivers/common/qat/meson.build b/drivers/common/qat/meson.build
> index 31e06f4376..7f12f2b09d 100644
> --- a/drivers/common/qat/meson.build
> +++ b/drivers/common/qat/meson.build

[ ... ]

> @@ -108,9 +118,8 @@ cflags += no_shadow_cflag
>  
>  if qat_crypto
>      foreach f: ['qat_sym.c', 'qat_sym_session.c',
> -            'qat_asym.c', 'qat_crypto.c',
> +            'qat_crypto.c',
>              'dev/qat_sym_pmd_gen1.c',
> -            'dev/qat_asym_pmd_gen1.c',
>              'dev/qat_crypto_pmd_gen2.c',
>              'dev/qat_crypto_pmd_gen3.c',
>              'dev/qat_crypto_pmd_gen4.c',
> @@ -119,6 +128,14 @@ if qat_crypto
>          ]
>          sources += files(join_paths(qat_crypto_relpath, f))
>      endforeach
> +    cflags += '-DBUILD_QAT_SYM'
> +    if qat_crypto_asym
> +        foreach f: ['asym/qat_asym.c',
> +                    'dev/qat_asym_pmd_gen1.c',
> +            ]
> +            sources += files(join_paths(qat_crypto_relpath, f))
> +        endforeach
> +        cflags += '-DBUILD_QAT_ASYM'
> +    endif
>      deps += ['security']
> -    cflags += ['-DBUILD_QAT_SYM', '-DBUILD_QAT_ASYM']
>  endif

[ ... ]

> diff --git a/drivers/crypto/qat/qat_crypto.c b/drivers/crypto/qat/qat_crypto.c
> index 861679373b..daa36c5eed 100644
> --- a/drivers/crypto/qat/qat_crypto.c
> +++ b/drivers/crypto/qat/qat_crypto.c
> @@ -53,9 +53,11 @@ qat_cryptodev_info_get(struct rte_cryptodev *dev,
>  			qat_qps_per_service(qat_dev, service_type);
>  		info->feature_flags = dev->feature_flags;
>  		info->capabilities = qat_private->qat_dev_capabilities;
> +
> +#ifdef BUILD_QAT_ASYM
>  		if (service_type == QAT_SERVICE_ASYMMETRIC)
>  			info->driver_id = qat_asym_driver_id;
> -
> +#endif

In qat_cryptodev_info_get(), does the code handle service_type ==
QAT_SERVICE_ASYMMETRIC when BUILD_QAT_ASYM is not defined? If asymmetric
service is requested but disabled at build time, will info->driver_id remain
uninitialized? Should this condition return an error or log a warning?

>  		if (service_type == QAT_SERVICE_SYMMETRIC)
>  			info->driver_id = qat_sym_driver_id;
>  		/* No limit of number of sessions */
> @@ -168,8 +170,10 @@ qat_cryptodev_qp_setup(struct rte_cryptodev *dev, uint16_t qp_id,
>  	for (i = 0; i < qp->nb_descriptors; i++) {
>  		if (service_type == QAT_SERVICE_SYMMETRIC)
>  			qat_sym_init_op_cookie(qp->op_cookies[i]);
> +#ifdef BUILD_QAT_ASYM
>  		else
>  			qat_asym_init_op_cookie(qp->op_cookies[i]);
> +#endif

In qat_cryptodev_qp_setup(), does this code correctly handle service_type
!= QAT_SERVICE_SYMMETRIC when BUILD_QAT_ASYM is not defined? If service_type
is QAT_SERVICE_ASYMMETRIC but BUILD_QAT_ASYM is disabled, will the
op_cookies[i] remain uninitialized? Should there be an else clause to
handle this condition with an error?

>  	}
>  
>  	if (qat_private->cipher_crc_offload_enable) {

```