bnxt_en: Add bp->rx_dir field for rx buffer DMA direction.
authorMichael Chan <michael.chan@broadcom.com>
Mon, 6 Feb 2017 21:55:34 +0000 (16:55 -0500)
committerDavid S. Miller <davem@davemloft.net>
Tue, 7 Feb 2017 18:30:58 +0000 (13:30 -0500)
When driver is running in XDP mode, rx buffers are DMA mapped as
DMA_BIDIRECTIONAL.  Add a field so the code will map/unmap rx buffers
according to this field.

Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnxt/bnxt.c
drivers/net/ethernet/broadcom/bnxt/bnxt.h

index 43eb52ced2cad1903e40f2d80916fbdf16594fce..ee2cada76f0f46ce76a4483157edd2dc66aafa0a 100644 (file)
@@ -584,7 +584,7 @@ static inline u8 *__bnxt_alloc_rx_data(struct bnxt *bp, dma_addr_t *mapping,
                return NULL;
 
        *mapping = dma_map_single(&pdev->dev, data + BNXT_RX_DMA_OFFSET,
-                                 bp->rx_buf_use_size, PCI_DMA_FROMDEVICE);
+                                 bp->rx_buf_use_size, bp->rx_dir);
 
        if (dma_mapping_error(&pdev->dev, *mapping)) {
                kfree(data);
@@ -772,7 +772,7 @@ static struct sk_buff *bnxt_rx_skb(struct bnxt *bp,
 
        skb = build_skb(data, 0);
        dma_unmap_single(&bp->pdev->dev, dma_addr, bp->rx_buf_use_size,
-                        PCI_DMA_FROMDEVICE);
+                        bp->rx_dir);
        if (!skb) {
                kfree(data);
                return NULL;
@@ -878,15 +878,14 @@ static inline struct sk_buff *bnxt_copy_skb(struct bnxt_napi *bnapi, u8 *data,
        if (!skb)
                return NULL;
 
-       dma_sync_single_for_cpu(&pdev->dev, mapping,
-                               bp->rx_copy_thresh, PCI_DMA_FROMDEVICE);
+       dma_sync_single_for_cpu(&pdev->dev, mapping, bp->rx_copy_thresh,
+                               bp->rx_dir);
 
        memcpy(skb->data - NET_IP_ALIGN, data - NET_IP_ALIGN,
               len + NET_IP_ALIGN);
 
-       dma_sync_single_for_device(&pdev->dev, mapping,
-                                  bp->rx_copy_thresh,
-                                  PCI_DMA_FROMDEVICE);
+       dma_sync_single_for_device(&pdev->dev, mapping, bp->rx_copy_thresh,
+                                  bp->rx_dir);
 
        skb_put(skb, len);
        return skb;
@@ -1261,7 +1260,7 @@ static inline struct sk_buff *bnxt_tpa_end(struct bnxt *bp,
 
                skb = build_skb(data, 0);
                dma_unmap_single(&bp->pdev->dev, mapping, bp->rx_buf_use_size,
-                                PCI_DMA_FROMDEVICE);
+                                bp->rx_dir);
 
                if (!skb) {
                        kfree(data);
@@ -1878,11 +1877,9 @@ static void bnxt_free_rx_skbs(struct bnxt *bp)
                                if (!data)
                                        continue;
 
-                               dma_unmap_single(
-                                       &pdev->dev,
-                                       tpa_info->mapping,
-                                       bp->rx_buf_use_size,
-                                       PCI_DMA_FROMDEVICE);
+                               dma_unmap_single(&pdev->dev, tpa_info->mapping,
+                                                bp->rx_buf_use_size,
+                                                bp->rx_dir);
 
                                tpa_info->data = NULL;
 
@@ -1898,8 +1895,7 @@ static void bnxt_free_rx_skbs(struct bnxt *bp)
                                continue;
 
                        dma_unmap_single(&pdev->dev, rx_buf->mapping,
-                                        bp->rx_buf_use_size,
-                                        PCI_DMA_FROMDEVICE);
+                                        bp->rx_buf_use_size, bp->rx_dir);
 
                        rx_buf->data = NULL;
 
@@ -2563,6 +2559,7 @@ void bnxt_set_ring_params(struct bnxt *bp)
 
 static int bnxt_set_rx_skb_mode(struct bnxt *bp)
 {
+       bp->rx_dir = DMA_FROM_DEVICE;
        bp->rx_skb_func = bnxt_rx_skb;
        return 0;
 }
index b4a04a3945e993f1dc9f6138968f3b15ec7f05f5..353b6d179ed21d9de6de710bdba66478916b278c 100644 (file)
@@ -997,6 +997,7 @@ struct bnxt {
 
        u32                     rx_buf_size;
        u32                     rx_buf_use_size;        /* useable size */
+       enum dma_data_direction rx_dir;
        u32                     rx_ring_size;
        u32                     rx_agg_ring_size;
        u32                     rx_copy_thresh;