diff options
| author | Jay Vosburgh <fubar@us.ibm.com> | 2010-07-21 12:14:48 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-07-22 14:14:47 -0700 |
| commit | 03dc2f4c525afb9488edb687c2e1f7057d59b40e (patch) | |
| tree | 59702784f96f5a7427e4053c65845aad9a2d1d68 /net/caif | |
| parent | f35188faa0fbabefac476536994f4b6f3677380f (diff) | |
| download | op-kernel-dev-03dc2f4c525afb9488edb687c2e1f7057d59b40e.zip op-kernel-dev-03dc2f4c525afb9488edb687c2e1f7057d59b40e.tar.gz | |
bonding: don't lock when copying/clearing VLAN list on slave
When copying VLAN information to or removing from a slave
during slave addition or removal, the bonding code currently holds
the bond->lock for write to prevent concurrent modification of the
vlan_list / vlgrp.
This is unnecessary, as all of these operations occur under
RTNL. Holding the bond->lock also caused might_sleep issues for
some drivers' ndo_vlan_* functions. This patch removes the extra
locking.
Problem reported by Michael Chan <mchan@broadcom.com>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Cc: Michael Chan <mchan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/caif')
0 files changed, 0 insertions, 0 deletions
