From: Daniel Henrique Barboza <danielhb413@gmail.com>
To: BALATON Zoltan <balaton@eik.bme.hu>,
qemu-devel@nongnu.org, qemu-ppc@nongnu.org
Cc: Nicholas Piggin <npiggin@gmail.com>,
clg@kaod.org, philmd@linaro.org,
Bernhard Beschow <shentey@gmail.com>,
Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>,
Rene Engel <ReneEngel80@emailn.de>,
vr_qemu@t-online.de
Subject: Re: [PATCH v7 0/3] Add emulation of AmigaOne XE board
Date: Tue, 7 Nov 2023 14:20:27 -0300 [thread overview]
Message-ID: <697ad2e0-cb23-4efe-89e5-d1b521c0648f@gmail.com> (raw)
In-Reply-To: <cover.1698406922.git.balaton@eik.bme.hu>
Zoltan,
Gitlab is complaining about a missing file in one of the tests:
8/259 qemu:qtest+qtest-ppc / qtest-ppc/test-hmp ERROR 0.22s killed by signal 6 SIGABRT
4324>>> G_TEST_DBUS_DAEMON=/builds/danielhb/qemu/tests/dbus-vmstate-daemon.sh QTEST_QEMU_BINARY=./qemu-system-ppc MALLOC_PERTURB_=87 PYTHON=/builds/danielhb/qemu/build/pyvenv/bin/python3 /builds/danielhb/qemu/build/tests/qtest/test-hmp --tap -k
4325――――――――――――――――――――――――――――――――――――― ✀ ―――――――――――――――――――――――――――――――――――――
4326stderr:
4327qemu-system-ppc: Could not find firmware 'u-boot-amigaone.bin'
4328Broken pipe
4329../tests/qtest/libqtest.c:195: kill_qemu() tried to terminate QEMU process but encountered exit status 1 (expected 0)
4330(test program exited with status code -6)
4331TAP parsing error: Too few tests run (expected 13, got 0)
You can reproduce it like this:
$ make -j -C build && QTEST_QEMU_BINARY=./build/qemu-system-ppc64 ./build/tests/qtest/test-hmp
I ended up amending in-tree (downloaded the firmware, put it under pc-bios, updated pc-bios/meson.build).
My manual test now passes, but not sure if gitlab will nag about it. Let's wait and see.
I told you: code freeze is a blast! Let's see if it's still sunny for the
AmigaOne XE board emulation.
Thanks,
Daniel
On 10/27/23 08:54, BALATON Zoltan wrote:
> Changes in v7:
> - Increase default memory size to 512m to match pegasos2 and sam460ex
> and it's a better default for AmigaOS
>
> Changes in v6:
> - Dropped patch 1, now it's
>
> Based-on: <20231024224056.842607-1-mark.cave-ayland@ilande.co.uk>
>
> ([PATCH v2 0/3] ide: implement simple legacy/native mode switching for PCI IDE controllers)
> - Added Tested-by from Rene
>
> Changes in v5:
> - Fixed avocado test
>
> Changes in v4:
> - Found typo in comment in patch 1 so ended up rewording it again
> trying to make it more concise. Also take the idea of using
> range_covers_byte from Mark's patch
> - Added RFC patch for avocado test (untested, I don't have Avocado)
>
> Changes in v3:
> - Update values, comment and commit message in patch 1 again
>
> Changes in v2:
> - Update comment and commit message in patch 1 (Mark)
> - Fix irq mapping in patch 2 (Volker)
>
> Regards,
> BALATON Zoltan
>
> BALATON Zoltan (3):
> hw/pci-host: Add emulation of Mai Logic Articia S
> hw/ppc: Add emulation of AmigaOne XE board
> tests/avocado: Add test for amigaone board
>
> MAINTAINERS | 8 +
> configs/devices/ppc-softmmu/default.mak | 1 +
> hw/pci-host/Kconfig | 5 +
> hw/pci-host/articia.c | 293 ++++++++++++++++++++++++
> hw/pci-host/meson.build | 2 +
> hw/ppc/Kconfig | 7 +
> hw/ppc/amigaone.c | 164 +++++++++++++
> hw/ppc/meson.build | 2 +
> include/hw/pci-host/articia.h | 17 ++
> tests/avocado/ppc_amiga.py | 38 +++
> 10 files changed, 537 insertions(+)
> create mode 100644 hw/pci-host/articia.c
> create mode 100644 hw/ppc/amigaone.c
> create mode 100644 include/hw/pci-host/articia.h
> create mode 100644 tests/avocado/ppc_amiga.py
>
next prev parent reply other threads:[~2023-11-07 17:21 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-27 11:54 [PATCH v7 0/3] Add emulation of AmigaOne XE board BALATON Zoltan
2023-10-27 11:54 ` [PATCH v7 1/3] hw/pci-host: Add emulation of Mai Logic Articia S BALATON Zoltan
2023-11-07 18:53 ` Daniel Henrique Barboza
2023-10-27 11:54 ` [PATCH v7 2/3] hw/ppc: Add emulation of AmigaOne XE board BALATON Zoltan
2023-11-07 18:21 ` [PATCH v8 " BALATON Zoltan
2023-11-07 18:25 ` Peter Maydell
2023-11-07 18:28 ` BALATON Zoltan
2023-11-07 18:40 ` [PATCH v9 " BALATON Zoltan
2023-11-07 18:52 ` Daniel Henrique Barboza
2023-10-27 11:54 ` [PATCH v7 3/3] tests/avocado: Add test for amigaone board BALATON Zoltan
2023-11-07 18:53 ` Daniel Henrique Barboza
2023-10-28 14:52 ` [PATCH v7 0/3] Add emulation of AmigaOne XE board Bernhard Beschow
2023-10-28 18:20 ` BALATON Zoltan
2023-11-06 10:30 ` BALATON Zoltan
2023-11-07 8:50 ` BALATON Zoltan
2023-11-07 14:42 ` BALATON Zoltan
2023-11-07 16:20 ` Daniel Henrique Barboza
2023-11-07 16:41 ` BALATON Zoltan
2023-11-07 17:20 ` Daniel Henrique Barboza [this message]
2023-11-07 17:33 ` BALATON Zoltan
2023-11-07 17:42 ` Daniel Henrique Barboza
2023-11-07 17:49 ` BALATON Zoltan
2023-11-07 18:03 ` BALATON Zoltan
2023-11-07 18:14 ` Peter Maydell
2023-11-07 18:18 ` BALATON Zoltan
2023-11-07 18:21 ` Peter Maydell
2023-11-07 18:25 ` BALATON Zoltan
2023-11-07 18:41 ` BALATON Zoltan
2023-11-07 17:44 ` BALATON Zoltan
2023-11-07 18:53 ` Daniel Henrique Barboza
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=697ad2e0-cb23-4efe-89e5-d1b521c0648f@gmail.com \
--to=danielhb413@gmail.com \
--cc=ReneEngel80@emailn.de \
--cc=balaton@eik.bme.hu \
--cc=clg@kaod.org \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=npiggin@gmail.com \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=shentey@gmail.com \
--cc=vr_qemu@t-online.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).