|Subject:||Re: Configuration vs. compat hints [was Re: [Qemu-devel] [PATCHv3 03/13] qemu: add routines to manage PCI capabilities]|
|Date:||Fri, 12 Jun 2009 11:11:54 -0500|
|User-agent:||Thunderbird 22.214.171.124 (X11/20090320)|
Mark McLoughlin wrote:
On Fri, 2009-06-12 at 09:51 -0500, Anthony Liguori wrote:Mark McLoughlin wrote:What's the problem with parsing the device config and modifying it? Is it just complexity?On Wed, 2009-06-10 at 20:27 +0100, Jamie Lokier wrote:Michael S. Tsirkin wrote:I think the right long term answer to all this is a way to get QEMU to dump it's current machine configuration in glorious detail as a file which can be reloaded as a machine configuration.And then we'll have the same set of problems there.We will, and the solution will be the same: options to create devices as they were in older versions of QEMU. It only needs to cover device features which matter to guests, not every bug fix. However with a machine configuration which is generated by QEMU, there's less worry about proliferation of obscure options, compared with the command line. You don't necessarily have to document every backward-compatibility option in any detail, you just have to make sure it's written and read properly, which is much the same thing as the snapshot code does.This is a sensible plan, but I don't think we should mix these compat options in with the VM manager supplied configuration. There are two problems with that approach. = Problem 1 - VM manager needs to parse qemu config = Your proposal implies: - VM manager supplies a basic configuration to qemu- It then immediately asks qemu for a dump of the machine configuration in all its glorious detail and retains thatconfig- If the VM manager wishes to add a new device it needs to parse the qemu config and add it, rather than just generate an entirely new configYes, complexity is the issue.If we provided a mechanism to simplify manipulating a device config, would that eliminate the concern here?In libvirt's case, a lot of the complexity would come from needing to figure out what to change.
Right, libvirt wants to be able to easily say "add a scsi block device to this VM". The way I see this working is that there would be a default pc.dtc. We would still have a -drive file=foo.img,if=scsi option that would really just be a wrapper around first searching for an existing LSI controller, if one exists, attaching the lun, if not, create one, etc.
libvirt could continue to use this sort of interface. However, as it wants to do more advanced things, it may have to dive into the device tree itself.
On live migration, QEMU will save a copy of the device tree somewhere and libvirt needs to keep track of it. It can treat it as opaque. -M /path/to/foo.dtc -drive file=foo.img,if=scsi should continue working as expected IMHO.
Regards, Anthony Liguori
|[Prev in Thread]||Current Thread||[Next in Thread]|