diff options
author | Eli Cohen <eli@mellanox.co.il> | 2008-07-15 02:48:51 -0400 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2008-07-15 02:48:51 -0400 |
commit | bd3606715effbf37df986548c43bbed0842b49d5 (patch) | |
tree | 08f2a13d486a75b42244beca90ae78c0045c0fc1 | |
parent | c8c2afe360b7366f586f6bece1109a72ea334876 (diff) |
IPoIB: Use dev_set_mtu() to change mtu
When the driver sets the MTU of the net device outside of its
change_mtu method, it should make use of dev_set_mtu() instead of
directly setting the mtu field of struct netdevice. Otherwise
functions registered to be called upon MTU change will not get called
(this is done through call_netdevice_notifiers() in dev_set_mtu()).
Signed-off-by: Eli Cohen <eli@mellanox.co.il>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
-rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib_cm.c | 2 | ||||
-rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib_multicast.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_cm.c b/drivers/infiniband/ulp/ipoib/ipoib_cm.c index b4269139135b..87f9f3ef3b2d 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_cm.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_cm.c | |||
@@ -1458,7 +1458,7 @@ static ssize_t set_mode(struct device *d, struct device_attribute *attr, | |||
1458 | if (priv->hca_caps & IB_DEVICE_UD_TSO) | 1458 | if (priv->hca_caps & IB_DEVICE_UD_TSO) |
1459 | dev->features |= NETIF_F_TSO; | 1459 | dev->features |= NETIF_F_TSO; |
1460 | } | 1460 | } |
1461 | dev->mtu = min(priv->mcast_mtu, dev->mtu); | 1461 | dev_set_mtu(dev, min(priv->mcast_mtu, dev->mtu)); |
1462 | rtnl_unlock(); | 1462 | rtnl_unlock(); |
1463 | ipoib_flush_paths(dev); | 1463 | ipoib_flush_paths(dev); |
1464 | 1464 | ||
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c index be1ed38cdcfd..1fcc9a898d81 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c | |||
@@ -577,7 +577,7 @@ void ipoib_mcast_join_task(struct work_struct *work) | |||
577 | 577 | ||
578 | if (!ipoib_cm_admin_enabled(dev)) { | 578 | if (!ipoib_cm_admin_enabled(dev)) { |
579 | rtnl_lock(); | 579 | rtnl_lock(); |
580 | dev->mtu = min(priv->mcast_mtu, priv->admin_mtu); | 580 | dev_set_mtu(dev, min(priv->mcast_mtu, priv->admin_mtu)); |
581 | rtnl_unlock(); | 581 | rtnl_unlock(); |
582 | } | 582 | } |
583 | 583 | ||