Re: [PATCH] stats: remove useless memset's

[Thomas Monjalon <thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>]

2015-01-21 14:08, David Marchand:
> Hello Stephen,
> 
> > From: Stephen Hemminger <shemming-43mecJUBy8ZBDgjK7y7TUQ@public.gmane.org>
> >
> > 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-OTpzqLSitTUnbdJkjeBofR2eb7JE58TQ@public.gmane.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-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>

Applied with above fixes.

Thanks
-- 
Thomas