Commit 82a7fc6d authored by Larry Finger's avatar Larry Finger

staging: rtl8192e: Fix kernel panics due to RX skb allocation failures

This driver uses RX skb's of O(2), thus it is possible for memory fragmentation
to prevent the allocation of a new one to replace a newly-received buffer.
When such a failure occurs, the kernel panics.

The fix is to drop an incoming packet whenever such an allocation fails. This
fix matches the one done in rtlwifi for other Realtek PCI devices.
Signed-off-by: default avatarLarry Finger <Larry.Finger@lwfinger.net>
parent b1d5ee70
...@@ -2550,12 +2550,18 @@ void rtl8192_rx_normal(struct net_device *dev) ...@@ -2550,12 +2550,18 @@ void rtl8192_rx_normal(struct net_device *dev)
rx_desc *pdesc = &priv->rx_ring[rx_queue_idx][priv->rx_idx[rx_queue_idx]]; rx_desc *pdesc = &priv->rx_ring[rx_queue_idx][priv->rx_idx[rx_queue_idx]];
struct sk_buff *skb = priv->rx_buf[rx_queue_idx][priv->rx_idx[rx_queue_idx]]; struct sk_buff *skb = priv->rx_buf[rx_queue_idx][priv->rx_idx[rx_queue_idx]];
if (pdesc->OWN){ if (pdesc->OWN) {
return; return;
} else { } else {
struct sk_buff *new_skb;
struct sk_buff *new_skb = NULL; if (!priv->ops->rx_query_status_descriptor(dev, &stats,
if (!priv->ops->rx_query_status_descriptor(dev, &stats, pdesc, skb)) pdesc, skb))
goto done;
new_skb = dev_alloc_skb(priv->rxbuffersize);
/* if allocation of new skb failed - drop current packet
* and reuse skb */
if (unlikely(!new_skb))
goto done; goto done;
pci_unmap_single(priv->pdev, pci_unmap_single(priv->pdev,
...@@ -2564,24 +2570,24 @@ void rtl8192_rx_normal(struct net_device *dev) ...@@ -2564,24 +2570,24 @@ void rtl8192_rx_normal(struct net_device *dev)
PCI_DMA_FROMDEVICE); PCI_DMA_FROMDEVICE);
skb_put(skb, pdesc->Length); skb_put(skb, pdesc->Length);
skb_reserve(skb, stats.RxDrvInfoSize + stats.RxBufShift); skb_reserve(skb, stats.RxDrvInfoSize +
stats.RxBufShift);
skb_trim(skb, skb->len - 4/*sCrcLng*/); skb_trim(skb, skb->len - 4/*sCrcLng*/);
rtllib_hdr = (struct rtllib_hdr_1addr *)skb->data; rtllib_hdr = (struct rtllib_hdr_1addr *)skb->data;
if (is_broadcast_ether_addr(rtllib_hdr->addr1)) { if (!is_broadcast_ether_addr(rtllib_hdr->addr1) &&
}else if (is_multicast_ether_addr(rtllib_hdr->addr1)){ !is_multicast_ether_addr(rtllib_hdr->addr1)) {
}else {
/* unicast packet */ /* unicast packet */
unicast_packet = true; unicast_packet = true;
} }
fc = le16_to_cpu(rtllib_hdr->frame_ctl); fc = le16_to_cpu(rtllib_hdr->frame_ctl);
type = WLAN_FC_GET_TYPE(fc); type = WLAN_FC_GET_TYPE(fc);
if (type == RTLLIB_FTYPE_MGMT) if (type == RTLLIB_FTYPE_MGMT)
{
bLedBlinking = false; bLedBlinking = false;
}
if (bLedBlinking) if (bLedBlinking)
if (priv->rtllib->LedControlHandler) if (priv->rtllib->LedControlHandler)
priv->rtllib->LedControlHandler(dev, LED_CTL_RX); priv->rtllib->LedControlHandler(dev,
LED_CTL_RX);
if (stats.bCRC) { if (stats.bCRC) {
if (type != RTLLIB_FTYPE_MGMT) if (type != RTLLIB_FTYPE_MGMT)
...@@ -2592,28 +2598,16 @@ void rtl8192_rx_normal(struct net_device *dev) ...@@ -2592,28 +2598,16 @@ void rtl8192_rx_normal(struct net_device *dev)
skb_len = skb->len; skb_len = skb->len;
if (1)
{
if (!rtllib_rx(priv->rtllib, skb, &stats)){ if (!rtllib_rx(priv->rtllib, skb, &stats)){
dev_kfree_skb_any(skb); dev_kfree_skb_any(skb);
} else { } else {
priv->stats.rxok++; priv->stats.rxok++;
if (unicast_packet) { if (unicast_packet)
priv->stats.rxbytesunicast += skb_len; priv->stats.rxbytesunicast += skb_len;
}
}
}else{
dev_kfree_skb_any(skb);
} }
new_skb = dev_alloc_skb(priv->rxbuffersize); skb = new_skb;
if (unlikely(!new_skb)) skb->dev = dev;
{
printk("==========>can't alloc skb for rx\n");
goto done;
}
skb=new_skb;
skb->dev = dev;
priv->rx_buf[rx_queue_idx][priv->rx_idx[rx_queue_idx]] = skb; priv->rx_buf[rx_queue_idx][priv->rx_idx[rx_queue_idx]] = skb;
*((dma_addr_t *) skb->cb) = pci_map_single(priv->pdev, skb_tail_pointer_rsl(skb), priv->rxbuffersize, PCI_DMA_FROMDEVICE); *((dma_addr_t *) skb->cb) = pci_map_single(priv->pdev, skb_tail_pointer_rsl(skb), priv->rxbuffersize, PCI_DMA_FROMDEVICE);
......
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