Alex Bligh <alex@xxxxxxxxxxx> writes: [...] >>> +/* NB cirrus-vga default value is 8MB anyway, save if we >>> + * monkey patch it to change the default when the qemu-kvm-migration >>> + * machine parameter is selected >>> + */ >>> + >> >> This is too hacky for my taste. >> How about creating a new machine e.g. pc-qemu-kvm-1.0 and in >> pc_early_init_pci_1_0, changing compat_props for pc-1.0 to point to the >> compat_props of pc-qemu-kvm-1.0? > > Hang on a second! v2 of this patch DID use a new virtual machine, > called exactly that. I thought you were objecting to that and > wanting a machine parameter instead! It's far easier with a new > machine type, and I'd far prefer a new machine type. > > If you were just objecting to the fact that pc-1.0 was made to > be an alias of either one or the other at compile time, simply > drop the second patch of the v2 patchset. > > If we have a new machine type, I don't /think/ I need the early_init > thing at all (I may be wrong about that). I also prefer a new machine type. Ideally, the management application understands that there are two incompatibile versions QEMU (upstream and old qemu-kvm), and how to map their machine types to current QEMU's. If that's not practical, then downstream can still alias the machine types around to make things just work in the most important downstream scenarios. The most important upstream scenario is QEMU <-> QEMU, of course. -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list