[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 0/5] Geometry and blocksize detection for bac
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v5 0/5] Geometry and blocksize detection for backing devices. |
Date: |
Wed, 14 Jan 2015 13:57:57 +0000 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
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
pgpK4MwBgx_rc.pgp
Description: PGP signature