[PATCH v2 5/9] net/e1000: remove queue stats mapping
David Marchand
david.marchand at redhat.com
Fri Jul 24 09:31:21 CEST 2026
On Thu, 23 Jul 2026 at 22:30, Stephen Hemminger
<stephen at networkplumber.org> wrote:
> @@ -2070,19 +2060,11 @@ eth_igc_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *rte_stats,
> /* Get per-queue statuses */
> if (qstats) {
> for (i = 0; i < IGC_QUEUE_PAIRS_NUM; i++) {
> - /* GET TX queue statuses */
> - int map_id = igc->txq_stats_map[i];
> - if (map_id >= 0) {
> - qstats->q_opackets[map_id] += queue_stats->pqgptc[i];
> - qstats->q_obytes[map_id] += queue_stats->pqgotc[i];
> - }
> - /* Get RX queue statuses */
> - map_id = igc->rxq_stats_map[i];
> - if (map_id >= 0) {
> - qstats->q_ipackets[map_id] += queue_stats->pqgprc[i];
> - qstats->q_ibytes[map_id] += queue_stats->pqgorc[i];
> - qstats->q_errors[map_id] += queue_stats->rqdpc[i];
> - }
> + qstats->q_opackets[i] += queue_stats->pqgptc[i];
> + qstats->q_obytes[i] += queue_stats->pqgotc[i];
> + qstats->q_ipackets[i] += queue_stats->pqgprc[i];
> + qstats->q_ibytes[i] += queue_stats->pqgorc[i];
> + qstats->q_errors[i] += queue_stats->rqdpc[i];
> }
> }
>
I suppose this is harmless.. but strange to read all queues regardless
of what has been configured.
--
David Marchand
More information about the dev
mailing list