[PULL 0/1] Seabios 20191106 patches

Gerd Hoffmann posted 1 patch 4 years, 4 months ago
Test asan failed
Test checkpatch passed
Test FreeBSD passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu failed
Test docker-quick@centos7 failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20191106122524.5537-1-kraxel@redhat.com
pc-bios/bios-256k.bin             | Bin 262144 -> 262144 bytes
pc-bios/bios.bin                  | Bin 131072 -> 131072 bytes
pc-bios/vgabios-ati.bin           | Bin 38912 -> 39424 bytes
pc-bios/vgabios-bochs-display.bin | Bin 27648 -> 28160 bytes
pc-bios/vgabios-cirrus.bin        | Bin 38400 -> 39424 bytes
pc-bios/vgabios-qxl.bin           | Bin 38912 -> 39424 bytes
pc-bios/vgabios-ramfb.bin         | Bin 28160 -> 28672 bytes
pc-bios/vgabios-stdvga.bin        | Bin 38912 -> 39424 bytes
pc-bios/vgabios-virtio.bin        | Bin 38912 -> 39424 bytes
pc-bios/vgabios-vmware.bin        | Bin 38912 -> 39424 bytes
pc-bios/vgabios.bin               | Bin 38400 -> 38912 bytes
roms/Makefile                     |   2 +-
roms/seabios                      |   2 +-
13 files changed, 2 insertions(+), 2 deletions(-)
[PULL 0/1] Seabios 20191106 patches
Posted by Gerd Hoffmann 4 years, 4 months ago
The following changes since commit 36609b4fa36f0ac934874371874416f7533a5408:

  Merge remote-tracking branch 'remotes/palmer/tags/palmer-for-master-4.2-sf1' into staging (2019-11-02 17:59:03 +0000)

are available in the Git repository at:

  git://git.kraxel.org/qemu tags/seabios-20191106-pull-request

for you to fetch changes up to 58b16e57ded751e2e8be626124aad1d46a408a33:

  seabios: update to pre-1.13 snapshot (2019-11-06 13:23:02 +0100)

----------------------------------------------------------------
seabios: update to pre-1.13 snapshot

----------------------------------------------------------------

Gerd Hoffmann (1):
  seabios: update to pre-1.13 snapshot

 pc-bios/bios-256k.bin             | Bin 262144 -> 262144 bytes
 pc-bios/bios.bin                  | Bin 131072 -> 131072 bytes
 pc-bios/vgabios-ati.bin           | Bin 38912 -> 39424 bytes
 pc-bios/vgabios-bochs-display.bin | Bin 27648 -> 28160 bytes
 pc-bios/vgabios-cirrus.bin        | Bin 38400 -> 39424 bytes
 pc-bios/vgabios-qxl.bin           | Bin 38912 -> 39424 bytes
 pc-bios/vgabios-ramfb.bin         | Bin 28160 -> 28672 bytes
 pc-bios/vgabios-stdvga.bin        | Bin 38912 -> 39424 bytes
 pc-bios/vgabios-virtio.bin        | Bin 38912 -> 39424 bytes
 pc-bios/vgabios-vmware.bin        | Bin 38912 -> 39424 bytes
 pc-bios/vgabios.bin               | Bin 38400 -> 38912 bytes
 roms/Makefile                     |   2 +-
 roms/seabios                      |   2 +-
 13 files changed, 2 insertions(+), 2 deletions(-)

-- 
2.18.1


Re: [PULL 0/1] Seabios 20191106 patches
Posted by no-reply@patchew.org 4 years, 4 months ago
Patchew URL: https://patchew.org/QEMU/20191106122524.5537-1-kraxel@redhat.com/



Hi,

This series failed the docker-quick@centos7 build test. Please find the testing commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.

=== TEST SCRIPT BEGIN ===
#!/bin/bash
make docker-image-centos7 V=1 NETWORK=1
time make docker-test-quick@centos7 SHOW_ENV=1 J=14 NETWORK=1
=== TEST SCRIPT END ===

Passed all 108 iotests
  TEST    check-qtest-aarch64: tests/qos-test
**
ERROR:/tmp/qemu-test/src/tests/boot-sector.c:161:boot_sector_test: assertion failed (signature == SIGNATURE): (0x00000000 == 0x0000dead)
ERROR - Bail out! ERROR:/tmp/qemu-test/src/tests/boot-sector.c:161:boot_sector_test: assertion failed (signature == SIGNATURE): (0x00000000 == 0x0000dead)
make: *** [check-qtest-x86_64] Error 1
Traceback (most recent call last):
  File "./tests/docker/docker.py", line 662, in <module>
    sys.exit(main())
