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

> -----Original Message-----
> From: Karas, Krzysztof <krzysztof.ka...@intel.com>
> Sent: Monday, March 13, 2023 12:01 PM
> To: Thomas Monjalon <tho...@monjalon.net>; Ji, Kai <kai...@intel.com>;
> De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; Kusztal,
> ArkadiuszX <arkadiuszx.kusz...@intel.com>
> Cc: dev@dpdk.org; Karas, Krzysztof <krzysztof.ka...@intel.com>
> Subject: [PATCH v2] crypto/ipsec_mb: expect ENOTSUP
> 
> Commit b35848bc01f6 ("crypto/ipsec_mb: add multi-process IPC request
> handler") is part of DPDK 22.11 release and it enforces usage of shared
> config and multiprocess mode.
> 
> DPDK is not required to be run with more than one process in general, so
> instead of erroring out, catch the error and proceed with ipsec_mb
> device creation normally.
> 
> Fixes: b35848bc01f6 ("crypto/ipsec_mb: add multi-process IPC request
> handler")
> Signed-off-by: Krzysztof Karas <krzysztof.ka...@intel.com>
> ---
> v2:
>  * Modified patch title and added "Fixes:" note.
>  * Restored original comparison "rte_errno == EEXIST".
> 
>  .mailmap                                   | 1 +
>  drivers/crypto/ipsec_mb/ipsec_mb_private.c | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/.mailmap b/.mailmap
> index a9f4f28fba..dad475b9d0 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -1583,3 +1583,4 @@ Ziye Yang <ziye.y...@intel.com>  Zoltan Kiss
> <zoltan.k...@schaman.hu> <zoltan.k...@linaro.org>  Zorik Machulsky
> <zo...@amazon.com>  Zyta Szpak <z...@marvell.com> <z...@semihalf.com>
> <zyta.sz...@semihalf.com>
> +Krzysztof Karas <krzysztof.ka...@intel.com>
> diff --git a/drivers/crypto/ipsec_mb/ipsec_mb_private.c
> b/drivers/crypto/ipsec_mb/ipsec_mb_private.c
> index 50b789a29b..64f2b4b604 100644
> --- a/drivers/crypto/ipsec_mb/ipsec_mb_private.c
> +++ b/drivers/crypto/ipsec_mb/ipsec_mb_private.c
> @@ -170,7 +170,7 @@ ipsec_mb_create(struct rte_vdev_device *vdev,
> 
>       if (rte_eal_process_type() == RTE_PROC_PRIMARY) {
>               retval = ipsec_mb_mp_request_register();
> -             if (retval && (rte_errno == EEXIST))
> +             if (retval && ((rte_errno == EEXIST) || (rte_errno ==
> ENOTSUP)))
>                       /* Safe to proceed, return 0 */
>                       return 0;
> 
> --
> 2.34.1

Reply via email to