This patch backports bugfix patch series from qemu upstream v4.1.1 Signed-off-by: Ying Fang <fangying1@huawei.com>
42 lines
1.5 KiB
Diff
42 lines
1.5 KiB
Diff
From 0694c489cd240620fee5675e8d24c7ce02d1d67d Mon Sep 17 00:00:00 2001
|
|
From: Peter Lieven <pl@kamp.de>
|
|
Date: Tue, 10 Sep 2019 17:41:09 +0200
|
|
Subject: [PATCH] block/nfs: tear down aio before nfs_close
|
|
|
|
nfs_close is a sync call from libnfs and has its own event
|
|
handler polling on the nfs FD. Avoid that both QEMU and libnfs
|
|
are intefering here.
|
|
|
|
CC: qemu-stable@nongnu.org
|
|
Signed-off-by: Peter Lieven <pl@kamp.de>
|
|
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
(cherry picked from commit 601dc6559725f7a614b6f893611e17ff0908e914)
|
|
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
|
|
---
|
|
block/nfs.c | 6 ++++--
|
|
1 file changed, 4 insertions(+), 2 deletions(-)
|
|
|
|
diff --git a/block/nfs.c b/block/nfs.c
|
|
index d93241b3bb..2b7a078241 100644
|
|
--- a/block/nfs.c
|
|
+++ b/block/nfs.c
|
|
@@ -390,12 +390,14 @@ static void nfs_attach_aio_context(BlockDriverState *bs,
|
|
static void nfs_client_close(NFSClient *client)
|
|
{
|
|
if (client->context) {
|
|
+ qemu_mutex_lock(&client->mutex);
|
|
+ aio_set_fd_handler(client->aio_context, nfs_get_fd(client->context),
|
|
+ false, NULL, NULL, NULL, NULL);
|
|
+ qemu_mutex_unlock(&client->mutex);
|
|
if (client->fh) {
|
|
nfs_close(client->context, client->fh);
|
|
client->fh = NULL;
|
|
}
|
|
- aio_set_fd_handler(client->aio_context, nfs_get_fd(client->context),
|
|
- false, NULL, NULL, NULL, NULL);
|
|
nfs_destroy_context(client->context);
|
|
client->context = NULL;
|
|
}
|
|
--
|
|
2.23.0
|