diff mbox series

[v4,8/8] net/netvsc: avoid possible live lock

Message ID 20200331171404.23596-9-stephen@networkplumber.org (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers show
Series net/netvsc: bug fixes | expand


Context Check Description
ci/travis-robot success Travis build: passed
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Stephen Hemminger March 31, 2020, 5:14 p.m. UTC
Since the ring buffer with host is shared for both transmit
completions and receive packets, it is possible that transmitter
could get starved if receive ring gets full.

Better to process all outstanding events which frees up transmit
buffer slots, even if means dropping some packets.

Fixes: 7e6c82430702 ("net/netvsc: avoid over filling Rx descriptor ring")
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
 drivers/net/netvsc/hn_rxtx.c | 3 ---
 1 file changed, 3 deletions(-)
diff mbox series


diff --git a/drivers/net/netvsc/hn_rxtx.c b/drivers/net/netvsc/hn_rxtx.c
index cbdfcc628b75..19f00a05285f 100644
--- a/drivers/net/netvsc/hn_rxtx.c
+++ b/drivers/net/netvsc/hn_rxtx.c
@@ -1032,9 +1032,6 @@  uint32_t hn_process_events(struct hn_data *hv, uint16_t queue_id,
 		if (tx_limit && tx_done >= tx_limit)
-		if (rxq->rx_ring && rte_ring_full(rxq->rx_ring))
-			break;
 	if (bytes_read > 0)