@@ -465,6 +465,21 @@ static uint32_t virtio_net_get_features(VirtIODevice *vdev, uint32_t features)
features &= ~(0x1 << VIRTIO_NET_F_HOST_UFO);
}
+ if (!(features & (0x1 << VIRTIO_NET_F_CTRL_VQ))) {
+ features &= ~(0x1 << VIRTIO_NET_F_CTRL_GUEST_OFFLOADS);
+ features &= ~(0x1 << VIRTIO_NET_F_CTRL_RX);
+ features &= ~(0x1 << VIRTIO_NET_F_CTRL_VLAN);
+ features &= ~(0x1 << VIRTIO_NET_F_CTRL_RX_EXTRA);
+ features &= ~(0x1 << VIRTIO_NET_F_CTRL_MAC_ADDR);
+ features &= ~(0x1 << VIRTIO_NET_F_GUEST_ANNOUNCE);
+ features &= ~(0x1 << VIRTIO_NET_F_MQ);
+ features &= ~(0x1 << VIRTIO_NET_F_CTRL_MAC_ADDR);
+ }
+
+ if (!(features & (0x1 << VIRTIO_NET_F_CTRL_RX))) {
+ features &= ~(0x1 << VIRTIO_NET_F_CTRL_RX_EXTRA);
+ }
+
if (!get_vhost_net(nc->peer)) {
return features;
}
We should not advertise the feature that depends on ctrl vq is it was not enabled. And not advertise the VIRTIO_NET_F_CTRL_RX_EXTRA is VIRTIO_NET_F_CTRL_RX is not enabled. Otherwise, this may confuse guest driver and lead unexpected bugs. Signed-off-by: Jason Wang <jasowang@redhat.com> --- hw/net/virtio-net.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+)