diff options
author | Michael S. Tsirkin <mst@redhat.com> | 2010-06-10 11:16:11 -0400 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2010-06-23 09:19:06 -0400 |
commit | 686d363786a53ed28ee875b84ef24e6d5126ef6f (patch) | |
tree | c97b5c5961df1f051e42a59dcb9ee1884d8f3086 /drivers/virtio | |
parent | 7e27d6e778cd87b6f2415515d7127eba53fe5d02 (diff) |
virtio: return ENOMEM on out of memory
add_buf returns ring size on out of memory,
this is not what devices expect.
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Amit Shah <amit.shah@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Cc: stable@kernel.org # .34.x
Diffstat (limited to 'drivers/virtio')
-rw-r--r-- | drivers/virtio/virtio_ring.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c index 1ca88908723b..afe7e21dd0ae 100644 --- a/drivers/virtio/virtio_ring.c +++ b/drivers/virtio/virtio_ring.c | |||
@@ -119,7 +119,7 @@ static int vring_add_indirect(struct vring_virtqueue *vq, | |||
119 | 119 | ||
120 | desc = kmalloc((out + in) * sizeof(struct vring_desc), gfp); | 120 | desc = kmalloc((out + in) * sizeof(struct vring_desc), gfp); |
121 | if (!desc) | 121 | if (!desc) |
122 | return vq->vring.num; | 122 | return -ENOMEM; |
123 | 123 | ||
124 | /* Transfer entries from the sg list into the indirect page */ | 124 | /* Transfer entries from the sg list into the indirect page */ |
125 | for (i = 0; i < out; i++) { | 125 | for (i = 0; i < out; i++) { |