qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Nicholas Piggin" <npiggin@gmail.com>
To: "Philippe Mathieu-Daudé" <philmd@linaro.org>,
	"Alex Bennée" <alex.bennee@linaro.org>
Cc: <qemu-ppc@nongnu.org>, "Cleber Rosa" <crosa@redhat.com>,
	"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
	"Beraldo Leal" <bleal@redhat.com>, <qemu-devel@nongnu.org>
Subject: Re: [RFC PATCH 07/11] tests/avocado: Add ppc boot tests for non-free AIX images
Date: Wed, 11 Oct 2023 07:21:49 +1000	[thread overview]
Message-ID: <CW52OWKBHGW2.NCXUFO464ZE7@wheely> (raw)
In-Reply-To: <d854691d-e133-c8cd-a785-959720114100@linaro.org>

On Tue Oct 10, 2023 at 10:49 PM AEST, Philippe Mathieu-Daudé wrote:
> On 10/10/23 14:43, Alex Bennée wrote:
> > 
> > Nicholas Piggin <npiggin@gmail.com> writes:
> > 
> >> An AIX image can be provided by setting AIX_IMAGE environment
> >> variable when running avocado.
> >>
> >> It's questionable whether we should carry these in upstream QEMU.
> >> It's convenient to see how to run these things, but simple enough
> >> to maintain in out of tree branch. I just wanted to see opinions
> >> about it.
> > 
> > Yeah there is no point adding a test no one else can run. We already
> > have tests that utilise dead URLs that can only run if you happen to
> > have the image in the avocado cache which should arguably be removed.
>
> This isn't quite the same problem. This test image can possibly be
> shared among developers within the IBM realm.

Yeah, and other people can have AIX and MacOS legally by other means
(IIRC you could register an acount and get AIX, but that may have
changed).

We do have some people not at IBM who test AIX against upstram and
report bugs. Not too sure if they would find these avocado tests useful
though.

Thanks,
Nick


  reply	other threads:[~2023-10-10 21:22 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-10  7:52 [RFC PATCH 00/11] ppc: avocado test additions and new defaults Nicholas Piggin
2023-10-10  7:52 ` [RFC PATCH 01/11] tests/avocado: ppc add powernv10 boot_linux_console test Nicholas Piggin
2023-10-10  7:52 ` [RFC PATCH 02/11] tests/avocado: Add ppc pseries and powernv Hash MMU tests Nicholas Piggin
2023-10-16  8:40   ` Cédric Le Goater
2023-10-10  7:52 ` [RFC PATCH 03/11] tests/avocado: Add pseries KVM boot_linux test Nicholas Piggin
2023-10-10  7:52 ` [RFC PATCH 04/11] tests/avocado: ppc add hypervisor tests Nicholas Piggin
2023-10-16  6:39   ` Harsh Prateek Bora
2023-10-10  7:52 ` [RFC PATCH 05/11] testing/avocado: ppc add new BookE boot_linux_console.py tests Nicholas Piggin
2023-10-10  8:02   ` Philippe Mathieu-Daudé
2023-10-10 12:03   ` Joel Stanley
2023-10-10 20:53     ` Nicholas Piggin
2023-10-16  8:07       ` Cédric Le Goater
2023-10-16  8:32   ` Cédric Le Goater
2023-10-10  7:52 ` [RFC PATCH 06/11] tests/avocado: Add FreeBSD distro boot tests for ppc Nicholas Piggin
2023-10-10 21:55   ` Warner Losh
2023-10-11  0:36     ` Nicholas Piggin
2023-10-11  4:24       ` Warner Losh
2023-10-11  2:23     ` Nicholas Piggin
2023-10-11  4:25       ` Warner Losh
2023-10-10  7:52 ` [RFC PATCH 07/11] tests/avocado: Add ppc boot tests for non-free AIX images Nicholas Piggin
2023-10-10 12:43   ` Alex Bennée
2023-10-10 12:49     ` Philippe Mathieu-Daudé
2023-10-10 21:21       ` Nicholas Piggin [this message]
2023-10-10 13:01     ` Daniel P. Berrangé
2023-10-10 21:14       ` Nicholas Piggin
2023-10-10  7:52 ` [RFC PATCH 08/11] tests/avocado: Add ppc MacOS tests Nicholas Piggin
2023-10-10  7:58   ` Philippe Mathieu-Daudé
2023-10-11  0:25     ` Nicholas Piggin
2023-10-10  7:52 ` [RFC PATCH 09/11] tests/avocado: Use default CPU for pseries machine Nicholas Piggin
2023-10-10  7:52 ` [RFC PATCH 10/11] ppc/spapr: change pseries machine default to POWER10 CPU Nicholas Piggin
2023-10-10 12:08   ` Joel Stanley
2023-10-16  8:16   ` Cédric Le Goater
2023-10-10  7:52 ` [RFC PATCH 11/11] ppc/pnv: Change powernv default to powernv10 Nicholas Piggin
2023-10-10 12:05   ` Joel Stanley
2023-10-10 20:57     ` Nicholas Piggin
2023-10-16  1:59       ` Joel Stanley
2023-10-16  8:15   ` Cédric Le Goater
2023-10-10  7:59 ` [RFC PATCH 00/11] ppc: avocado test additions and new defaults Philippe Mathieu-Daudé

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=CW52OWKBHGW2.NCXUFO464ZE7@wheely \
    --to=npiggin@gmail.com \
    --cc=alex.bennee@linaro.org \
    --cc=bleal@redhat.com \
    --cc=crosa@redhat.com \
    --cc=philmd@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=wainersm@redhat.com \
    /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).