[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] virtio-serial: fix ANY_LAYOUT
From: |
Jason Wang |
Subject: |
Re: [Qemu-devel] [PATCH] virtio-serial: fix ANY_LAYOUT |
Date: |
Mon, 27 Jul 2015 15:41:07 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.8.0 |
On 07/27/2015 02:37 PM, Michael S. Tsirkin wrote:
> Don't assume a specific layout for control messages.
> Required by virtio 1.
>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
> ---
> hw/char/virtio-serial-bus.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/hw/char/virtio-serial-bus.c b/hw/char/virtio-serial-bus.c
> index 78c73e5..16de50f 100644
> --- a/hw/char/virtio-serial-bus.c
> +++ b/hw/char/virtio-serial-bus.c
> @@ -195,7 +195,8 @@ static size_t send_control_msg(VirtIOSerial *vser, void
> *buf, size_t len)
> return 0;
> }
>
> - memcpy(elem.in_sg[0].iov_base, buf, len);
> + /* TODO: detect a buffer that's too short, set NEEDS_RESET */
> + iov_from_buf(&elem.in_sg, buf, len);
>
> virtqueue_push(vq, &elem, len);
> virtio_notify(VIRTIO_DEVICE(vser), vq);
Reviewed-by: Jason Wang <address@hidden>