[dpdk-dev] [PATCH] stats: remove useless memset's
Thomas Monjalon
thomas.monjalon at 6wind.com
Tue Jan 27 13:06:20 CET 2015
2015-01-21 14:08, David Marchand:
> Hello Stephen,
>
> > From: Stephen Hemminger <shemming at 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 at 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 at 6wind.com>
Applied with above fixes.
Thanks
--
Thomas
More information about the dev
mailing list