IPoIB: Use dev_set_mtu() to change mtu
authorEli Cohen <eli@mellanox.co.il>
Tue, 15 Jul 2008 06:48:51 +0000 (23:48 -0700)
committerRoland Dreier <rolandd@cisco.com>
Tue, 15 Jul 2008 06:48:51 +0000 (23:48 -0700)
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>
drivers/infiniband/ulp/ipoib/ipoib_cm.c
drivers/infiniband/ulp/ipoib/ipoib_multicast.c

index b4269139135bbb4a740b17c115efa4ffcfab7c95..87f9f3ef3b2d7394dbc956d32c2b8956295ca0a1 100644 (file)
@@ -1458,7 +1458,7 @@ static ssize_t set_mode(struct device *d, struct device_attribute *attr,
                        if (priv->hca_caps & IB_DEVICE_UD_TSO)
                                dev->features |= NETIF_F_TSO;
                }
-               dev->mtu = min(priv->mcast_mtu, dev->mtu);
+               dev_set_mtu(dev, min(priv->mcast_mtu, dev->mtu));
                rtnl_unlock();
                ipoib_flush_paths(dev);
 
index be1ed38cdcfda02b94d400548699a808f15f0c34..1fcc9a898d81c8fefd4ceb852926221fb3261da8 100644 (file)
@@ -577,7 +577,7 @@ void ipoib_mcast_join_task(struct work_struct *work)
 
        if (!ipoib_cm_admin_enabled(dev)) {
                rtnl_lock();
-               dev->mtu = min(priv->mcast_mtu, priv->admin_mtu);
+               dev_set_mtu(dev, min(priv->mcast_mtu, priv->admin_mtu));
                rtnl_unlock();
        }