[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] virtio-serial: Simplify virtio_serial_load()
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH] virtio-serial: Simplify virtio_serial_load() |
Date: |
Wed, 30 Jun 2010 20:39:25 +0200 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Mon, May 31, 2010 at 04:19:43PM +0200, Markus Armbruster wrote:
> For all i, ports_map[i] is used in and only in the i-th iteration.
> Replace the dynamic array by a scalar variable.
>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> hw/virtio-serial-bus.c | 12 +++---------
> 1 files changed, 3 insertions(+), 9 deletions(-)
Thanks, applied.
> diff --git a/hw/virtio-serial-bus.c b/hw/virtio-serial-bus.c
> index 3ce95e8..bcc6d5d 100644
> --- a/hw/virtio-serial-bus.c
> +++ b/hw/virtio-serial-bus.c
> @@ -492,8 +492,7 @@ static int virtio_serial_load(QEMUFile *f, void *opaque,
> int version_id)
> {
> VirtIOSerial *s = opaque;
> VirtIOSerialPort *port;
> - size_t ports_map_size;
> - uint32_t max_nr_ports, nr_active_ports, *ports_map;
> + uint32_t max_nr_ports, nr_active_ports, ports_map;
> unsigned int i;
>
> if (version_id > 2) {
> @@ -517,22 +516,17 @@ static int virtio_serial_load(QEMUFile *f, void
> *opaque, int version_id)
> return -EINVAL;
> }
>
> - ports_map_size = sizeof(uint32_t) * (max_nr_ports + 31) / 32;
> - ports_map = qemu_malloc(ports_map_size);
> -
> for (i = 0; i < (max_nr_ports + 31) / 32; i++) {
> - qemu_get_be32s(f, &ports_map[i]);
> + qemu_get_be32s(f, &ports_map);
>
> - if (ports_map[i] != s->ports_map[i]) {
> + if (ports_map != s->ports_map[i]) {
> /*
> * Ports active on source and destination don't
> * match. Fail migration.
> */
> - qemu_free(ports_map);
> return -EINVAL;
> }
> }
> - qemu_free(ports_map);
>
> qemu_get_be32s(f, &nr_active_ports);
>
> --
> 1.6.6.1
>
>
>
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net