[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 0/5] block: Drop BDS.filename
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-block] [PATCH v2 0/5] block: Drop BDS.filename |
Date: |
Thu, 6 Aug 2015 10:44:36 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Wed, Aug 05, 2015 at 10:52:44PM +0200, Max Reitz wrote:
> Regarding the fear that this might change current behavior, especially
> for libvirt which used filenames to identify nodes in the BDS graph:
> Since bdrv_open() already calls bdrv_refresh_filename() today, and
> apparently everything works fine, this series will most likely not break
> anything. The only thing that will change is if the BDS graph is
> dynamically reconfigured at runtime, and in that case it's most likely a
> bug fix. Also, I don't think libvirt supports such cases already.
I haven't reviewed the patches but seems like a perfectly reasonable
thing to do at the beginning of the QEMU 2.5 development cycle. If
anything comes up there will be time ot fix it before the next QEMU
version is released.
pgpiaweuCBVLA.pgp
Description: PGP signature
- [Qemu-block] [PATCH v2 0/5] block: Drop BDS.filename, Max Reitz, 2015/08/05
- [Qemu-block] [PATCH v2 1/5] block: Change bdrv_get_encrypted_filename(), Max Reitz, 2015/08/05
- [Qemu-block] [PATCH v2 2/5] block: Avoid BlockDriverState.filename, Max Reitz, 2015/08/05
- [Qemu-block] [PATCH v2 3/5] block: Add bdrv_filename(), Max Reitz, 2015/08/05
- [Qemu-block] [PATCH v2 4/5] block: Drop BlockDriverState.filename, Max Reitz, 2015/08/05
- [Qemu-block] [PATCH v2 5/5] iotests: Test changed Quorum filename, Max Reitz, 2015/08/05
- Re: [Qemu-block] [PATCH v2 0/5] block: Drop BDS.filename,
Stefan Hajnoczi <=