[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers |
Date: |
Mon, 26 Jan 2015 22:10:21 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
On 26/01/2015 21:43, Max Reitz wrote:
>> If the NBD server is attached to the BDS, it should keep serving the BDS.
>
> The problem is that it is no longer attached to the BDS, but to the BB.
That's not necessarily a problem. :) It is the cause of the problem though.
Is it possible to attach two BBs to the same BDS?
Because part of the solution could be to introduce a new blockdev-serve
command that takes a BDS, creates a BB and exports that BB.
> I think to retain compatibility we could either just do what we always
> did (although I find it wrong), or we could simply set up an eject
> blocker when attaching an NBD server to a BB. What do you think?
An eject blocker would also break backwards-compatibility though. What
about an eject notifier? Would that concept make sense?
Paolo
- Re: [Qemu-devel] [PATCH 01/21] block: Guard remaining unsafe blk_bs() callers, (continued)
- [Qemu-devel] [PATCH 08/21] block: Make bdrv_close() static, Max Reitz, 2015/01/26
- [Qemu-devel] [PATCH 03/21] block: Add bdrv_close_all() notifiers, Max Reitz, 2015/01/26
- [Qemu-devel] [PATCH 09/21] block: Add blk_name_taken(), Max Reitz, 2015/01/26
- [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers, Max Reitz, 2015/01/26
- Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers, Paolo Bonzini, 2015/01/26
- Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers, Max Reitz, 2015/01/26
- Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers, Max Reitz, 2015/01/26
- Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers, Paolo Bonzini, 2015/01/26
- Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers, Eric Blake, 2015/01/28
- Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers, Max Reitz, 2015/01/28
- Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers, Paolo Bonzini, 2015/01/29
Re: [Qemu-devel] [PATCH 04/21] block: Add bdrv_close_all() handlers, Eric Blake, 2015/01/28
[Qemu-devel] [PATCH 05/21] block: Remove per-BDS close notifiers, Max Reitz, 2015/01/26
[Qemu-devel] [PATCH 10/21] block: Add blk_next_inserted(), Max Reitz, 2015/01/26