diff options
| author | Jay Vosburgh <fubar@us.ibm.com> | 2010-07-21 14:14:48 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-07-22 23:14:47 +0200 |
| commit | 03dc2f4c525afb9488edb687c2e1f7057d59b40e (patch) | |
| tree | 59702784f96f5a7427e4053c65845aad9a2d1d68 /net/caif/cfrfml.c | |
| parent | bonding: change test for presence of VLANs (diff) | |
| download | linux-03dc2f4c525afb9488edb687c2e1f7057d59b40e.tar.xz linux-03dc2f4c525afb9488edb687c2e1f7057d59b40e.zip | |
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/cfrfml.c')
0 files changed, 0 insertions, 0 deletions