]> git.proxmox.com Git - mirror_qemu.git/commit - blockdev-nbd.c
nbd: Grab aio context lock in more places
authorEric Blake <eblake@redhat.com>
Tue, 17 Sep 2019 02:39:17 +0000 (21:39 -0500)
committerEric Blake <eblake@redhat.com>
Tue, 24 Sep 2019 12:30:19 +0000 (07:30 -0500)
commit61bc846d8c58535af6884b637a4005dd6111ea95
treea5c98306bdac3b6311aa472745ec81879a3cbdb2
parentb4961249af0403fa55aae57c4c8806b24f7a7b33
nbd: Grab aio context lock in more places

When iothreads are in use, the failure to grab the aio context results
in an assertion failure when trying to unlock things during blk_unref,
when trying to unlock a mutex that was not locked.  In short, all
calls to nbd_export_put need to done while within the correct aio
context.  But since nbd_export_put can recursively reach itself via
nbd_export_close, and recursively grabbing the context would deadlock,
we can't do the context grab directly in those functions, but must do
so in their callers.

Hoist the use of the correct aio_context from nbd_export_new() to its
caller qmp_nbd_server_add().  Then tweak qmp_nbd_server_remove(),
nbd_eject_notifier(), and nbd_esport_close_all() to grab the right
context, so that all callers during qemu now own the context before
nbd_export_put() can call blk_unref().

Remaining uses in qemu-nbd don't matter (since that use case does not
support iothreads).

Suggested-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20190917023917.32226-1-eblake@redhat.com>
Reviewed-by: Sergio Lopez <slp@redhat.com>
blockdev-nbd.c
include/block/nbd.h
nbd/server.c