[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 4/7] virtio-net: handle virtio_net_receive() errors
From: |
Greg Kurz |
Subject: |
[Qemu-devel] [PATCH 4/7] virtio-net: handle virtio_net_receive() errors |
Date: |
Wed, 21 Sep 2016 15:14:25 +0200 |
User-agent: |
StGit/0.17.1-dirty |
All these errors are caused by a buggy guest: let's switch the device to
the broken state instead of terminating QEMU.
Signed-off-by: Greg Kurz <address@hidden>
---
hw/net/virtio-net.c | 25 +++++++++++++------------
1 file changed, 13 insertions(+), 12 deletions(-)
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index 68a448acd81b..8f59402306eb 100644
--- a/hw/net/virtio-net.c
+++ b/hw/net/virtio-net.c
@@ -1122,21 +1122,22 @@ static ssize_t virtio_net_receive(NetClientState *nc,
const uint8_t *buf, size_t
elem = virtqueue_pop(q->rx_vq, sizeof(VirtQueueElement));
if (!elem) {
- if (i == 0)
- return -1;
- error_report("virtio-net unexpected empty queue: "
- "i %zd mergeable %d offset %zd, size %zd, "
- "guest hdr len %zd, host hdr len %zd "
- "guest features 0x%" PRIx64,
- i, n->mergeable_rx_bufs, offset, size,
- n->guest_hdr_len, n->host_hdr_len,
- vdev->guest_features);
- exit(1);
+ if (i) {
+ virtio_error(vdev, "virtio-net unexpected empty queue: "
+ "i %zd mergeable %d offset %zd, size %zd, "
+ "guest hdr len %zd, host hdr len %zd "
+ "guest features 0x%" PRIx64,
+ i, n->mergeable_rx_bufs, offset, size,
+ n->guest_hdr_len, n->host_hdr_len,
+ vdev->guest_features);
+ }
+ return -1;
}
if (elem->in_num < 1) {
- error_report("virtio-net receive queue contains no in buffers");
- exit(1);
+ virtio_error(vdev,
+ "virtio-net receive queue contains no in buffers");
+ return -1;
}
sg = elem->in_sg;
- Re: [Qemu-devel] [PATCH 1/7] virtio-9p: handle handle_9p_output() error, (continued)
[Qemu-devel] [PATCH 2/7] virtio-blk: handle virtio_blk_handle_request() errors, Greg Kurz, 2016/09/21
[Qemu-devel] [PATCH 3/7] virtio-net: handle virtio_net_handle_ctrl() error, Greg Kurz, 2016/09/21
[Qemu-devel] [PATCH 4/7] virtio-net: handle virtio_net_receive() errors,
Greg Kurz <=
[Qemu-devel] [PATCH 5/7] virtio-net: handle virtio_net_flush_tx() errors, Greg Kurz, 2016/09/21
[Qemu-devel] [PATCH 6/7] virtio-scsi: convert virtio_scsi_bad_req() to use virtio_error(), Greg Kurz, 2016/09/21
[Qemu-devel] [PATCH 7/7] virtio-scsi: handle virtio_scsi_set_config() error, Greg Kurz, 2016/09/21
Re: [Qemu-devel] [PATCH 0/7] virtio: avoid inappropriate QEMU termination, no-reply, 2016/09/21