Просмотр исходного кода

dwc_eth_qos: release descriptors outside netif_tx_lock

To prepare for using the CMA, we can not be in atomic context when
de-allocating DMA buffers.

The tx lock was needed only to protect the hw reset against the xmit
handler. Now we briefly grab the tx lock while stopping the queue to
make sure no thread is inside or will enter the xmit handler.

Signed-off-by: Lars Persson <larper@axis.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Lars Persson 9 лет назад
Родитель
Сommit
d4dc35f26e
1 измененных файлов с 6 добавлено и 4 удалено
  1. 6 4
      drivers/net/ethernet/synopsys/dwc_eth_qos.c

+ 6 - 4
drivers/net/ethernet/synopsys/dwc_eth_qos.c

@@ -1918,15 +1918,17 @@ static int dwceqos_stop(struct net_device *ndev)
 	phy_stop(lp->phy_dev);
 	phy_stop(lp->phy_dev);
 
 
 	tasklet_disable(&lp->tx_bdreclaim_tasklet);
 	tasklet_disable(&lp->tx_bdreclaim_tasklet);
-	netif_stop_queue(ndev);
 	napi_disable(&lp->napi);
 	napi_disable(&lp->napi);
 
 
-	dwceqos_drain_dma(lp);
+	/* Stop all tx before we drain the tx dma. */
+	netif_tx_lock_bh(lp->ndev);
+	netif_stop_queue(ndev);
+	netif_tx_unlock_bh(lp->ndev);
 
 
-	netif_tx_lock(lp->ndev);
+	dwceqos_drain_dma(lp);
 	dwceqos_reset_hw(lp);
 	dwceqos_reset_hw(lp);
+
 	dwceqos_descriptor_free(lp);
 	dwceqos_descriptor_free(lp);
-	netif_tx_unlock(lp->ndev);
 
 
 	return 0;
 	return 0;
 }
 }