Wed, Feb 22, 2017 at 02:08:16AM CET, mah...@bandewar.net wrote: >From: Mahesh Bandewar <mahe...@google.com> > >Few cleanup patches that I have accumulated over some time now. > >(a) First two patches are basically to move the work-queue initialization > from every ndo_open / bond_open operation to once at the beginning while > port creation. Work-queue initialization is an unnecessary operation > for every 'ifup' operation. However we have some mode-specific work-queues > and mode can change anytime after port creation. So the second patch is > to ensure the correct work-handler is called based on the mode. > >(b) Third patch is simple and straightforward that removes hard-coded value > that was added into the initial commit and replaces it with the default > value configured. > >(c) The final patch in the series removes the unimplemented "port-moved" state > from the LACP state machine. This state is defined but never set so > removing from the state machine logic makes code little cleaner. > >Note: None of these patches are making any functional changes. > >Mahesh Bandewar (4):
Mahesh. I understand that you are still using bonding. What's stopping you from using team instead? Isn't about the time to start deprecate process of bonding? :O > bonding: restructure arp-monitor > bonding: initialize work-queues during creation of bond > bonding: remove hardcoded value > bonding: remove "port-moved" state that was never implemented > > drivers/net/bonding/bond_3ad.c | 11 +++-------- > drivers/net/bonding/bond_main.c | 42 ++++++++++++++++++++++++++++------------- > 2 files changed, 32 insertions(+), 21 deletions(-) > >-- >2.11.0.483.g087da7b7c-goog >
