On Thu, Jan 17, 2019 at 12:48:22PM +0000, Peter Maydell wrote:
> On Tue, 15 Jan 2019 at 20:03, Michael S. Tsirkin <m...@redhat.com> wrote:
> >
> > Changes from v1:
> > - fix an out of tree build by unmangling a patch I mangled when applying
> > - fix a gcc 8 build by applying philippe's patches
> >
> >
> > The following changes since commit 89bd861c2b470e3fb45596945509079c72af3ac2:
> >
> > Merge remote-tracking branch
> > 'remotes/ehabkost/tags/x86-next-pull-request' into staging (2019-01-14
> > 17:35:00 +0000)
> >
> > are available in the Git repository at:
> >
> > git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream
> >
> > for you to fetch changes up to 64ed76db175f7ccb5f47195a7617d0106bf01245:
> >
> > migration: Use strnlen() for fixed-size string (2019-01-15 14:56:57 -0500)
> >
> > ----------------------------------------------------------------
> > pci, pc, virtio: fixes, features
> >
> > tpm physical presence interface
> > rsc support in virtio net
> > ivshmem is removed
> > misc cleanups and fixes all over the place
> >
> > Signed-off-by: Michael S. Tsirkin <m...@redhat.com>
> >
> This fails 'make check' on s390x host:
>
> MALLOC_PERTURB_=${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))}
> QTEST_QEMU_BINARY=i386-softmmu/qemu-system-i386
> QTEST_QEMU_IMG=qemu-img tests/bios-tables-test -m=quick -k --tap <
> /dev/null | ./scripts/tap-driver.pl --test-name="bios-tables-test"
>
> Looking for expected file 'tests/data/acpi/pc/FACP'
> Using expected file 'tests/data/acpi/pc/FACP'
> **
> ERROR:/home/linux1/qemu/tests/bios-tables-test.c:314:load_expected_aml:
> assertion failed: (exp_sdt.aml_len)
> ERROR - too few tests run (expected 15, got 0)
> Aborted (core dumped)
> /home/linux1/qemu/tests/Makefile.include:856: recipe for target
> 'check-qtest-i386' failed
>
>
> (all the other build hosts passed)
>
> thanks
> -- PMM
Oh. I am pretty sure it's endian-ness :(
Any chance you can quickly test the below?
That would be appreciated ...
diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c
index 0bf7164590..a506dcbb29 100644
--- a/tests/bios-tables-test.c
+++ b/tests/bios-tables-test.c
@@ -274,6 +274,7 @@ static GArray *load_expected_aml(test_data *data)
AcpiSdtTable *sdt;
GError *error = NULL;
gboolean ret;
+ gsize aml_len;
GArray *exp_tables = g_array_new(false, true, sizeof(AcpiSdtTable));
if (getenv("V")) {
@@ -307,7 +308,8 @@ try_again:
fprintf(stderr, "Using expected file '%s'\n", aml_file);
}
ret = g_file_get_contents(aml_file, (gchar **)&exp_sdt.aml,
- (gsize *)&exp_sdt.aml_len, &error);
+ &aml_len, &error);
+ exp_sdt.aml_len = aml_len;
g_assert(ret);
g_assert_no_error(error);
g_assert(exp_sdt.aml);
