Re: [PATCH next 0/5] bonding: winter cleanup

[David Miller]

From: Mahesh Bandewar 
Date: Wed,  8 Mar 2017 10:55:45 -0800

> 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.
> 
> (d) Reduce scope of some global variables to local.
> 
> Note: None of these patches are making any functional changes.

This series looks fine to me, applied to net-next, thanks!

Re: [PATCH next 0/5] bonding: winter cleanup

[Jay Vosburgh]

Mahesh Bandewar  wrote:

>From: Mahesh Bandewar 
>
>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.
>
>(d) Reduce scope of some global variables to local.

For all patches in the series:

Signed-off-by: Jay Vosburgh 

>Note: None of these patches are making any functional changes.
>
>Mahesh Bandewar (5):
>  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
>  bonding: reduce scope of some global variables
>
> drivers/net/bonding/bond_3ad.c  | 11 +++--
> drivers/net/bonding/bond_main.c | 53 ++---
> 2 files changed, 37 insertions(+), 27 deletions(-)
>
>-- 
>2.12.0.246.ga2ecc84866-goog

---
-Jay Vosburgh, jay.vosbu...@canonical.com

[PATCH next 0/5] bonding: winter cleanup

[Mahesh Bandewar]

From: Mahesh Bandewar 

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.

(d) Reduce scope of some global variables to local.

Note: None of these patches are making any functional changes.

Mahesh Bandewar (5):
  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
  bonding: reduce scope of some global variables

 drivers/net/bonding/bond_3ad.c  | 11 +++--
 drivers/net/bonding/bond_main.c | 53 ++---
 2 files changed, 37 insertions(+), 27 deletions(-)

-- 
2.12.0.246.ga2ecc84866-goog