Re: [PATCH net v3] bonding: 3ad: select a port when the TX array is empty

[Paolo Abeni <pabeni@redhat.com>]

On 10/12/26 11:52, Hangbin Liu wrote:
> On Mon, Oct 12, 2026 at 03:20:09PM +0800, luyun wrote:
>> On cost: the fallback only runs when the array is empty, so the normal
>> transmit path is untouched. Inside the window it is a lockless RCU list
>> walk with no new state; the second pass and the broadcast branch are
>> there to keep the array path's hash mapping and broadcast_neighbor
>> semantics.
>>
>> I would also like Jay's feedback on the preferred direction. If a
>> control-path solution is preferred, I am open to working on separating
>> array preparation from port enablement, as long as it does not change
>> LACP convergence behavior.
> 
> +1
I guess we are not getting Jay's feedback anytime soon.

I think a control-plane solution is highly preferable, even if it's
going to be a more significant work. I have just skimmed over the code
and it feels like removing the rtnl lock requirement for
bond_update_slave_arr() is doable - i.e. moving ->all_slaves and
->usable_slaves under a new bond specific mutex).

Thanks,

Paolo