stmmac: fix memory barriers
authorPavel Machek <pavel@ucw.cz>
Sun, 18 Dec 2016 20:38:12 +0000 (21:38 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 19 Dec 2016 16:05:02 +0000 (11:05 -0500)
Fix up memory barriers in stmmac driver. They are meant to protect
against DMA engine, so smp_ variants are certainly wrong, and dma_
variants are preferable.

Signed-off-by: Pavel Machek <pavel@denx.de>
Tested-by: Niklas Cassel <niklas.cassel@axis.com>
Acked-by: Giuseppe Cavallaro <peppe.cavallaro@st.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/stmicro/stmmac/dwmac4_descs.c
drivers/net/ethernet/stmicro/stmmac/enh_desc.c
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c

index a340fc8bd0debf305ff2cb2d6b72b1925ec8b86e..8816515e1bbbc5c4a1fd5a443a478f2061d27c50 100644 (file)
@@ -334,7 +334,7 @@ static void dwmac4_rd_prepare_tx_desc(struct dma_desc *p, int is_fs, int len,
                 * descriptors for the same frame has to be set before, to
                 * avoid race condition.
                 */
-               wmb();
+               dma_wmb();
 
        p->des3 = cpu_to_le32(tdes3);
 }
@@ -377,7 +377,7 @@ static void dwmac4_rd_prepare_tso_tx_desc(struct dma_desc *p, int is_fs,
                 * descriptors for the same frame has to be set before, to
                 * avoid race condition.
                 */
-               wmb();
+               dma_wmb();
 
        p->des3 = cpu_to_le32(tdes3);
 }
index ce97e522566a8910e05707870d5c3f511a20ee3b..f0d86321dfe22b0d4455436326c93a8af513764a 100644 (file)
@@ -350,7 +350,7 @@ static void enh_desc_prepare_tx_desc(struct dma_desc *p, int is_fs, int len,
                 * descriptors for the same frame has to be set before, to
                 * avoid race condition.
                 */
-               wmb();
+               dma_wmb();
 
        p->des0 = cpu_to_le32(tdes0);
 }
index 3e405785b81c84e551e499af4fe85cf0c0983f40..bb40382e205deffd9a3312099723b72f84da18a5 100644 (file)
@@ -2125,7 +2125,7 @@ static netdev_tx_t stmmac_tso_xmit(struct sk_buff *skb, struct net_device *dev)
         * descriptor and then barrier is needed to make sure that
         * all is coherent before granting the DMA engine.
         */
-       smp_wmb();
+       dma_wmb();
 
        if (netif_msg_pktdata(priv)) {
                pr_info("%s: curr=%d dirty=%d f=%d, e=%d, f_p=%p, nfrags %d\n",
@@ -2338,7 +2338,7 @@ static netdev_tx_t stmmac_xmit(struct sk_buff *skb, struct net_device *dev)
                 * descriptor and then barrier is needed to make sure that
                 * all is coherent before granting the DMA engine.
                 */
-               smp_wmb();
+               dma_wmb();
        }
 
        netdev_sent_queue(dev, skb->len);
@@ -2443,14 +2443,14 @@ static inline void stmmac_rx_refill(struct stmmac_priv *priv)
                        netif_dbg(priv, rx_status, priv->dev,
                                  "refill entry #%d\n", entry);
                }
-               wmb();
+               dma_wmb();
 
                if (unlikely(priv->synopsys_id >= DWMAC_CORE_4_00))
                        priv->hw->desc->init_rx_desc(p, priv->use_riwt, 0, 0);
                else
                        priv->hw->desc->set_rx_owner(p);
 
-               wmb();
+               dma_wmb();
 
                entry = STMMAC_GET_ENTRY(entry, DMA_RX_SIZE);
        }