Skip to content

bonding: couple of bug fixes

[email protected] netdev 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest2 patches 2 repliesIn review
  2. v12 patches 5 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2bonding: reset master mac on first enslave failuredrivers/net/bonding/bond_main.c+4 -00 replies
2/2bonding: fix igmp_retrans type and two related racesdrivers/net/bonding/bond_main.c drivers/net/bonding/bonding.h+12 -50 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [PATCH v2 0/2] bonding: couple of bug fixes

From: Nikolay Aleksandrov <[email protected]>

Hello,
Patch 01 resets the master's mac if the first enslave fails and the slave's mac was set to the master's prior.
Patch 02 fixes a type bug of igmp_retrans (which is the counter of igmp_resend param and was silently not working if the value was >127) so it can go up to 255 (as per documentation). It also fixes two tricky race conditions which were hidden because of the previous bug.

V2: Updated patch 02 commit message as per Jay's comment

Best regards,
Nikolay Aleksandrov

Nikolay Aleksandrov (2):
bonding: reset master mac on first enslave failure
bonding: fix igmp_retrans type and two related races

 drivers/net/bonding/bond_main.c | 19 +++++++++++++++----
 drivers/net/bonding/bonding.h   |  2 +-
 2 files changed, 16 insertions(+), 5 deletions(-)
Signature · 1 line
1.8.1.4