Re: [dpdk-dev] [PATCH] stats: remove useless memset's

[David Marchand <david.marchand@6wind.com>]

Hello Stephen,

On Wed, Jan 21, 2015 at 5:16 AM, <stephen@networkplumber.org> wrote:

> From: Stephen Hemminger <shemming@brocade.com>
>
> The rte_eth_stats_get is the only API that should call the device
> statistics function directly, and it already does a memset of the
> resulting structure. Therefore doing memset() in the driver is
> redundant and should be removed.
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  lib/librte_pmd_af_packet/rte_eth_af_packet.c | 2 --
>  lib/librte_pmd_bond/rte_eth_bond_pmd.c       | 4 ----
>  lib/librte_pmd_enic/enic_main.c              | 1 -
>  lib/librte_pmd_i40e/i40e_ethdev_vf.c         | 1 -
>  lib/librte_pmd_ixgbe/ixgbe_ethdev.c          | 1 -
>  lib/librte_pmd_ring/rte_eth_ring.c           | 1 -
>  6 files changed, 10 deletions(-)
>

I think you missed some :
- lib/librte_pmd_e1000/igb_ethdev.c function eth_igbvf_stats_get()
- lib/librte_pmd_pcap/rte_eth_pcap.c function eth_stats_get()

With these fixed :
Acked-By: David Marchand <david.marchand@6wind.com>

-- 
David Marchand