[dpdk-dev] [PATCH] examples/vhost_crypto: fix bracket

Maxime Coquelin maxime.coquelin at redhat.com
Tue Dec 11 19:14:30 CET 2018



On 12/11/18 4:09 PM, Fan Zhang wrote:
> This patch fixes a missed bracket bug in the code.
> 
> Fixes: e1d90eaa83ff ("examples/vhost_crypto: fix zero copy")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Fan Zhang <roy.fan.zhang at intel.com>
> ---
>   examples/vhost_crypto/main.c | 9 +++++----
>   1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/examples/vhost_crypto/main.c b/examples/vhost_crypto/main.c
> index f08babd97..3deb5263f 100644
> --- a/examples/vhost_crypto/main.c
> +++ b/examples/vhost_crypto/main.c
> @@ -503,11 +503,12 @@ main(int argc, char *argv[])
>   			if (strstr(dev_info.driver_name,
>   				RTE_STR(VHOST_CRYPTO_CDEV_NAME_AESNI_MB_PMD)) ||
>   				strstr(dev_info.driver_name,
> -				RTE_STR(VHOST_CRYPTO_CDEV_NAME_AESNI_GCM_PMD)))
> -			RTE_LOG(ERR, USER1, "Cannot enable zero-copy in %s\n",
> +				RTE_STR(VHOST_CRYPTO_CDEV_NAME_AESNI_GCM_PMD))) {
> +				RTE_LOG(ERR, USER1, "Cannot enable zero-copy in %s\n",
>   					dev_info.driver_name);
> -			ret = -EPERM;
> -			goto error_exit;
> +				ret = -EPERM;
> +				goto error_exit;
> +			}
>   		}
>   
>   		if (dev_info.max_nb_queue_pairs < info->qid + 1) {
> 


Reviewed-by: Maxime Coquelin <maxime.coquelin at redhat.com>

Thanks,
Maxime


More information about the dev mailing list