diff options
author | Paul Durrant <Paul.Durrant@citrix.com> | 2014-01-08 12:41:58 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-01-09 23:05:46 -0500 |
commit | 11b57f90257c1d6a91cee720151b69e0c2020cf6 (patch) | |
tree | 6da4cdb6bbaf71ab38ad9a133e2391aa4910f32a /drivers/net/ifb.c | |
parent | 451cd14e082c631aeaec37bfa872d3fa594d1845 (diff) | |
download | blackbird-op-linux-11b57f90257c1d6a91cee720151b69e0c2020cf6.tar.gz blackbird-op-linux-11b57f90257c1d6a91cee720151b69e0c2020cf6.zip |
xen-netback: stop vif thread spinning if frontend is unresponsive
The recent patch to improve guest receive side flow control (ca2f09f2) had a
slight flaw in the wait condition for the vif thread in that any remaining
skbs in the guest receive side netback internal queue would prevent the
thread from sleeping. An unresponsive frontend can lead to a permanently
non-empty internal queue and thus the thread will spin. In this case the
thread should really sleep until the frontend becomes responsive again.
This patch adds an extra flag to the vif which is set if the shared ring
is full and cleared when skbs are drained into the shared ring. Thus,
if the thread runs, finds the shared ring full and can make no progress the
flag remains set. If the flag remains set then the thread will sleep,
regardless of a non-empty queue, until the next event from the frontend.
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
Cc: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: David Vrabel <david.vrabel@citrix.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ifb.c')
0 files changed, 0 insertions, 0 deletions