On Fri, Nov 10, 2023 at 9:52 AM Maxime Coquelin
<maxime.coque...@redhat.com> wrote:
>
> From: Stephen Hemminger <step...@networkplumber.org>
>
> Transistion API's added from 2008 to 2022 to stable, except
> Vhost-async's which are likely to require changes to be adopted.
>
> Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
> Signed-off-by: Maxime Coquelin <maxime.coque...@redhat.com>

I have one comment below, but this can probably be fixed when applying.

Reviewed-by: David Marchand <david.march...@redhat.com>


> diff --git a/lib/vhost/rte_vhost_crypto.h b/lib/vhost/rte_vhost_crypto.h
> index 2b01ecda08..71a30ba202 100644
> --- a/lib/vhost/rte_vhost_crypto.h
> +++ b/lib/vhost/rte_vhost_crypto.h
> @@ -40,7 +40,6 @@ enum rte_vhost_crypto_zero_copy {
>   * @return
>   *  0 on success, -1 on failure
>   */
> -__rte_experimental
>  int
>  rte_vhost_crypto_driver_start(const char *path);
>

No need to include rte_compat.h in rte_vhost_crypto.h anymore: no
symbol is marked internal or experimental after this change.


-- 
David Marchand

Reply via email to