[PATCH 3/8] net/bonding: skip unavailable members in device info
From: Stephen Hemminger <stephen@networkplumber.org>
Date: 2026-08-30 20:27:05
Subsystem:
bonding driver, networking drivers, the rest · Maintainers:
Jay Vosburgh, Andrew Lunn, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds
bond_ethdev_info() queries every member to derive the maximum number of
Rx and Tx queues the bonding device can support. A single failing
member aborted the whole call.
Skip members that cannot be queried and derive the queue limits from
the rest.
A bonding device with no members at all keeps reporting UINT16_MAX, as
it did before: no member has constrained it yet.
Fixes: acfb51e2fe96 ("net/bonding: fix number of bonding Tx/Rx queues")
Cc: stable@dpdk.org
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
drivers/net/bonding/rte_eth_bond_pmd.c | 20 ++++++++++++++++----
1 file changed, 16 insertions(+), 4 deletions(-)
diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c b/drivers/net/bonding/rte_eth_bond_pmd.c
index 6235c07679..7579b97b06 100644
--- a/drivers/net/bonding/rte_eth_bond_pmd.c
+++ b/drivers/net/bonding/rte_eth_bond_pmd.c@@ -2360,28 +2360,40 @@ bond_ethdev_info(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info) /* Max number of tx/rx queues that the bonding device can support is the * minimum values of the bonding members, as all members must be capable * of supporting the same number of tx/rx queues. + * + * A member may be unqueryable here: it can be removed concurrently, or + * simply not be probed in this process. Skip those, but do not report + * the UINT16_MAX default if no member could be queried at all, since + * that would let any queue count pass rte_eth_dev_configure(). */ if (internals->member_count > 0) { struct rte_eth_dev_info member_info; + uint16_t queried = 0; uint16_t idx; for (idx = 0; idx < internals->member_count; idx++) { member = internals->members[idx]; ret = rte_eth_dev_info_get(member.port_id, &member_info); if (ret != 0) { - RTE_BOND_LOG(ERR, - "Error getting device (port %u) info: %s", + RTE_BOND_LOG(WARNING, + "Skipping device (port %u) info: %s", member.port_id, strerror(-ret)); - - return ret; + continue; } + queried++; + if (member_info.max_rx_queues < max_nb_rx_queues) max_nb_rx_queues = member_info.max_rx_queues; if (member_info.max_tx_queues < max_nb_tx_queues) max_nb_tx_queues = member_info.max_tx_queues; } + + if (queried == 0) { + RTE_BOND_LOG(ERR, "No member device info available"); + return -ENODEV; + } } dev_info->max_rx_queues = max_nb_rx_queues;
--
2.53.0