Re: [Qemu-devel] [PATCH] SMBIOS: Update Type 0 struct generator for machines >= 2.1

[Gabriel L. Somlo]

On Tue, May 13, 2014 at 05:16:24PM +0200, Laszlo Ersek wrote:
> The idea and the implementation in this patch seems fine to me (and
> thanks for it!), except I object to the conversion of
> "bios_characteristics" to uint64_t. I think that will break when you
> emulate eg. an x86_64 target (ie. an SMBIOS-consuming, little endian
> guest) on a big endian host (where you produce the SMBIOS payload).
> 
> If you back out the changes to "bios_characteristics", I'll add my R-b.

Would it be acceptable if I used

        t->bios_characteristics = cpu_to_le64(0x08);

instead ? The smbios spec pdf does say "QWORD", after all :)

If that's a bad idea for some other reason I haven't figured
out yet, I have no problem backing it out...

Thanks,
--Gabriel

PS Now that you mention emulating x86_64 on a BE machine, I think
there may actually be a few more places that require cpu_to_le*()
wrappers...