diff options
author | Stephen Hemminger <stephen@networkplumber.org> | 2018-07-27 16:43:21 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-07-29 15:57:26 -0400 |
commit | 3260155ac784ae9f4889c6d3e40b85f33e94d098 (patch) | |
tree | 648292baa821322d4db544de9fa9ddd1037b394e | |
parent | 4b09384aaa2a9b2ac09a584d7a9345cf003617f2 (diff) |
failover: change mtu has RTNL
When changing MTU, RTNL is held so use rtnl_dereference
instead of rcu_dereference.
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/net_failover.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/net_failover.c b/drivers/net/net_failover.c index d00d42c845b7..7ae1856d1f18 100644 --- a/drivers/net/net_failover.c +++ b/drivers/net/net_failover.c | |||
@@ -220,14 +220,14 @@ static int net_failover_change_mtu(struct net_device *dev, int new_mtu) | |||
220 | struct net_device *primary_dev, *standby_dev; | 220 | struct net_device *primary_dev, *standby_dev; |
221 | int ret = 0; | 221 | int ret = 0; |
222 | 222 | ||
223 | primary_dev = rcu_dereference(nfo_info->primary_dev); | 223 | primary_dev = rtnl_dereference(nfo_info->primary_dev); |
224 | if (primary_dev) { | 224 | if (primary_dev) { |
225 | ret = dev_set_mtu(primary_dev, new_mtu); | 225 | ret = dev_set_mtu(primary_dev, new_mtu); |
226 | if (ret) | 226 | if (ret) |
227 | return ret; | 227 | return ret; |
228 | } | 228 | } |
229 | 229 | ||
230 | standby_dev = rcu_dereference(nfo_info->standby_dev); | 230 | standby_dev = rtnl_dereference(nfo_info->standby_dev); |
231 | if (standby_dev) { | 231 | if (standby_dev) { |
232 | ret = dev_set_mtu(standby_dev, new_mtu); | 232 | ret = dev_set_mtu(standby_dev, new_mtu); |
233 | if (ret) { | 233 | if (ret) { |