On Thu, Jul 25, 2019 at 5:49 PM Ye Xiaolong <xiaolong...@intel.com> wrote: > > On 07/25, David Marchand wrote: > >This Tx counter is now unused. > > > >Fixes: 10edf857fde4 ("net/af_xdp: make reserve/submit peek/release > >consistent") > > err_pkts seems not introduced by this commit?
This commit removed the only place where it was used. > > > > >Signed-off-by: David Marchand <david.march...@redhat.com> > >--- > > drivers/net/af_xdp/rte_eth_af_xdp.c | 2 -- > > 1 file changed, 2 deletions(-) > > > >diff --git a/drivers/net/af_xdp/rte_eth_af_xdp.c > >b/drivers/net/af_xdp/rte_eth_af_xdp.c > >index 33352e1..41ed5b2 100644 > >--- a/drivers/net/af_xdp/rte_eth_af_xdp.c > >+++ b/drivers/net/af_xdp/rte_eth_af_xdp.c > >@@ -97,7 +97,6 @@ struct pkt_rx_queue { > > > > struct tx_stats { > > uint64_t tx_pkts; > >- uint64_t err_pkts; > > uint64_t tx_bytes; > > }; > > > >@@ -456,7 +455,6 @@ eth_stats_get(struct rte_eth_dev *dev, struct > >rte_eth_stats *stats) > > stats->imissed += xdp_stats.rx_dropped; > > > > stats->opackets += stats->q_opackets[i]; > >- stats->oerrors += txq->stats.err_pkts; > > stats->obytes += stats->q_obytes[i]; > > } > > > >-- > >1.8.3.1 > > > > Reviewed-by: Xiaolong Ye <xiaolong...@intel.com> Thanks. -- David Marchand