On Mon, Jan 25, 2021 at 10:13:51 +0000, Daniel Berrange wrote: > On Mon, Jan 25, 2021 at 11:11:43AM +0100, Peter Krempa wrote: > > On Mon, Jan 25, 2021 at 10:04:41 +0000, Daniel Berrange wrote: [...] > > Unfortunately that's after the frontend is initialized (and thus > > remembers the readonly state) and after the storage is already open > > (thus it already asserted-or wanted to assert the write lock). > > > > 'blockdev-create' is used to prevent another invocation of qemu-img > > Wouldn't it make life simpler to just use qemu-img and avoid these > problems in the first place ? The use of 'qemu-img' in the qemu driver code is limited to inactive snapshots and I'd really like us to keep it that way. I'm more willing to declare that sharing of the original image with a <transient/> disk is unsupported rather than cave in using qemu-img. The user can always provide their own overlay and discard it appropriately.