[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 3/6] virtio-net: unbreak any layout
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PULL 3/6] virtio-net: unbreak any layout |
Date: |
Mon, 20 Jul 2015 08:26:41 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 |
On 07/20/2015 06:12 AM, Michael S. Tsirkin wrote:
> From: Jason Wang <address@hidden>
>
> Commit 032a74a1c0fcdd5fd1c69e56126b4c857ee36611
> ("virtio-net: byteswap virtio-net header") breaks any layout by
> requiring out_sg[0].iov_len >= n->guest_hdr_len. Fixing this by
> copying header to temporary buffer if swap is needed, and then use
> this buffer as part of out_sg.
>
> Fixes 032a74a1c0fcdd5fd1c69e56126b4c857ee36611
> ("virtio-net: byteswap virtio-net header")
> Cc: address@hidden
> Cc: address@hidden
> Signed-off-by: Jason Wang <address@hidden>
> Reviewed-by: Michael S. Tsirkin <address@hidden>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
I think my R-b was intended for 2/6, not this one. But if this has
already been pulled, it's not a show-stopper.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PULL 0/6] virtio, vhost, pc fixes for 2.4, Michael S. Tsirkin, 2015/07/20
- [Qemu-devel] [PULL 1/6] ich9: fix skipped vmstate_memhp_state subsection, Michael S. Tsirkin, 2015/07/20
- [Qemu-devel] [PULL 2/6] Revert "vhost-user: add multi queue support", Michael S. Tsirkin, 2015/07/20
- [Qemu-devel] [PULL 3/6] virtio-net: unbreak any layout, Michael S. Tsirkin, 2015/07/20
- Re: [Qemu-devel] [PULL 3/6] virtio-net: unbreak any layout,
Eric Blake <=
- [Qemu-devel] [PULL 5/6] virtio-net: Flush incoming queues when DRIVER_OK is being set, Michael S. Tsirkin, 2015/07/20
- [Qemu-devel] [PULL 6/6] virtio-net: remove virtio queues if the guest doesn't support multiqueue, Michael S. Tsirkin, 2015/07/20
- [Qemu-devel] [PULL 4/6] pci_add_capability: remove duplicate comments, Michael S. Tsirkin, 2015/07/20
- Re: [Qemu-devel] [PULL 0/6] virtio, vhost, pc fixes for 2.4, Peter Maydell, 2015/07/20