Re: [PATCH] tests/qtest: Fix boot-serial-test when using --without-default-devices

[BALATON Zoltan]

On Mon, 19 Feb 2024, Thomas Huth wrote:
> If "configure" has been run with "--without-default-devices", there is
> no e1000 device in the binaries, so the boot-serial-test currently fails
> in that case since it tries to use the e1000 with the sam460ex machine.
>
> Since we're testing the serial output here, and not the NIC, let's
> simply switch to the "pci-bridge" device here instead, which should
> always be there for PCIe-based machines like the sam460ex.

It's not actually testing PCIe but PCI bus but I think that does not 
matter. PCIe on sam460ex does not work yet, I've only implemented it 
partially to pass the firmware init but devices attached to the PCIe bus 
probably won't work. I have some patches to improve that but not yet 
ready.

> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> tests/qtest/boot-serial-test.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/qtest/boot-serial-test.c b/tests/qtest/boot-serial-test.c
> index 6dd06aeaf4..e3b7d65fe5 100644
> --- a/tests/qtest/boot-serial-test.c
> +++ b/tests/qtest/boot-serial-test.c
> @@ -156,7 +156,7 @@ static const testdef_t tests[] = {
>       "Open Firmware" },
>     { "ppc64", "powernv8", "", "OPAL" },
>     { "ppc64", "powernv9", "", "OPAL" },
> -    { "ppc64", "sam460ex", "-device e1000", "8086  100e" },
> +    { "ppc64", "sam460ex", "-device pci-bridge,chassis_nr=2", "1b36  0001" },

So if you want to check if PCI bus works then maybe there's no need to add 
a device at all just look for the sm501 display chip ("126f 0501") that's 
soldered on the board so it's always created even with -nodefaults and 
should always present on sam460ex. The -device option just adds a device 
that appears before the sm501 and stops the test there. Not sure if this 
is testing more than looking for a PCI device created by the board code.

Regards,
BALATON Zoltan

>     { "i386", "isapc", "-cpu qemu32 -M graphics=off", "SeaBIOS" },
>     { "i386", "pc", "-M graphics=off", "SeaBIOS" },
>     { "i386", "q35", "-M graphics=off", "SeaBIOS" },