From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:44412)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <thuth@redhat.com>) id 1bumaW-0000bQ-Q3
	for qemu-devel@nongnu.org; Thu, 13 Oct 2016 16:26:49 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <thuth@redhat.com>) id 1bumaQ-00087y-K9
	for qemu-devel@nongnu.org; Thu, 13 Oct 2016 16:26:48 -0400
References: <1476388410-4187-1-git-send-email-lvivier@redhat.com>
From: Thomas Huth <thuth@redhat.com>
Message-ID: <dc58ae15-eb7a-f6c3-5690-4493d1cbfa14@redhat.com>
Date: Thu, 13 Oct 2016 22:26:37 +0200
MIME-Version: 1.0
In-Reply-To: <1476388410-4187-1-git-send-email-lvivier@redhat.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH] tests: add mac99 and g3beige in
 boot-serial-test
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel/>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Laurent Vivier <lvivier@redhat.com>, Alexander Graf <agraf@suse.de>
Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org

On 13.10.2016 21:53, Laurent Vivier wrote:
> g3beige (pmac_oldworld) and mac99 (pmac_newworld) are missing in
> boot-serial-test.
>=20
> Perhaps because serial output of OpenBIOS is only enabled with
> '-nographic'

IIRC clearly, I've left them out because they are basically already
tested with the prom-env test. I was a little bit afraid that the
testing time would become too long, but since this test is quite fast,
and it makes sense to check whether the serial output is working, too, I
think it's also OK if you add them here.

Two thoughts though:

1) I think you do *not* need the "-nographic" here, because the test is
using the "-serial" parameter to get the output of the serial console.

2) While you're at it, you could enable the test for a sparc and a
sparc64 machine, too (preferably one that is not tested by the prom-env
test yet)

 Thomas

> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
>  tests/boot-serial-test.c | 4 ++++
>  1 file changed, 4 insertions(+)
>=20
> diff --git a/tests/boot-serial-test.c b/tests/boot-serial-test.c
> index d98c564..d477a6a 100644
> --- a/tests/boot-serial-test.c
> +++ b/tests/boot-serial-test.c
> @@ -26,8 +26,12 @@ static testdef_t tests[] =3D {
>      { "alpha", "clipper", "", "PCI:" },
>      { "ppc", "ppce500", "", "U-Boot" },
>      { "ppc", "prep", "", "Open Hack'Ware BIOS" },
> +    { "ppc", "mac99", "-nographic", "OpenBIOS" },
> +    { "ppc", "g3beige", "-nographic", "OpenBIOS" },
>      { "ppc64", "ppce500", "", "U-Boot" },
>      { "ppc64", "prep", "", "Open Hack'Ware BIOS" },
> +    { "ppc64", "mac99", "-nographic", "OpenBIOS" },
> +    { "ppc64", "g3beige", "-nographic", "OpenBIOS" },
>      { "ppc64", "pseries", "", "Open Firmware" },
>      { "i386", "isapc", "-cpu qemu32 -device sga", "SGABIOS" },
>      { "i386", "pc", "-device sga", "SGABIOS" },
>=20