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?

>
>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>

Reply via email to