diff options
author | nikolay@redhat.com <nikolay@redhat.com> | 2013-09-02 07:51:40 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-09-04 00:27:24 -0400 |
commit | c509316b5b33664b08b2a40d09534e0bd3c6b648 (patch) | |
tree | 3769e355a452894b9eaa4694f0486670c8adde64 /drivers/net/bonding/bond_sysfs.c | |
parent | ee8487c0e1aed52b534f9bf31d3934af4c50bf33 (diff) |
bonding: simplify bond_3ad_update_lacp_rate and use RTNL for sync
We can drop the use of bond->lock for mutual exclusion in
bond_3ad_update_lacp_rate and use RTNL in the sysfs store function
instead. This way we'll prevent races with mode change and interface
up/down as well as simplify update_lacp_rate by removing the check for
port->slave because it'll always be initialized (done while enslaving
with RTNL). This change will also help in the future removal of reader
bond->lock from bond_enslave.
Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding/bond_sysfs.c')
-rw-r--r-- | drivers/net/bonding/bond_sysfs.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/net/bonding/bond_sysfs.c b/drivers/net/bonding/bond_sysfs.c index 0f539de640dc..ce4677668e2c 100644 --- a/drivers/net/bonding/bond_sysfs.c +++ b/drivers/net/bonding/bond_sysfs.c | |||
@@ -852,8 +852,11 @@ static ssize_t bonding_store_lacp(struct device *d, | |||
852 | struct device_attribute *attr, | 852 | struct device_attribute *attr, |
853 | const char *buf, size_t count) | 853 | const char *buf, size_t count) |
854 | { | 854 | { |
855 | int new_value, ret = count; | ||
856 | struct bonding *bond = to_bond(d); | 855 | struct bonding *bond = to_bond(d); |
856 | int new_value, ret = count; | ||
857 | |||
858 | if (!rtnl_trylock()) | ||
859 | return restart_syscall(); | ||
857 | 860 | ||
858 | if (bond->dev->flags & IFF_UP) { | 861 | if (bond->dev->flags & IFF_UP) { |
859 | pr_err("%s: Unable to update LACP rate because interface is up.\n", | 862 | pr_err("%s: Unable to update LACP rate because interface is up.\n", |
@@ -883,6 +886,8 @@ static ssize_t bonding_store_lacp(struct device *d, | |||
883 | ret = -EINVAL; | 886 | ret = -EINVAL; |
884 | } | 887 | } |
885 | out: | 888 | out: |
889 | rtnl_unlock(); | ||
890 | |||
886 | return ret; | 891 | return ret; |
887 | } | 892 | } |
888 | static DEVICE_ATTR(lacp_rate, S_IRUGO | S_IWUSR, | 893 | static DEVICE_ATTR(lacp_rate, S_IRUGO | S_IWUSR, |