On 29/04/2016 14:44, Igor Mammedov wrote: > 'make check' fails with: > > ERROR:tests/bios-tables-test.c:493:load_expected_aml: > assertion failed: (g_file_test(aml_file, G_FILE_TEST_EXISTS)) > > since commit: > caf50c7166a6ed96c462ab5db4b495e1234e4cc6 > tests: pc: acpi: drop not needed 'expected SSDT' blobs > > Assert happens because qemu-system-x86_64 generates > SSDT table and test looks for a corresponding expected > table to compare with. > > However there is no expected SSDT blob anymore, since > QEMU souldn't generate one. As it happens BIOS is not > able to read ACPI tables from QEMU and fallbacks to > embeded legacy ACPI codepath, which generates SSDT. > That happens due to wrongly sized endiannes conversion > which makes > uint8_t BiosLinkerLoaderEntry.alloc.zone > end up with 0 due to truncation of 32 bit integer > which on host is 1 or 2. > > Fix it by dropping invalid cpu_to_le32() as uint8_t > doesn't require any conversion. > > RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1330174 > > Signed-off-by: Igor Mammedov <[email protected]>
Tested-by: Laurent Vivier <[email protected]> Fix the problem. We have always some warnings but they were already here in the previous releases. Laurent
