summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRusty Russell <rusty@rustcorp.com.au>2008-05-02 21:50:43 -0500
committerRusty Russell <rusty@rustcorp.com.au>2008-05-02 21:50:43 +1000
commit5ef827526fc01820a7a80827802e9fad3f34f937 (patch)
treec41345c6de6ad5b4b4d7071aa46a9e27a773247c
parent886c35fbcf6fb2eee15687efc2d64d99b6ad9a4a (diff)
downloadlinux-5ef827526fc01820a7a80827802e9fad3f34f937.tar.gz
linux-5ef827526fc01820a7a80827802e9fad3f34f937.tar.bz2
linux-5ef827526fc01820a7a80827802e9fad3f34f937.zip
virtio: ignore corrupted virtqueues rather than spinning.
A corrupt virtqueue (caused by the other end screwing up) can have strange results such as a driver spinning: just bail when we try to get a buffer from a known-broken queue. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
-rw-r--r--drivers/virtio/virtio_ring.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c
index c2fa5c630813..937a49d6772c 100644
--- a/drivers/virtio/virtio_ring.c
+++ b/drivers/virtio/virtio_ring.c
@@ -184,6 +184,11 @@ static void *vring_get_buf(struct virtqueue *_vq, unsigned int *len)
START_USE(vq);
+ if (unlikely(vq->broken)) {
+ END_USE(vq);
+ return NULL;
+ }
+
if (!more_used(vq)) {
pr_debug("No more buffers in queue\n");
END_USE(vq);