Commit 7c0b6bce authored by Luuk Paulussen's avatar Luuk Paulussen Committed by Greg Kroah-Hartman

staging: octeon: Fix logic for waking octeon ethernet tx queue.

Only wake tx queue when driver queue is back within bounds.

The logic here was just reenabling the queue when any buffers had been
freed.  the queue was stopped whenever the length exceeded 1000
(MAX_OUT_QUEUE_DEPTH), but then was essentially immediately started again.
On a congested link, the queue length would just keep increasing up to
around 8000 (for average size packets), at which point the hardware would
start refusing the packets and they would begin to be dropped.
This prevented the qdisc layer from effectively managing and prioritising
packets, as essentially all packets were being allowed into the driver
queue and then were being dropped by the hardware.

This change only restarts the queue if the length is less than 1000
(MAX_OUT_QUEUE_DEPTH).
Reviewed-by: default avatarKyeong Yoo <kyeong.yoo@alliedtelesis.co.nz>
Reviewed-by: default avatarChris Packham <chris.packham@alliedtelesis.co.nz>
Reviewed-by: default avatarRichard Laing <richard.laing@alliedtelesis.co.nz>
Signed-off-by: default avatarLuuk Paulussen <luuk.paulussen@alliedtelesis.co.nz>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 5a89a875
...@@ -126,7 +126,7 @@ static void cvm_oct_free_tx_skbs(struct net_device *dev) ...@@ -126,7 +126,7 @@ static void cvm_oct_free_tx_skbs(struct net_device *dev)
} }
total_remaining += skb_queue_len(&priv->tx_free_list[qos]); total_remaining += skb_queue_len(&priv->tx_free_list[qos]);
} }
if (total_freed >= 0 && netif_queue_stopped(dev)) if (total_remaining < MAX_OUT_QUEUE_DEPTH && netif_queue_stopped(dev))
netif_wake_queue(dev); netif_wake_queue(dev);
if (total_remaining) if (total_remaining)
cvm_oct_kick_tx_poll_watchdog(); cvm_oct_kick_tx_poll_watchdog();
......
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