Skip to content

bonding: fix primary problem for bonding

Ding Tianhong netdev 1 version NAK

Status from this message: Nacked-by: Veaceslav Falico <[email protected]>

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 1 replyNAK

Patches in v1

#Subject and filesSizeRepliesApplied
1/3bonding: update the primary slave when changing slave's namedrivers/net/bonding/bond_main.c+22 -20 replies
2/3bonding: clean the primary slave if there is no slave matching new primarydrivers/net/bonding/bond_options.c+6 -00 replies
3/3bonding: update bonding.txt for primary descriptionDocumentation/networking/bonding.txt+2 -10 replies

Reviews trailers per patch

None of the 3 patches has a Reviewed-by or Acked-by.

PatchVeaceslav Falico
SeriesNacked-by
1/3
2/3
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH net-next v2.1 0/3] bonding: fix primary problem for bonding

If the slave's name changed, and the bond params primary is exist, the bond should deal with the situation in two ways:

1) If the slave was the primary slave yet, clean the primary slave and reselect active slave.
2) If the slave's new name is as same as bond primary, set the slave as primary slave and reselect active slave.

If the new primary is not matching any slave in the bond, the bond should record it to params, clean the primary slave and select a new active slave.

Update bonding.txt for primary description.

v2.1 Because there are too many indentions and useless verification, so rewrite
the logic for updating the primary slave.
Modify some comments for to clean the typos.

Ding Tianhong (3):
bonding: update the primary slave when changing slave's name
bonding: clean the primary slave if there is no slave matching new
primary
bonding: update bonding.txt for primary description.

 Documentation/networking/bonding.txt |  3 ++-
 drivers/net/bonding/bond_main.c      | 24 ++++++++++++++++++++++--
 drivers/net/bonding/bond_options.c   |  6 ++++++
 3 files changed, 30 insertions(+), 3 deletions(-)
Signature · 1 line
1.8.0