Commit 5e20d55a authored Aug 09, 2017 by stephen hemminger Committed by David S. Miller Aug 11, 2017

netvsc: don't signal host twice if empty



When hv_pkt_iter_next() returns NULL, it has already called
hv_pkt_iter_close(). Calling it twice can lead to extra host signal.

Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent 6123c668

drivers/net/hyperv/netvsc.c

+0 −4

Original line number	Diff line number	Diff line
		@@ -1191,10 +1191,6 @@ int netvsc_poll(struct napi_struct *napi, int budget)
		nvchan->desc = hv_pkt_iter_next(channel, nvchan->desc);
		}

		/* if ring is empty, signal host */
		if (!nvchan->desc)
		hv_pkt_iter_close(channel);

		/* If send of pending receive completions suceeded
		* and did not exhaust NAPI budget this time
		* and not doing busy poll