---
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['sudo', '-n', 'docker', 'run', '--label', 'com.qemu.instance.uuid=94fa7e2078d84908a9f9a7f2f442c434', '-u', '1003', '--security-opt', 'seccomp=unconfined', '--rm', '-e', 'TARGET_LIST=', '-e', 'EXTRA_CONFIGURE_OPTS=', '-e', 'V=', '-e', 'J=14', '-e', 'DEBUG=', '-e', 'SHOW_ENV=1', '-e', 'CCACHE_DIR=/var/tmp/ccache', '-v', '/home/patchew2/.cache/qemu-docker-ccache:/var/tmp/ccache:z', '-v', '/var/tmp/patchew-tester-tmp-qy60v805/src/docker-src.2019-11-06-14.28.30.29417:/var/tmp/qemu:z,ro', 'qemu:centos7', '/var/tmp/qemu/run', 'test-quick']' returned non-zero exit status 2.
filter=--filter=label=com.qemu.instance.uuid=94fa7e2078d84908a9f9a7f2f442c434
make[1]: *** [docker-run] Error 1
make[1]: Leaving directory `/var/tmp/patchew-tester-tmp-qy60v805/src'
make: *** [docker-run-test-quick@centos7] Error 2

real    14m4.054s
user    0m8.513s


The full log is available at
http://patchew.org/logs/20191106122524.5537-1-kraxel@redhat.com/testing.docker-quick@centos7/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
Re: [PULL 0/1] Seabios 20191106 patches
Posted by Peter Maydell 4 years, 4 months ago
On Wed, 6 Nov 2019 at 12:26, Gerd Hoffmann <kraxel@redhat.com> wrote:
>
> The following changes since commit 36609b4fa36f0ac934874371874416f7533a5408:
>
>   Merge remote-tracking branch 'remotes/palmer/tags/palmer-for-master-4.2-sf1' into staging (2019-11-02 17:59:03 +0000)
>
> are available in the Git repository at:
>
>   git://git.kraxel.org/qemu tags/seabios-20191106-pull-request
>
> for you to fetch changes up to 58b16e57ded751e2e8be626124aad1d46a408a33:
>
>   seabios: update to pre-1.13 snapshot (2019-11-06 13:23:02 +0100)
>
> ----------------------------------------------------------------
> seabios: update to pre-1.13 snapshot
>
> ----------------------------------------------------------------
>
> Gerd Hoffmann (1):
>   seabios: update to pre-1.13 snapshot

Hi; this fails 'make check' on at least
aarch64, aarch32, FreeBSD, NetBSD, s390:

ERROR:/home/linux1/qemu/tests/boot-sector.c:161:boot_sector_test:
assertion failed (signature == SIGNATURE): (0x00000000 == 0x0000dead)
ERROR - Bail out!
ERROR:/home/linux1/qemu/tests/boot-sector.c:161:boot_sector_test:
assertion failed (signature == SIGNATURE): (0x00000000 == 0x0000dead)
PASS 9 bios-tables-test /x86_64/acpi/q35/bridge
Aborted (core dumped)
/home/linux1/qemu/tests/Makefile.include:916: recipe for target
'check-qtest-i386' failed
make: *** [check-qtest-i386] Error 1

the x86-64 bootsector tests seem to fail similarly.

thanks
-- PMM

[SeaBIOS] Re: [PULL 0/1] Seabios 20191106 patches
Posted by Gerd Hoffmann 4 years, 4 months ago
On Thu, Nov 07, 2019 at 11:56:03AM +0000, Peter Maydell wrote:
> On Wed, 6 Nov 2019 at 12:26, Gerd Hoffmann <kraxel@redhat.com> wrote:
> >
> > The following changes since commit 36609b4fa36f0ac934874371874416f7533a5408:
> >
> >   Merge remote-tracking branch 'remotes/palmer/tags/palmer-for-master-4.2-sf1' into staging (2019-11-02 17:59:03 +0000)
> >
> > are available in the Git repository at:
> >
> >   git://git.kraxel.org/qemu tags/seabios-20191106-pull-request
> >
> > for you to fetch changes up to 58b16e57ded751e2e8be626124aad1d46a408a33:
> >
> >   seabios: update to pre-1.13 snapshot (2019-11-06 13:23:02 +0100)
> >
> > ----------------------------------------------------------------
> > seabios: update to pre-1.13 snapshot
> >
> > ----------------------------------------------------------------
> >
> > Gerd Hoffmann (1):
> >   seabios: update to pre-1.13 snapshot
> 
> Hi; this fails 'make check' on at least
> aarch64, aarch32, FreeBSD, NetBSD, s390:
> 
> ERROR:/home/linux1/qemu/tests/boot-sector.c:161:boot_sector_test:
> assertion failed (signature == SIGNATURE): (0x00000000 == 0x0000dead)
> ERROR - Bail out!
> ERROR:/home/linux1/qemu/tests/boot-sector.c:161:boot_sector_test:
> assertion failed (signature == SIGNATURE): (0x00000000 == 0x0000dead)
> PASS 9 bios-tables-test /x86_64/acpi/q35/bridge
> Aborted (core dumped)
> /home/linux1/qemu/tests/Makefile.include:916: recipe for target
> 'check-qtest-i386' failed
> make: *** [check-qtest-i386] Error 1
> 
> the x86-64 bootsector tests seem to fail similarly.

[ full quote for seabios list ]

Re-ran test on x86-64 box -> works.
Tried on aarch64 machine -> fails.

Given the arch list above this pretty much looks like it is tcg-related,
even though a quick check with "qemu -accel tcg -cdrom /some/live/iso"
(on x86_64) doesn't show any obvious problems.

Recompiled seabios with gcc 4.8 instead of gcc 8
  -> Works on both x86-64 and aarch64.
  -> I'll redo the pull request with that.

I'll go try find the root cause next week.  On a quick glance this
looks like a bug in tcg or gcc.  In case anyone has hints what might
have caused this drop me a note.

thanks,
  Gerd
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org