On 18.04.2018 08:02, Gerd Hoffmann wrote:
On Wed, Apr 18, 2018 at 12:40:54AM +0200, Laszlo Ersek wrote:
Add a schema that describes the different uses and properties of virtual machine firmware.
Looks good to me overall.
+{ 'enum' : 'FirmwareType', + 'data' : [ 'bios', 'slof', 'uboot', 'uefi' ] }
openbios missing.
As mentioned somewhere else in this thread, I think it's best to just use "openfirmware" here instead of "slof" and "openbios"
+{ 'enum' : 'FirmwareArchitecture', + 'data' : [ 'aarch64', 'arm', 'i386', 'x86_64' ] }
ppc(64) missing (but you have slof above ;) ...
Since you distinguish between 32-bit and 64-bit, we need both "ppc" (32-bit) and "ppc64" (64-bit) here.
s390 too.
Please name that "s390x". "s390" without x means the old 31-bit architecture which is hardly used anymore today. "s390x" is the modern 64-bit architecture. Thomas