On Thu, Nov 18, 2021 at 4:31 PM Kevin Traynor <[email protected]> wrote: > > @@ -1583,6 +1584,13 @@ netdev_dpdk_get_xstat_name(struct netdev_dpdk *dev, > > uint64_t id) > > return dev->rte_xstats_names[id].name; > > } > > > > +/* We filter out everything except per rxq/txq basic stats, and dropped, > > + * error and management counters. > > + * Note: rx_qX_errors is handled by the _errors$ pattern. > > + */ > > +#define DPDK_STATS_REGEX_FILTER \ > > + "(^(rx_q|tx_q)[0-9]*_(packets|bytes)$|_errors$|_dropped$|_management_)" > > + > > iirc, there was some pmds that were not conforming correctly with this > rx_q syntax, but I *think* they were fixed a few releases ago in DPDK. > Just mentioning because if there was still some, they could be fixed or > the regex could be expanded to capture them.
In theory, those per q stats are part of the "basic" stats handled at the ethdev level. There should be no issue for them. I remember an issue with incorrect stats, but it was a conflict between OVS and i40e stats. Is this what you had in mind? -- David Marchand _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
