net: mvneta: fix mtu change on port without link
authorYelena Krivosheev <yelena@marvell.com>
Wed, 18 Jul 2018 16:10:51 +0000 (18:10 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 29 Jul 2018 05:12:55 +0000 (22:12 -0700)
It is incorrect to enable TX/RX queues (call by mvneta_port_up()) for
port without link. Indeed MTU change for interface without link causes TX
queues to stuck.

Fixes: c5aff18204da ("net: mvneta: driver for Marvell Armada 370/XP
network unit")
Signed-off-by: Yelena Krivosheev <yelena@marvell.com>
[gregory.clement: adding Fixes tags and rewording commit log]
Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/marvell/mvneta.c

index ec84db47d82df2bafa5872f5a0ce7653ebd2b703..3d1f4335b0d341555c017310b63125f6c46236cb 100644 (file)
@@ -3196,7 +3196,6 @@ static int mvneta_change_mtu(struct net_device *dev, int mtu)
 
        on_each_cpu(mvneta_percpu_enable, pp, true);
        mvneta_start_dev(pp);
-       mvneta_port_up(pp);
 
        netdev_update_features(dev);