Commit 5da63cc4 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont Committed by David S. Miller

f_phonet: lock-less MTU change

With the current driver, the MTU is purely indicative, so there is no
need to synchronize with the receive path.
Signed-off-by: default avatarRémi Denis-Courmont <remi.denis-courmont@nokia.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent aa11d958
...@@ -261,20 +261,10 @@ out: ...@@ -261,20 +261,10 @@ out:
static int pn_net_mtu(struct net_device *dev, int new_mtu) static int pn_net_mtu(struct net_device *dev, int new_mtu)
{ {
struct phonet_port *port = netdev_priv(dev);
unsigned long flags;
int err = -EBUSY;
if ((new_mtu < PHONET_MIN_MTU) || (new_mtu > PHONET_MAX_MTU)) if ((new_mtu < PHONET_MIN_MTU) || (new_mtu > PHONET_MAX_MTU))
return -EINVAL; return -EINVAL;
spin_lock_irqsave(&port->lock, flags);
if (!netif_carrier_ok(dev)) {
dev->mtu = new_mtu; dev->mtu = new_mtu;
err = 0; return 0;
}
spin_unlock_irqrestore(&port->lock, flags);
return err;
} }
static const struct net_device_ops pn_netdev_ops = { static const struct net_device_ops pn_netdev_ops = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment