[PATCH] vhost-vdpa: negotiate VIRTIO_NET_F_STATUS with driver

[PATCH] vhost-vdpa: negotiate VIRTIO_NET_F_STATUS with driver

[Si-Wei Liu]

Vendor driver may not support or implement config
interrupt delivery for link status notifications.
In this event, vendor driver is expected to NACK
the feature, but guest will keep link always up.

Signed-off-by: Si-Wei Liu <si-wei.liu@oracle.com>
---
 net/vhost-vdpa.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/vhost-vdpa.c b/net/vhost-vdpa.c
index bc0e0d2..55e01d9 100644
--- a/net/vhost-vdpa.c
+++ b/net/vhost-vdpa.c
@@ -55,6 +55,7 @@ const int vdpa_feature_bits[] = {
     VIRTIO_F_IOMMU_PLATFORM,
     VIRTIO_F_RING_PACKED,
     VIRTIO_NET_F_GUEST_ANNOUNCE,
+    VIRTIO_NET_F_STATUS,
     VHOST_INVALID_FEATURE_BIT
 };
 
-- 
1.8.3.1

Re: [PATCH] vhost-vdpa: negotiate VIRTIO_NET_F_STATUS with driver

[Jason Wang]

On 2020/10/2 上午4:09, Si-Wei Liu wrote:
> Vendor driver may not support or implement config
> interrupt delivery for link status notifications.
> In this event, vendor driver is expected to NACK
> the feature, but guest will keep link always up.
>
> Signed-off-by: Si-Wei Liu <si-wei.liu@oracle.com>
> ---
>   net/vhost-vdpa.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/net/vhost-vdpa.c b/net/vhost-vdpa.c
> index bc0e0d2..55e01d9 100644
> --- a/net/vhost-vdpa.c
> +++ b/net/vhost-vdpa.c
> @@ -55,6 +55,7 @@ const int vdpa_feature_bits[] = {
>       VIRTIO_F_IOMMU_PLATFORM,
>       VIRTIO_F_RING_PACKED,
>       VIRTIO_NET_F_GUEST_ANNOUNCE,
> +    VIRTIO_NET_F_STATUS,
>       VHOST_INVALID_FEATURE_BIT
>   };
>   


Acked-by: Jason Wang <jasowang@redhat.com>

Michael, do you want to pick this patch or I will do that?

Thanks

Re: [PATCH] vhost-vdpa: negotiate VIRTIO_NET_F_STATUS with driver

[Michael S. Tsirkin]

On Thu, Oct 15, 2020 at 03:24:29PM +0800, Jason Wang wrote:
> 
> On 2020/10/2 上午4:09, Si-Wei Liu wrote:
> > Vendor driver may not support or implement config
> > interrupt delivery for link status notifications.
> > In this event, vendor driver is expected to NACK
> > the feature, but guest will keep link always up.
> > 
> > Signed-off-by: Si-Wei Liu <si-wei.liu@oracle.com>
> > ---
> >   net/vhost-vdpa.c | 1 +
> >   1 file changed, 1 insertion(+)
> > 
> > diff --git a/net/vhost-vdpa.c b/net/vhost-vdpa.c
> > index bc0e0d2..55e01d9 100644
> > --- a/net/vhost-vdpa.c
> > +++ b/net/vhost-vdpa.c
> > @@ -55,6 +55,7 @@ const int vdpa_feature_bits[] = {
> >       VIRTIO_F_IOMMU_PLATFORM,
> >       VIRTIO_F_RING_PACKED,
> >       VIRTIO_NET_F_GUEST_ANNOUNCE,
> > +    VIRTIO_NET_F_STATUS,
> >       VHOST_INVALID_FEATURE_BIT
> >   };
> 
> 
> Acked-by: Jason Wang <jasowang@redhat.com>
> 
> Michael, do you want to pick this patch or I will do that?
> 
> Thanks

Tagged, thanks!