On 2/17/2021 4:26 PM, Ferruh Yigit wrote:
From: Vadim Podovinnikov <[email protected]>In bond (LACP) we have several NICs (ports), when we have negotiation with peer about what port we prefer, we send information about what system we preferred in partner system name field. Peer also sends us what partner system name it prefer. When we receive a message from it we must compare its preferred system name with our system name, but not with our port mac address In my test I have several problems with that: 1. If master port (mac address same as system address) shuts down (I have two ports) I loose connection 2. If secondary port (mac address not same as system address) receives message before master port, my connection is not established. Fixes: 56cbc0817399 ("net/bonding: fix LACP negotiation") Cc: [email protected] Signed-off-by: Vadim Podovinnikov <[email protected]> --- Cc: [email protected] Cc: Declan Doherty <[email protected]> v3: Re-sent with rebase * Patch title updated, commit log updated with info shared in email * Sign-off updated with full name * Debug log slightly updated * Syntax slightly updated
This patch is waiting for review for a long time, if there is no objection I am planning to get it early in the release to give it a time to fix any possible issues.
