Re: [Qemu-devel] [PATCH] vhost-user: ignore qemu-only features

[Jason Wang <jasowang@redhat.com>]

On 11/18/2015 10:08 AM, Jason Wang wrote:
>
> On 11/17/2015 10:57 PM, Michael S. Tsirkin wrote:
>> Some feratures (such as ctrl vq) are supported
>> by qemu without need to communicate with the
>> backend.
>>
>> Drop them from the feature mask so we set them
>> unconditionally.
>>
>> Reported-by: Victor Kaplansky <vkaplans@redhat.com>
>> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
>> ---
>>  hw/net/vhost_net.c | 9 ---------
>>  1 file changed, 9 deletions(-)
>>
>> diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
>> index 14337a4..8c9b52d 100644
>> --- a/hw/net/vhost_net.c
>> +++ b/hw/net/vhost_net.c
>> @@ -77,15 +77,6 @@ static const int user_feature_bits[] = {
>>      VIRTIO_NET_F_HOST_ECN,
>>      VIRTIO_NET_F_HOST_UFO,
>>      VIRTIO_NET_F_MRG_RXBUF,
>> -    VIRTIO_NET_F_STATUS,
>> -    VIRTIO_NET_F_CTRL_VQ,
>> -    VIRTIO_NET_F_CTRL_RX,
>> -    VIRTIO_NET_F_CTRL_VLAN,
>> -    VIRTIO_NET_F_CTRL_RX_EXTRA,
>> -    VIRTIO_NET_F_CTRL_MAC_ADDR,
>> -    VIRTIO_NET_F_CTRL_GUEST_OFFLOADS,
>> -
>> -    VIRTIO_NET_F_GUEST_ANNOUNCE,
>>  
>>      VIRTIO_NET_F_MQ,
>>  
> Acked-by: Jason Wang <jasowang@redhat.com>

How about VIRTIO_NET_F_GUEST_* ?