fjes: Fix unnecessary spinlock_irqsave
authorTaku Izumi <izumi.taku@jp.fujitsu.com>
Tue, 10 May 2016 01:28:49 +0000 (10:28 +0900)
committerDavid S. Miller <davem@davemloft.net>
Tue, 10 May 2016 02:05:51 +0000 (22:05 -0400)
commit-bd5a256 introduces a deadlock bug in fjes_change_mtu().
This spin_lock_irqsave() is obviously unnecessary.

This patch eliminates unnecessary spin_lock_irqsave() in
fjes_change_mtu()

Signed-off-by: Taku Izumi <izumi.taku@jp.fujitsu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/fjes/fjes_main.c

index f4e69261a3ce0cadc09532d255551f434a7c3656..86c331bb5eb38ffbb9833740fd9fd05ea3bf237c 100644 (file)
@@ -819,7 +819,6 @@ static int fjes_change_mtu(struct net_device *netdev, int new_mtu)
        netdev->mtu = new_mtu;
 
        if (running) {
-               spin_lock_irqsave(&hw->rx_status_lock, flags);
                for (epidx = 0; epidx < hw->max_epid; epidx++) {
                        if (epidx == hw->my_epid)
                                continue;