Re: [dpdk-dev] [PATCH v3 5/6] net/virtio-user: don't assume vhost status feature

Wed, 28 Oct 2020 03:38:23 -0700 


On 10/26/20 5:39 PM, Adrian Moreno wrote:
> There are some status reads and updates that need to happen before the
> protocol features are negotiated. Therefore, assuming the backend does
> support this feature can lead to failures.
> 
> On server mode, do not assume the backend supports
> VHOST_USER_PROTOCOL_F_STATUS. Activate it back on reconnection and
> cleare it on disconnection.
> 
> Fixes: 57912824615f ("net/virtio-user: support vhost status setting")
> Cc: [email protected]
> Cc: [email protected]
> 
> Signed-off-by: Adrian Moreno <[email protected]>
> ---
>  drivers/net/virtio/virtio_user/virtio_user_dev.c | 6 ++++++
>  drivers/net/virtio/virtio_user_ethdev.c          | 8 ++++++++
>  2 files changed, 14 insertions(+)
> 


Reviewed-by: Maxime Coquelin <[email protected]>

Thanks,
Maxime























					Reply via email to