Hi John, On Fri, 16 Dec 2016 14:28:21 +0000, "Mcnamara, John" <john.mcnam...@intel.com> wrote: > > -----Original Message----- > > From: stable [mailto:stable-boun...@dpdk.org] On Behalf Of Olivier > > Matz Sent: Friday, December 16, 2016 9:44 AM > > To: dev@dpdk.org; thomas.monja...@6wind.com > > Cc: Horton, Remy <remy.hor...@intel.com>; sta...@dpdk.org > > Subject: [dpdk-stable] [PATCH 2/2] ethdev: clarify xstats Api > > documentation > > > > Reword the Api documentation of xstats ethdev. > > > > CC: sta...@dpdk.org > > Signed-off-by: Olivier Matz <olivier.m...@6wind.com> > > --- > > lib/librte_ether/rte_ethdev.h | 45 > > ++++++++++++++++++++++++-------------- ----- > > ... > > int rte_eth_xstats_get_names(uint8_t port_id, > > struct rte_eth_xstat_name *xstats_names, @@ > > -2296,19 +2300,20 @@ int rte_eth_xstats_get_names(uint8_t port_id, > > * The port identifier of the Ethernet device. > > * @param xstats > > * A pointer to a table of structure of type *rte_eth_xstat* > > - * to be filled with device statistics ids and values. > > + * to be filled with device statistics ids and values: id is the > > + * index of the name string in xstats_names (@see > > rte_eth_xstats_get_names), > > The @see directive starts a new "See also" section and > breaks/interrupts the parameter description. Probably what you want > is: > > index of the name string in xstats_names (see > rte_eth_xstats_get_names()), > > Otherwise it is a good update. > > John
Thank you for the review. I'll send a new version of the patch addressing this. Regards, Olivier
