Re: [PATCH 0/3] net/bond: fix secondary process crash and related cleanup

[Stephen Hemminger <stephen@networkplumber.org>]

On Fri, 17 Apr 2026 09:51:34 -0700
Stephen Hemminger <stephen@networkplumber.org> wrote:

> Patch 1 fixes the 8023ad dedicated-queue APIs which ignored the return
> of bond_ethdev_mode_set() and left the enabled flag inconsistent on
> failure. Ordered first so the secondary guard added next does not
> leave shared state corrupted.
> 
> Patch 2 fixes the secondary process crash: a bonding port attached in
> a secondary crashes on the first Rx or Tx burst because the probe
> path never installs burst functions. Blackhole stubs are installed
> and mode changes from secondary are rejected. Fully sharing bonding
> state across processes is out of scope.
> 
> Patch 3 drops redundant %s/func from log call sites now that
> RTE_BOND_LOG supplies the prefix.
> 
> Stephen Hemminger (3):
>   net/bonding: restore dedicated queue state on mode set error
>   net/bonding: prevent crash on Rx/Tx from secondary process
>   net/bonding: remove redundant function names from log
> 
>  drivers/net/bonding/rte_eth_bond_8023ad.c | 19 ++++---
>  drivers/net/bonding/rte_eth_bond_api.c    |  4 +-
>  drivers/net/bonding/rte_eth_bond_pmd.c    | 66 ++++++++++++++++++-----
>  3 files changed, 66 insertions(+), 23 deletions(-)
> 

Would like review of this before adding to next-net