[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 1/9] virtio-serial-bus: save/load: Ensure target has
From: |
Amit Shah |
Subject: |
[Qemu-devel] [PATCH 1/9] virtio-serial-bus: save/load: Ensure target has enough ports |
Date: |
Tue, 23 Mar 2010 20:00:11 +0530 |
The target could be started with max_nr_ports for a virtio-serial device
lesser than what was available on the source machine. Fail the migration
in such a case.
Signed-off-by: Amit Shah <address@hidden>
Reported-by: Juan Quintela <address@hidden>
---
hw/virtio-serial-bus.c | 10 +++++++++-
1 files changed, 9 insertions(+), 1 deletions(-)
diff --git a/hw/virtio-serial-bus.c b/hw/virtio-serial-bus.c
index 17c1ec1..36985a1 100644
--- a/hw/virtio-serial-bus.c
+++ b/hw/virtio-serial-bus.c
@@ -374,6 +374,8 @@ static void virtio_serial_save(QEMUFile *f, void *opaque)
/* Items in struct VirtIOSerial */
+ qemu_put_be32s(f, &s->bus->max_nr_ports);
+
/* Do this because we might have hot-unplugged some ports */
nr_active_ports = 0;
QTAILQ_FOREACH(port, &s->ports, next)
@@ -399,7 +401,7 @@ static int virtio_serial_load(QEMUFile *f, void *opaque,
int version_id)
{
VirtIOSerial *s = opaque;
VirtIOSerialPort *port;
- uint32_t nr_active_ports;
+ uint32_t max_nr_ports, nr_active_ports;
unsigned int i;
if (version_id > 2) {
@@ -420,6 +422,12 @@ static int virtio_serial_load(QEMUFile *f, void *opaque,
int version_id)
/* Items in struct VirtIOSerial */
+ qemu_get_be32s(f, &max_nr_ports);
+ if (max_nr_ports > s->bus->max_nr_ports) {
+ /* Source could have more ports than us. Fail migration. */
+ return -EINVAL;
+ }
+
qemu_get_be32s(f, &nr_active_ports);
/* Items in struct VirtIOSerialPort */
--
1.6.2.5
- [Qemu-devel] [PATCH 0/9] v2: Fixes, new way of discovering ports, Amit Shah, 2010/03/23
- [Qemu-devel] [PATCH 1/9] virtio-serial-bus: save/load: Ensure target has enough ports,
Amit Shah <=
- [Qemu-devel] [PATCH 2/9] virtio-serial-bus: save/load: Ensure nr_ports on src and dest are same., Amit Shah, 2010/03/23
- [Qemu-devel] [PATCH 3/9] virtio-serial: Remove redundant check for 0-sized write request, Amit Shah, 2010/03/23
- [Qemu-devel] [PATCH 4/9] virtio-serial: Update copyright year to 2010, Amit Shah, 2010/03/23
- [Qemu-devel] [PATCH 5/9] virtio-serial: save/load: Ensure we have hot-plugged ports instantiated, Amit Shah, 2010/03/23
- [Qemu-devel] [PATCH 6/9] virtio-serial-bus: Use control messages to notify guest of new ports, Amit Shah, 2010/03/23
- [Qemu-devel] [PATCH 7/9] virtio-serial-bus: Let the guest know of host connection changes after migration, Amit Shah, 2010/03/23
- [Qemu-devel] [PATCH 8/9] virtio-serial: Handle scatter-gather buffers for control messages, Amit Shah, 2010/03/23
- [Qemu-devel] [PATCH 9/9] virtio-serial: Handle scatter/gather input from the guest, Amit Shah, 2010/03/23
- [Qemu-devel] Re: [PATCH 9/9] virtio-serial: Handle scatter/gather input from the guest, Amit Shah, 2010/03/23
[Qemu-devel] Re: [PATCH 0/9] v2: Fixes, new way of discovering ports, Juan Quintela, 2010/03/23