Commit a58c891a authored by Eric Dumazet's avatar Eric Dumazet Committed by David S. Miller

b44: GFP_DMA skb should not escape from driver

b44 chip has some hardware limitations, that need GFP_DMA bounce
buffers in some situations.

In order to not deplete DMA zone, we should keep allocated GFP_DMA skb
only for driver use. At rx time, we copy such skb to newly allocated
skb, reusing existing copybreak infrastructure.

On machines with low amount of memory, all skb meet the hardware limitation,
so no copy is needed. We detect this situation using a new device flag, set
to one if one GFP_DMA skb was ever allocated by b44_alloc_rx_skb().

Previously allocated skb, even outside from DMA zone will then be recycled,
to have minimal impact on DMA zone use.
Signed-off-by: default avatarEric Dumazet <dada1@cosmosbay.com>
Tested-by: default avatarIonut Leonte <ionut.leonte@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 1c5625cf
...@@ -679,6 +679,7 @@ static int b44_alloc_rx_skb(struct b44 *bp, int src_idx, u32 dest_idx_unmasked) ...@@ -679,6 +679,7 @@ static int b44_alloc_rx_skb(struct b44 *bp, int src_idx, u32 dest_idx_unmasked)
dev_kfree_skb_any(skb); dev_kfree_skb_any(skb);
return -ENOMEM; return -ENOMEM;
} }
bp->force_copybreak = 1;
} }
rh = (struct rx_header *) skb->data; rh = (struct rx_header *) skb->data;
...@@ -800,7 +801,7 @@ static int b44_rx(struct b44 *bp, int budget) ...@@ -800,7 +801,7 @@ static int b44_rx(struct b44 *bp, int budget)
/* Omit CRC. */ /* Omit CRC. */
len -= 4; len -= 4;
if (len > RX_COPY_THRESHOLD) { if (!bp->force_copybreak && len > RX_COPY_THRESHOLD) {
int skb_size; int skb_size;
skb_size = b44_alloc_rx_skb(bp, cons, bp->rx_prod); skb_size = b44_alloc_rx_skb(bp, cons, bp->rx_prod);
if (skb_size < 0) if (skb_size < 0)
...@@ -2152,6 +2153,7 @@ static int __devinit b44_init_one(struct ssb_device *sdev, ...@@ -2152,6 +2153,7 @@ static int __devinit b44_init_one(struct ssb_device *sdev,
bp = netdev_priv(dev); bp = netdev_priv(dev);
bp->sdev = sdev; bp->sdev = sdev;
bp->dev = dev; bp->dev = dev;
bp->force_copybreak = 0;
bp->msg_enable = netif_msg_init(b44_debug, B44_DEF_MSG_ENABLE); bp->msg_enable = netif_msg_init(b44_debug, B44_DEF_MSG_ENABLE);
......
...@@ -395,7 +395,7 @@ struct b44 { ...@@ -395,7 +395,7 @@ struct b44 {
u32 rx_pending; u32 rx_pending;
u32 tx_pending; u32 tx_pending;
u8 phy_addr; u8 phy_addr;
u8 force_copybreak;
struct mii_if_info mii_if; struct mii_if_info mii_if;
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment