[libvirt] [PATCH v1 2/3] qemu: Implement extended loader and nvram
Michal Privoznik
mprivozn at redhat.com
Fri Aug 8 12:26:41 UTC 2014
On 08.08.2014 14:08, Paolo Bonzini wrote:
> Il 08/08/2014 12:17, Michal Privoznik ha scritto:
>> + if (loader->nvram) {
>> + virBufferFreeAndReset(&buf);
>> + virBufferAsprintf(&buf,
>> + "file=%s,if=pflash,format=raw,unit=1",
>> + loader->nvram);
>> +
>> + virCommandAddArg(cmd, "-drive");
>> + virCommandAddArgBuffer(cmd, &buf);
>> + }
>
> Note that other machines may not need unit=1, for example pseries
> doesn't need it (it uses -bios for the firmware, not -drive if=pflash).
> It would be nice to make this easily configurable.
Is there a table anywhere where I can see what machine types support
this? Or does it makes sense to enable this only for
qemu-system-{x86_64,i386} -M pc and error out for all other combinations?
>
> Alternatively you could use unit=1 if there is a <loader type='pflash'>
> element, and unit=0 otherwise. We can then patch QEMU to reject unit=1
> on machines that use -bios + -drive if=pflash,unit=0.
>
> Laszlo, what happens on x86 with -bios + -drive if=pflash,unit=0?
>
> Paolo
>
Michal
More information about the libvir-list
mailing list