[dpdk-dev] [PATCH v3 5/6] net/virtio-user: don't assume vhost status feature
Maxime Coquelin
maxime.coquelin at redhat.com
Wed Oct 28 11:37:01 CET 2020
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: maxime.coquelin at redhat.com
> Cc: stable at dpdk.org
>
> Signed-off-by: Adrian Moreno <amorenoz at redhat.com>
> ---
> 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 <maxime.coquelin at redhat.com>
Thanks,
Maxime
More information about the dev
mailing list