Re: [Qemu-devel] [PATCH v5 0/5] Geometry and blocksize detection for backing devices.

[Stefan Hajnoczi <stefanha@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1357 bytes --]

On Tue, Jan 13, 2015 at 08:07:15PM +0100, Christian Borntraeger wrote:
> Am 13.01.2015 um 17:04 schrieb Stefan Hajnoczi:
> [...]
> > I'm really starting to get worried that you are going to break things.
> > This DASD hack is a layering violation but okay, go ahead if you want.
> > But now you are also thinking about breaking live migration.
> > 
> > The proper thing to do is to introduce libvirt XML syntax for DASD.
> > That way the geometry can be handled as part of the machine
> > configuration.  Then live migration and storage management tools can do
> > the right thing.
> > 
> > I've said this should be done in libvirt repeatedly but you keep wanting
> > to hack QEMU instead of doing this cleanly :(.
> > 
> > If you have plans to expand on this hack, please scrap this series and
> > introduce libvirt XML syntax instead.
> 
> We had no plans to expand on this band-aid. I just tried to come up with ideas beyond this hack to make this more acceptible to you (because I though that
> you want something on top). Seems that I misunderstood you, so if you  prefer
> to just keep this hack and not do anything further in that direction, this is
> totally fine with me.
> 
> So lets just fix up the small nits and go ahead, ok?

Yes, I'm fine with this series (modulo the review comments which require
a v6).

Stefan

[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]