[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/4] block: move block exports to libblockdev
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH 3/4] block: move block exports to libblockdev |
Date: |
Fri, 25 Sep 2020 16:06:02 +0100 |
On Fri, Sep 25, 2020 at 04:11:54PM +0200, Paolo Bonzini wrote:
> On 25/09/20 15:42, Stefan Hajnoczi wrote:
> > Block exports are used by softmmu, qemu-storage-daemon, and qemu-nbd.
> > They are not used by other programs and are not otherwise needed in
> > libblock.
> >
> > Undo the recent move of blockdev-nbd.c from blockdev_ss into block_ss.
> > Two stubs are required to support this:
> > 1. bdrv_close_all() (libblock) calls blk_exp_close_all() (libblockdev).
> > 2. qemu_system_killed() is called by os-posix.c (libblockdev) and not
> > implemented in qemu-nbd.
> >
> > Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
>
> Another possibility is to call os_setup_signal_handling in qemu-nbd.c
> where we currently set the SIGTERM handler. The existing
> termsig_handler can be repurposed as qemu_system_killed.
That is nicer. Will fix in v2.
Stefan
signature.asc
Description: PGP signature