Re: [dpdk-dev] [PATCH v2 01/13] ethdev: change promiscuous mode controllers to return errors

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 9/9/2019 12:58 PM, Andrew Rybchenko wrote:
> From: Ivan Ilchenko <Ivan.Ilchenko@oktetlabs.ru>
> 
> Change rte_eth_promiscuous_enable()/rte_eth_promiscuous_disable()
> return value from void to int and return negative errno values
> in case of error conditions.
> Modify usage of these functions across the ethdev according
> to new return type.
> 
> Signed-off-by: Ivan Ilchenko <Ivan.Ilchenko@oktetlabs.ru>
> Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>

<...>

> diff --git a/doc/guides/sample_app_ug/flow_filtering.rst b/doc/guides/sample_app_ug/flow_filtering.rst
> index 02fc675506..de3e4ab0b6 100644
> --- a/doc/guides/sample_app_ug/flow_filtering.rst
> +++ b/doc/guides/sample_app_ug/flow_filtering.rst
> @@ -193,7 +193,13 @@ application is shown below:
>                     }
>            }
>  
> -           rte_eth_promiscuous_enable(port_id);
> +           ret = rte_eth_promiscuous_enable(port_id);
> +           if (ret != 0) {
> +                   rte_exit(EXIT_FAILURE,
> +                           ":: cannot enable promiscuous mode: err=%d, port=%u\n",
> +                           ret, port_id);
> +           }
> +

Not for this patch, for future, I am for not putting code into the documents...