Commit dd9d9f59 authored by Bryan Whitehead's avatar Bryan Whitehead Committed by David S. Miller

lan743x: Fix RX Kernel Panic

It has been noticed that running the speed test at
www.speedtest.net occasionally causes a kernel panic.

Investigation revealed that under this test RX buffer allocation
sometimes fails and returns NULL. But the lan743x driver did
not handle this case.

This patch fixes this issue by attempting to allocate a buffer
before sending the new rx packet to the OS. If the allocation
fails then the new rx packet is dropped and the existing buffer
is reused in the DMA ring.

Updates for v2:
    Additional 2 locations where allocation was not checked,
        has been changed to reuse existing buffer.

Fixes: 23f0703c ("lan743x: Add main source files for new lan743x driver")
Signed-off-by: default avatarBryan Whitehead <Bryan.Whitehead@microchip.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent fe543b2f
...@@ -1902,7 +1902,17 @@ static int lan743x_rx_next_index(struct lan743x_rx *rx, int index) ...@@ -1902,7 +1902,17 @@ static int lan743x_rx_next_index(struct lan743x_rx *rx, int index)
return ((++index) % rx->ring_size); return ((++index) % rx->ring_size);
} }
static int lan743x_rx_allocate_ring_element(struct lan743x_rx *rx, int index) static struct sk_buff *lan743x_rx_allocate_skb(struct lan743x_rx *rx)
{
int length = 0;
length = (LAN743X_MAX_FRAME_SIZE + ETH_HLEN + 4 + RX_HEAD_PADDING);
return __netdev_alloc_skb(rx->adapter->netdev,
length, GFP_ATOMIC | GFP_DMA);
}
static int lan743x_rx_init_ring_element(struct lan743x_rx *rx, int index,
struct sk_buff *skb)
{ {
struct lan743x_rx_buffer_info *buffer_info; struct lan743x_rx_buffer_info *buffer_info;
struct lan743x_rx_descriptor *descriptor; struct lan743x_rx_descriptor *descriptor;
...@@ -1911,9 +1921,7 @@ static int lan743x_rx_allocate_ring_element(struct lan743x_rx *rx, int index) ...@@ -1911,9 +1921,7 @@ static int lan743x_rx_allocate_ring_element(struct lan743x_rx *rx, int index)
length = (LAN743X_MAX_FRAME_SIZE + ETH_HLEN + 4 + RX_HEAD_PADDING); length = (LAN743X_MAX_FRAME_SIZE + ETH_HLEN + 4 + RX_HEAD_PADDING);
descriptor = &rx->ring_cpu_ptr[index]; descriptor = &rx->ring_cpu_ptr[index];
buffer_info = &rx->buffer_info[index]; buffer_info = &rx->buffer_info[index];
buffer_info->skb = __netdev_alloc_skb(rx->adapter->netdev, buffer_info->skb = skb;
length,
GFP_ATOMIC | GFP_DMA);
if (!(buffer_info->skb)) if (!(buffer_info->skb))
return -ENOMEM; return -ENOMEM;
buffer_info->dma_ptr = dma_map_single(&rx->adapter->pdev->dev, buffer_info->dma_ptr = dma_map_single(&rx->adapter->pdev->dev,
...@@ -2060,8 +2068,19 @@ static int lan743x_rx_process_packet(struct lan743x_rx *rx) ...@@ -2060,8 +2068,19 @@ static int lan743x_rx_process_packet(struct lan743x_rx *rx)
/* packet is available */ /* packet is available */
if (first_index == last_index) { if (first_index == last_index) {
/* single buffer packet */ /* single buffer packet */
struct sk_buff *new_skb = NULL;
int packet_length; int packet_length;
new_skb = lan743x_rx_allocate_skb(rx);
if (!new_skb) {
/* failed to allocate next skb.
* Memory is very low.
* Drop this packet and reuse buffer.
*/
lan743x_rx_reuse_ring_element(rx, first_index);
goto process_extension;
}
buffer_info = &rx->buffer_info[first_index]; buffer_info = &rx->buffer_info[first_index];
skb = buffer_info->skb; skb = buffer_info->skb;
descriptor = &rx->ring_cpu_ptr[first_index]; descriptor = &rx->ring_cpu_ptr[first_index];
...@@ -2081,7 +2100,7 @@ static int lan743x_rx_process_packet(struct lan743x_rx *rx) ...@@ -2081,7 +2100,7 @@ static int lan743x_rx_process_packet(struct lan743x_rx *rx)
skb_put(skb, packet_length - 4); skb_put(skb, packet_length - 4);
skb->protocol = eth_type_trans(skb, skb->protocol = eth_type_trans(skb,
rx->adapter->netdev); rx->adapter->netdev);
lan743x_rx_allocate_ring_element(rx, first_index); lan743x_rx_init_ring_element(rx, first_index, new_skb);
} else { } else {
int index = first_index; int index = first_index;
...@@ -2094,26 +2113,23 @@ static int lan743x_rx_process_packet(struct lan743x_rx *rx) ...@@ -2094,26 +2113,23 @@ static int lan743x_rx_process_packet(struct lan743x_rx *rx)
if (first_index <= last_index) { if (first_index <= last_index) {
while ((index >= first_index) && while ((index >= first_index) &&
(index <= last_index)) { (index <= last_index)) {
lan743x_rx_release_ring_element(rx, lan743x_rx_reuse_ring_element(rx,
index); index);
lan743x_rx_allocate_ring_element(rx,
index);
index = lan743x_rx_next_index(rx, index = lan743x_rx_next_index(rx,
index); index);
} }
} else { } else {
while ((index >= first_index) || while ((index >= first_index) ||
(index <= last_index)) { (index <= last_index)) {
lan743x_rx_release_ring_element(rx, lan743x_rx_reuse_ring_element(rx,
index); index);
lan743x_rx_allocate_ring_element(rx,
index);
index = lan743x_rx_next_index(rx, index = lan743x_rx_next_index(rx,
index); index);
} }
} }
} }
process_extension:
if (extension_index >= 0) { if (extension_index >= 0) {
descriptor = &rx->ring_cpu_ptr[extension_index]; descriptor = &rx->ring_cpu_ptr[extension_index];
buffer_info = &rx->buffer_info[extension_index]; buffer_info = &rx->buffer_info[extension_index];
...@@ -2290,7 +2306,9 @@ static int lan743x_rx_ring_init(struct lan743x_rx *rx) ...@@ -2290,7 +2306,9 @@ static int lan743x_rx_ring_init(struct lan743x_rx *rx)
rx->last_head = 0; rx->last_head = 0;
for (index = 0; index < rx->ring_size; index++) { for (index = 0; index < rx->ring_size; index++) {
ret = lan743x_rx_allocate_ring_element(rx, index); struct sk_buff *new_skb = lan743x_rx_allocate_skb(rx);
ret = lan743x_rx_init_ring_element(rx, index, new_skb);
if (ret) if (ret)
goto cleanup; goto cleanup;
} }
......
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