qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tests/acceptance: Test powernv machines


From: Wainer dos Santos Moschetta
Subject: Re: [PATCH] tests/acceptance: Test powernv machines
Date: Mon, 16 Aug 2021 14:09:04 -0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0

Hi Cédric,

On 8/11/21 11:54 AM, Cédric Le Goater wrote:
On 8/11/21 11:07 AM, Thomas Huth wrote:
On 10/08/2021 11.09, Cédric Le Goater wrote:
On 8/10/21 10:36 AM, Joel Stanley wrote:
On Tue, 10 Aug 2021 at 08:34, Cédric Le Goater <clg@kaod.org> wrote:
Fetch the OpenPOWER images to boot the powernv8 and powernv9 machines
with a simple PCI layout.

Cc: Cleber Rosa <crosa@redhat.com>
Cc: Wainer dos Santos Moschetta <wainersm@redhat.com>
Signed-off-by: Cédric Le Goater <clg@kaod.org>
---
   tests/acceptance/boot_linux_console.py | 42 ++++++++++++++++++++++++++
   1 file changed, 42 insertions(+)

diff --git a/tests/acceptance/boot_linux_console.py 
b/tests/acceptance/boot_linux_console.py
index 5248c8097df9..da93a475ca87 100644
--- a/tests/acceptance/boot_linux_console.py
+++ b/tests/acceptance/boot_linux_console.py
@@ -1176,6 +1176,48 @@ def test_ppc64_e500(self):
           tar_hash = '6951d86d644b302898da2fd701739c9406527fe1'
           self.do_test_advcal_2018('19', tar_hash, 'uImage')

+    def do_test_ppc64_powernv(self, proc):
+
+        images_url = 
('https://github.com/open-power/op-build/releases/download/v2.7/')
+
+        skiboot_url = images_url + 'skiboot.lid'
+        skiboot_hash = 
'a9ffcddbf238f86cda4b2cae2882d6bd13cff8489109758a4980efaf154f4a29'
+        skiboot_path = self.fetch_asset(skiboot_url, asset_hash=skiboot_hash,
+                                       algorithm='sha256')
What's the thought that led you to using this instead of the one that
gets packaged with qemu?
Good question.

I considered that the skiboot.lid shipped with QEMU was somewhat a default
to make things work. The official released versions are the ones used by
the outside world on real systems and were a better target for tests.

That said, using the default version might be enough. Maintainers, please
advise !
IMHO:

- We want to test the things that *we* ship

- We want to download as few things as possible, since downloads
   often slow down the tests and break CI runs if the network to
   the download site is not available

  ==> I'd prefer to use the internal skiboot.lid unless there is
      really a compelling reason to use the external one.
OK. I changed the test to use the internal skiboot.lid.

The code of patch v1 looks good to me but I will wait the v2 to put my reviewed-by stamp.

Thanks!

- Wainer


Thanks,

C.









reply via email to

[Prev in Thread] Current Thread [Next in Thread]