Return -EINVAL if device's ack is VIRTIO_NET_ERR

[PATCH RFC 0/1] Return -EINVAL if device's ack is VIRTIO_NET_ERR

Hawkins Jiawei posted 1 patch 11 months, 1 week ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/cover.1686746406.git.yin31149@gmail.com

Maintainers: Jason Wang <jasowang@redhat.com>

net/vhost-vdpa.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)

Expand all Fold all

[PATCH RFC 0/1] Return -EINVAL if device's ack is VIRTIO_NET_ERR

Posted by Hawkins Jiawei 11 months, 1 week ago

This patch fixes the problem that vhost_net_start_one() doesn't cancel
the device startup and returns 0 even if the device's ack is VIRTIO_NET_ERR
in net->nc->info->load().

Note that this problem also exists in
patch "vdpa: Add vhost_vdpa_net_load_offloads()" at [1].
Because this patch has not been merged yet, so I will submit the v2 patch
to fix the problem in that patch once the pending patch
has been successfully merged.

[1]. https://lists.nongnu.org/archive/html/qemu-devel/2023-06/msg00504.html

Hawkins Jiawei (1):
  vdpa: Return -EINVAL if device's ack is VIRTIO_NET_ERR

 net/vhost-vdpa.c | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

-- 
2.25.1