[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH for-2.4] Revert "vhost-user: Send VHOST_RESET_OWNER
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] [PATCH for-2.4] Revert "vhost-user: Send VHOST_RESET_OWNER on vhost stop" |
Date: |
Thu, 30 Jul 2015 11:36:56 +0300 |
This reverts commit 294ce717e0f212ed0763307f3eab72b4a1bdf4d0.
vhost stop happens e.g. when guest unloads the driver,
so closing the backend connection is not the right
thing to do here.
VHOST_RESET_OWNER should happen on vhost_dev_cleanup - it's
the counterpart of VHOST_SET_OWNER.
Cc: Luke Gorrie <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
---
I think we need this in 2.4 to avoid introducing regressions
in the protocol. We'll fix properly in 2.5.
Luke, can you comment please?
hw/net/vhost_net.c | 7 -------
1 file changed, 7 deletions(-)
diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
index c864237..b8d88ec 100644
--- a/hw/net/vhost_net.c
+++ b/hw/net/vhost_net.c
@@ -284,13 +284,6 @@ static void vhost_net_stop_one(struct vhost_net *net,
&file);
assert(r >= 0);
}
- } else if (net->nc->info->type == NET_CLIENT_OPTIONS_KIND_VHOST_USER) {
- for (file.index = 0; file.index < net->dev.nvqs; ++file.index) {
- const VhostOps *vhost_ops = net->dev.vhost_ops;
- int r = vhost_ops->vhost_call(&net->dev, VHOST_RESET_OWNER,
- NULL);
- assert(r >= 0);
- }
}
if (net->nc->info->poll) {
net->nc->info->poll(net->nc, true);
--
MST
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-devel] [PATCH for-2.4] Revert "vhost-user: Send VHOST_RESET_OWNER on vhost stop",
Michael S. Tsirkin <=