net/hyperv: Fix the stop/wake queue mechanism
authorHaiyang Zhang <haiyangz@microsoft.com>
Fri, 2 Dec 2011 19:56:25 +0000 (11:56 -0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sat, 10 Dec 2011 00:26:50 +0000 (16:26 -0800)
commit1d06825b0ede541f63b5577435abd2fc649a9b5e
treef3b72c945f340d1f6e81ec124a8fb352d9de747f
parent9d41c5bb07ad97b7777283d7922292f456ba4bfd
net/hyperv: Fix the stop/wake queue mechanism

The ring buffer is only used to pass meta data for outbound packets. The
actual payload is accessed by DMA from the host. So the stop/wake queue
mechanism based on counting and comparing number of pages sent v.s. number
of pages in the ring buffer is wrong. Also, there is a race condition in
the stop/wake queue calls, which can stop xmit queue forever.

The new stop/wake queue mechanism is based on the actual bytes used by
outbound packets in the ring buffer. The check for number of outstanding
sends after stop queue prevents the race condition that can cause wake
queue happening earlier than stop queue.

Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Reported-by: Long Li <longli@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/net/hyperv/netvsc.c
drivers/net/hyperv/netvsc_drv.c