From: Pavel Machek <pa...@ucw.cz> Date: Wed, 7 Dec 2016 22:37:57 +0100
> diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > index 982c952..7415bc2 100644 > --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c > @@ -1308,7 +1308,7 @@ static void stmmac_tx_clean(struct stmmac_priv *priv) > unsigned int bytes_compl = 0, pkts_compl = 0; > unsigned int entry = priv->dirty_tx; > > - spin_lock(&priv->tx_lock); > + netif_tx_lock_bh(priv->dev); > > priv->xstats.tx_clean++; > stmmac_tx_clean() runs from either the timer or the NAPI poll handler, both execute from software interrupts, therefore _bh() should be unnecessary.
