RE: [PATCH v5 1/3] common/qat: isolate parser arguments configuration

["Dooley, Brian" <brian.dooley@intel.com>]

Hi Arek,

> -----Original Message-----
> From: Arkadiusz Kusztal <arkadiuszx.kusztal@intel.com>
> Sent: Friday, March 1, 2024 3:53 PM
> To: dev@dpdk.org
> Cc: gakhil@marvell.com; Power, Ciara <ciara.power@intel.com>; Kusztal,
> ArkadiuszX <arkadiuszx.kusztal@intel.com>
> Subject: [PATCH v5 1/3] common/qat: isolate parser arguments configuration
> 
> This commit isolates qat device arguments from the common code. Now
> arguments are defined per service, and only appear in the application if the
> service is compiled-in.
> 
> Depends-on: patch-137678 ("common/qat: add virtual qat device (vQAT)")
> 
> Signed-off-by: Arkadiusz Kusztal <arkadiuszx.kusztal@intel.com>
> ---
> v2:
> - added pmd isolation
> v3:
> - added fix for legacy flag
> v4:
> - fixed an issue with devargs segfault
> v5:
> - rebased against the newest changes in the pmd
> 
>  drivers/common/qat/qat_common.c     |  11 +++
>  drivers/common/qat/qat_common.h     |   3 +
>  drivers/common/qat/qat_device.c     | 189 +++++++++++++++++++-----------
> ------
>  drivers/common/qat/qat_device.h     |  27 ++----
>  drivers/compress/qat/qat_comp_pmd.c |  31 ++++--
>  drivers/compress/qat/qat_comp_pmd.h |   3 +-
>  drivers/crypto/qat/qat_asym.c       |  38 +++++---
>  drivers/crypto/qat/qat_sym.c        |  49 ++++++----
>  8 files changed, 198 insertions(+), 153 deletions(-)
> 
<snip>

Acked-by: Brian Dooley <brian.dooley@intel.com>