Re: [Qemu-devel] [PATCH v2 6/8] virtio-blk: Use VirtIOBlockReq.in to drop VirtIOBlockReq.inhdr

[Paolo Bonzini <pbonzini@redhat.com>]

Il 05/06/2014 03:58, Fam Zheng ha scritto:
>      /* Grab inhdr for later */
> -    in_size = iov_size(in_iov, in_num);
> -    if (in_size < sizeof(struct virtio_blk_inhdr)) {
> -        error_report("virtio_blk request inhdr too short");
> -        return -EFAULT;
> -    }
> -    inhdr = g_slice_new(QEMUIOVector);
> -    qemu_iovec_init(inhdr, 1);
> -    qemu_iovec_concat_iov(inhdr, in_iov, in_num,
> -            in_size - sizeof(struct virtio_blk_inhdr),
> -            sizeof(struct virtio_blk_inhdr));
> -    iov_discard_back(in_iov, &in_num, sizeof(struct virtio_blk_inhdr));
> +    inhdr = (void *)in_iov[in_num - 1].iov_base;

This would assume a particular layout for the virtio buffers.  You need 
to add in_iov[in_num - 1].iov_len - sizeof(struct virtio_blk_header).

It's probably also good to ahve an assertion that in_iov[in_num - 
1].iov_len > sizeof(struct virtio_blk_header), and a comment saying that 
the assertion is always true because the struct is only 1-byte long.

Paolo