[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-stable] [Qemu-devel] [PATCH 2/2] virtio-net: mark VIRTIO_NET_F
From: |
Halil Pasic |
Subject: |
Re: [Qemu-stable] [Qemu-devel] [PATCH 2/2] virtio-net: mark VIRTIO_NET_F_GSO as legacy |
Date: |
Mon, 7 Nov 2016 20:25:44 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 |
On 11/04/2016 12:01 PM, Michael S. Tsirkin wrote:
> virtio 1.0 spec says this is a legacy feature bit,
> hide it from guests in legacy mode.
>
Probably the other way around: since its legacy
it is supposed to be hidden from non-legacy )and
exposed to legacy) or?
> Note: for cross-version migration compatibility,
> we keep the bit set in host_features.
> The result will be that a guest migrating cross-version
> will see host features change under it.
> As guests only seem to read it once, this should
> not be an issue. Meanwhile, will work to fix guests to
> ignore this bit in virtio1 mode, too.
>
Makes sense to me (except for the probably typo).
Regards,
Halil
> Cc: address@hidden
> Signed-off-by: Michael S. Tsirkin <address@hidden>
> ---
> hw/net/virtio-net.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
> index 20aa63e..b68c69d 100644
> --- a/hw/net/virtio-net.c
> +++ b/hw/net/virtio-net.c
> @@ -1942,6 +1942,7 @@ static void virtio_net_class_init(ObjectClass *klass,
> void *data)
> vdc->guest_notifier_pending = virtio_net_guest_notifier_pending;
> vdc->load = virtio_net_load_device;
> vdc->save = virtio_net_save_device;
> + vdc->legacy_features |= (0x1 << VIRTIO_NET_F_GSO);
> }
>
> static const TypeInfo virtio_net_info = {
>