On 9/9/2019 12:58 PM, Andrew Rybchenko wrote: > From: Ivan Ilchenko <ivan.ilche...@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.ilche...@oktetlabs.ru> > Signed-off-by: Andrew Rybchenko <arybche...@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...