[dpdk-dev] [PATCH 2/4] ethdev: expose extended error stats

2015-06-22 Thread Olivier MATZ
Hello Maryam, On 06/22/2015 04:12 PM, Tahhan, Maryam wrote: > >> 2015-06-05 18:35, Maryam Tahhan: >>> Extend rte_eth_xstats_get to retrieve additional stats from the device >>> driver as well the top level extended stats. Add additional drop >>> counters to the extended stats. >>> >>> Signed-off-

[dpdk-dev] [PATCH 2/4] ethdev: expose extended error stats

2015-06-22 Thread Tahhan, Maryam
> >>> + packets. */ > >>> }; > >> > >> You are extending the generic stats. This is not the idea behind xstats. > >> The xstats are specific to the driver. > > > > I'd followed the example of: > > http://patchwork.dpdk.org/dev/patchwork/patch/85/ > > to added generic extended stats (at least what

[dpdk-dev] [PATCH 2/4] ethdev: expose extended error stats

2015-06-22 Thread Tahhan, Maryam
> 2015-06-05 18:35, Maryam Tahhan: > > Extend rte_eth_xstats_get to retrieve additional stats from the device > > driver as well the top level extended stats. Add additional drop > > counters to the extended stats. > > > > Signed-off-by: Maryam Tahhan > [..] > Patch 1/4 doesn't compile without pa

[dpdk-dev] [PATCH 2/4] ethdev: expose extended error stats

2015-06-17 Thread Thomas Monjalon
2015-06-05 18:35, Maryam Tahhan: > Extend rte_eth_xstats_get to retrieve additional stats from the device > driver as well the top level extended stats. Add additional drop > counters to the extended stats. > > Signed-off-by: Maryam Tahhan [..] Patch 1/4 doesn't compile without patch 2/4. > ---

[dpdk-dev] [PATCH 2/4] ethdev: expose extended error stats

2015-06-17 Thread Kyle Larose
On Wed, Jun 17, 2015 at 9:58 AM, Thomas Monjalon wrote: [...] > > You are extending the generic stats. This is not the idea behind xstats. > The xstats are specific to the driver. > Furthermore we should migrate some "not really generic stats" to xstats > in order to keep only the really basic and

[dpdk-dev] [PATCH 2/4] ethdev: expose extended error stats

2015-06-05 Thread Maryam Tahhan
Extend rte_eth_xstats_get to retrieve additional stats from the device driver as well the top level extended stats. Add additional drop counters to the extended stats. Signed-off-by: Maryam Tahhan --- lib/librte_ether/rte_ethdev.c | 12 lib/librte_ether/rte_ethdev.h | 4 2 fil