[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/5] Revert "virtio-net: enable virtio 1.0"
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH 4/5] Revert "virtio-net: enable virtio 1.0" |
Date: |
Mon, 13 Jul 2015 08:16:17 +0200 |
On Mon, 13 Jul 2015 13:46:50 +0800
Jason Wang <address@hidden> wrote:
> This reverts commit df91055db5c9cee93d70ca8c08d72119a240b987.
>
> This is because:
> - vhost support virtio 1.0 now
> - transport code (e.g virtio-pci) set this feature when modern is
> enabled, setting this unconditionally will break disable-modern=on.
*scratches head*
Why is transport code now supposed to set VERSION_1? I thought we
wanted to have the individual devices offer it, once they are converted.
No objection on removing the dependency on !vhost.
>
> Cc: Cornelia Huck <address@hidden>
> Signed-off-by: Jason Wang <address@hidden>
> ---
> hw/net/virtio-net.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
> index d728233..e3c2db3 100644
> --- a/hw/net/virtio-net.c
> +++ b/hw/net/virtio-net.c
> @@ -466,7 +466,6 @@ static uint64_t virtio_net_get_features(VirtIODevice
> *vdev, uint64_t features)
> }
>
> if (!get_vhost_net(nc->peer)) {
> - virtio_add_feature(&features, VIRTIO_F_VERSION_1);
> return features;
> }
> return vhost_net_get_features(get_vhost_net(nc->peer), features);
[Qemu-devel] [PATCH 5/5] virtio-net: unbreak any layout, Jason Wang, 2015/07/13