qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines
@ 2021-01-12 16:40 Thomas Huth
  2021-01-12 16:40 ` [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file Thomas Huth
                   ` (3 more replies)
  0 siblings, 4 replies; 19+ messages in thread
From: Thomas Huth @ 2021-01-12 16:40 UTC (permalink / raw)
  To: qemu-devel, David Gibson, Greg Kurz
  Cc: Philippe Mathieu-Daudé,
	qemu-ppc, Wainer dos Santos Moschetta, Cleber Rosa

Let's use the two new images from the QEMU advent calendar 2020 to test
the mpc8544ds & virtex-ml507 ppc machines.

Thomas Huth (3):
  tests/acceptance: Move the pseries test to a separate file
  tests/acceptance: Test the mpc8544ds machine
  tests/acceptance: Add a test for the virtex-ml507 ppc machine

 MAINTAINERS                            |  1 +
 tests/acceptance/boot_linux_console.py | 19 -------
 tests/acceptance/machine_ppc.py        | 69 ++++++++++++++++++++++++++
 3 files changed, 70 insertions(+), 19 deletions(-)
 create mode 100644 tests/acceptance/machine_ppc.py

-- 
2.27.0



^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file
  2021-01-12 16:40 [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines Thomas Huth
@ 2021-01-12 16:40 ` Thomas Huth
  2021-01-12 18:50   ` Wainer dos Santos Moschetta
                     ` (3 more replies)
  2021-01-12 16:40 ` [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine Thomas Huth
                   ` (2 subsequent siblings)
  3 siblings, 4 replies; 19+ messages in thread
From: Thomas Huth @ 2021-01-12 16:40 UTC (permalink / raw)
  To: qemu-devel, David Gibson, Greg Kurz
  Cc: Philippe Mathieu-Daudé,
	qemu-ppc, Wainer dos Santos Moschetta, Cleber Rosa

Let's gather the POWER-related tests in a separate file.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS                            |  1 +
 tests/acceptance/boot_linux_console.py | 19 --------------
 tests/acceptance/machine_ppc.py        | 34 ++++++++++++++++++++++++++
 3 files changed, 35 insertions(+), 19 deletions(-)
 create mode 100644 tests/acceptance/machine_ppc.py

diff --git a/MAINTAINERS b/MAINTAINERS
index 4be087b88e..189776a036 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -280,6 +280,7 @@ F: target/ppc/
 F: hw/ppc/
 F: include/hw/ppc/
 F: disas/ppc.c
+F: tests/acceptance/machine_ppc.py
 
 RISC-V TCG CPUs
 M: Palmer Dabbelt <palmer@dabbelt.com>
diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py
index fb41bb7144..41d2c86e98 100644
--- a/tests/acceptance/boot_linux_console.py
+++ b/tests/acceptance/boot_linux_console.py
@@ -976,25 +976,6 @@ class BootLinuxConsole(LinuxKernelTest):
         console_pattern = 'Kernel command line: %s' % kernel_command_line
         self.wait_for_console_pattern(console_pattern)
 
-    def test_ppc64_pseries(self):
-        """
-        :avocado: tags=arch:ppc64
-        :avocado: tags=machine:pseries
-        """
-        kernel_url = ('https://archives.fedoraproject.org/pub/archive'
-                      '/fedora-secondary/releases/29/Everything/ppc64le/os'
-                      '/ppc/ppc64/vmlinuz')
-        kernel_hash = '3fe04abfc852b66653b8c3c897a59a689270bc77'
-        kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
-
-        self.vm.set_console()
-        kernel_command_line = self.KERNEL_COMMON_COMMAND_LINE + 'console=hvc0'
-        self.vm.add_args('-kernel', kernel_path,
-                         '-append', kernel_command_line)
-        self.vm.launch()
-        console_pattern = 'Kernel command line: %s' % kernel_command_line
-        self.wait_for_console_pattern(console_pattern)
-
     def test_m68k_q800(self):
         """
         :avocado: tags=arch:m68k
diff --git a/tests/acceptance/machine_ppc.py b/tests/acceptance/machine_ppc.py
new file mode 100644
index 0000000000..51bbfd411c
--- /dev/null
+++ b/tests/acceptance/machine_ppc.py
@@ -0,0 +1,34 @@
+# Test that Linux kernel boots on ppc machines and check the console
+#
+# Copyright (c) 2018, 2020 Red Hat, Inc.
+#
+# This work is licensed under the terms of the GNU GPL, version 2 or
+# later.  See the COPYING file in the top-level directory.
+
+from avocado_qemu import Test
+from avocado_qemu import wait_for_console_pattern
+
+class PpcMachine(Test):
+
+    timeout = 90
+    KERNEL_COMMON_COMMAND_LINE = 'printk.time=0 '
+    panic_message = 'Kernel panic - not syncing'
+
+    def test_ppc64_pseries(self):
+        """
+        :avocado: tags=arch:ppc64
+        :avocado: tags=machine:pseries
+        """
+        kernel_url = ('https://archives.fedoraproject.org/pub/archive'
+                      '/fedora-secondary/releases/29/Everything/ppc64le/os'
+                      '/ppc/ppc64/vmlinuz')
+        kernel_hash = '3fe04abfc852b66653b8c3c897a59a689270bc77'
+        kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
+
+        self.vm.set_console()
+        kernel_command_line = self.KERNEL_COMMON_COMMAND_LINE + 'console=hvc0'
+        self.vm.add_args('-kernel', kernel_path,
+                         '-append', kernel_command_line)
+        self.vm.launch()
+        console_pattern = 'Kernel command line: %s' % kernel_command_line
+        wait_for_console_pattern(self, console_pattern, self.panic_message)
-- 
2.27.0



^ permalink raw reply related	[flat|nested] 19+ messages in thread

* [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine
  2021-01-12 16:40 [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines Thomas Huth
  2021-01-12 16:40 ` [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file Thomas Huth
@ 2021-01-12 16:40 ` Thomas Huth
  2021-01-12 18:51   ` Wainer dos Santos Moschetta
                     ` (2 more replies)
  2021-01-12 16:40 ` [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine Thomas Huth
  2021-02-04 14:08 ` [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines Philippe Mathieu-Daudé
  3 siblings, 3 replies; 19+ messages in thread
From: Thomas Huth @ 2021-01-12 16:40 UTC (permalink / raw)
  To: qemu-devel, David Gibson, Greg Kurz
  Cc: Philippe Mathieu-Daudé,
	qemu-ppc, Wainer dos Santos Moschetta, Cleber Rosa

We can use the "Stupid creek" image to test the mpc8544ds ppc machine.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 tests/acceptance/machine_ppc.py | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/tests/acceptance/machine_ppc.py b/tests/acceptance/machine_ppc.py
index 51bbfd411c..71025d296c 100644
--- a/tests/acceptance/machine_ppc.py
+++ b/tests/acceptance/machine_ppc.py
@@ -5,6 +5,7 @@
 # This work is licensed under the terms of the GNU GPL, version 2 or
 # later.  See the COPYING file in the top-level directory.
 
+from avocado.utils import archive
 from avocado_qemu import Test
 from avocado_qemu import wait_for_console_pattern
 
@@ -32,3 +33,19 @@ class PpcMachine(Test):
         self.vm.launch()
         console_pattern = 'Kernel command line: %s' % kernel_command_line
         wait_for_console_pattern(self, console_pattern, self.panic_message)
+
+    def test_ppc_mpc8544ds(self):
+        """
+        :avocado: tags=arch:ppc
+        :avocado: tags=machine:mpc8544ds
+        """
+        tar_url = ('https://www.qemu-advent-calendar.org'
+                   '/2020/download/day17.tar.gz')
+        tar_hash = '7a5239542a7c4257aa4d3b7f6ddf08fb6775c494'
+        file_path = self.fetch_asset(tar_url, asset_hash=tar_hash)
+        archive.extract(file_path, self.workdir)
+        self.vm.set_console()
+        self.vm.add_args('-kernel', self.workdir + '/creek/creek.bin')
+        self.vm.launch()
+        wait_for_console_pattern(self, 'QEMU advent calendar 2020',
+                                 self.panic_message)
-- 
2.27.0



^ permalink raw reply related	[flat|nested] 19+ messages in thread

* [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine
  2021-01-12 16:40 [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines Thomas Huth
  2021-01-12 16:40 ` [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file Thomas Huth
  2021-01-12 16:40 ` [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine Thomas Huth
@ 2021-01-12 16:40 ` Thomas Huth
  2021-01-12 18:51   ` Wainer dos Santos Moschetta
                     ` (2 more replies)
  2021-02-04 14:08 ` [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines Philippe Mathieu-Daudé
  3 siblings, 3 replies; 19+ messages in thread
From: Thomas Huth @ 2021-01-12 16:40 UTC (permalink / raw)
  To: qemu-devel, David Gibson, Greg Kurz
  Cc: Philippe Mathieu-Daudé,
	qemu-ppc, Wainer dos Santos Moschetta, Cleber Rosa

The "And a hippo new year" image from the QEMU advent calendar 2020
can be used to test the virtex-ml507 ppc machine.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 tests/acceptance/machine_ppc.py | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/tests/acceptance/machine_ppc.py b/tests/acceptance/machine_ppc.py
index 71025d296c..a836e2496f 100644
--- a/tests/acceptance/machine_ppc.py
+++ b/tests/acceptance/machine_ppc.py
@@ -49,3 +49,21 @@ class PpcMachine(Test):
         self.vm.launch()
         wait_for_console_pattern(self, 'QEMU advent calendar 2020',
                                  self.panic_message)
+
+    def test_ppc_virtex_ml507(self):
+        """
+        :avocado: tags=arch:ppc
+        :avocado: tags=machine:virtex-ml507
+        """
+        tar_url = ('https://www.qemu-advent-calendar.org'
+                   '/2020/download/hippo.tar.gz')
+        tar_hash = '306b95bfe7d147f125aa176a877e266db8ef914a'
+        file_path = self.fetch_asset(tar_url, asset_hash=tar_hash)
+        archive.extract(file_path, self.workdir)
+        self.vm.set_console()
+        self.vm.add_args('-kernel', self.workdir + '/hippo/hippo.linux',
+                         '-dtb', self.workdir + '/hippo/virtex440-ml507.dtb',
+                         '-m', '512')
+        self.vm.launch()
+        wait_for_console_pattern(self, 'QEMU advent calendar 2020',
+                                 self.panic_message)
-- 
2.27.0



^ permalink raw reply related	[flat|nested] 19+ messages in thread

* Re: [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file
  2021-01-12 16:40 ` [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file Thomas Huth
@ 2021-01-12 18:50   ` Wainer dos Santos Moschetta
  2021-01-13  5:30     ` Thomas Huth
  2021-01-12 20:36   ` Philippe Mathieu-Daudé
                     ` (2 subsequent siblings)
  3 siblings, 1 reply; 19+ messages in thread
From: Wainer dos Santos Moschetta @ 2021-01-12 18:50 UTC (permalink / raw)
  To: Thomas Huth, qemu-devel, David Gibson, Greg Kurz
  Cc: Philippe Mathieu-Daudé, qemu-ppc, Cleber Rosa

Hi,

On 1/12/21 1:40 PM, Thomas Huth wrote:
> Let's gather the POWER-related tests in a separate file.


Did you consider having others ppc/ppc64 boot tests together too?

Some candidates:

tests/acceptance/boot_linux.py:BootLinuxPPC64.test_pseries_tcg
tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc64_e500
tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_g3beige
tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_mac99
tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_factory_firmware_and_netbsd
tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_192m
tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_and_netbsd

>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   MAINTAINERS                            |  1 +
>   tests/acceptance/boot_linux_console.py | 19 --------------
>   tests/acceptance/machine_ppc.py        | 34 ++++++++++++++++++++++++++
>   3 files changed, 35 insertions(+), 19 deletions(-)
>   create mode 100644 tests/acceptance/machine_ppc.py


Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>


>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4be087b88e..189776a036 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -280,6 +280,7 @@ F: target/ppc/
>   F: hw/ppc/
>   F: include/hw/ppc/
>   F: disas/ppc.c
> +F: tests/acceptance/machine_ppc.py
>   
>   RISC-V TCG CPUs
>   M: Palmer Dabbelt <palmer@dabbelt.com>
> diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py
> index fb41bb7144..41d2c86e98 100644
> --- a/tests/acceptance/boot_linux_console.py
> +++ b/tests/acceptance/boot_linux_console.py
> @@ -976,25 +976,6 @@ class BootLinuxConsole(LinuxKernelTest):
>           console_pattern = 'Kernel command line: %s' % kernel_command_line
>           self.wait_for_console_pattern(console_pattern)
>   
> -    def test_ppc64_pseries(self):
> -        """
> -        :avocado: tags=arch:ppc64
> -        :avocado: tags=machine:pseries
> -        """
> -        kernel_url = ('https://archives.fedoraproject.org/pub/archive'
> -                      '/fedora-secondary/releases/29/Everything/ppc64le/os'
> -                      '/ppc/ppc64/vmlinuz')
> -        kernel_hash = '3fe04abfc852b66653b8c3c897a59a689270bc77'
> -        kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
> -
> -        self.vm.set_console()
> -        kernel_command_line = self.KERNEL_COMMON_COMMAND_LINE + 'console=hvc0'
> -        self.vm.add_args('-kernel', kernel_path,
> -                         '-append', kernel_command_line)
> -        self.vm.launch()
> -        console_pattern = 'Kernel command line: %s' % kernel_command_line
> -        self.wait_for_console_pattern(console_pattern)
> -
>       def test_m68k_q800(self):
>           """
>           :avocado: tags=arch:m68k
> diff --git a/tests/acceptance/machine_ppc.py b/tests/acceptance/machine_ppc.py
> new file mode 100644
> index 0000000000..51bbfd411c
> --- /dev/null
> +++ b/tests/acceptance/machine_ppc.py
> @@ -0,0 +1,34 @@
> +# Test that Linux kernel boots on ppc machines and check the console
> +#
> +# Copyright (c) 2018, 2020 Red Hat, Inc.
> +#
> +# This work is licensed under the terms of the GNU GPL, version 2 or
> +# later.  See the COPYING file in the top-level directory.
> +
> +from avocado_qemu import Test
> +from avocado_qemu import wait_for_console_pattern
> +
> +class PpcMachine(Test):
> +
> +    timeout = 90
> +    KERNEL_COMMON_COMMAND_LINE = 'printk.time=0 '
> +    panic_message = 'Kernel panic - not syncing'
> +
> +    def test_ppc64_pseries(self):
> +        """
> +        :avocado: tags=arch:ppc64
> +        :avocado: tags=machine:pseries
> +        """
> +        kernel_url = ('https://archives.fedoraproject.org/pub/archive'
> +                      '/fedora-secondary/releases/29/Everything/ppc64le/os'
> +                      '/ppc/ppc64/vmlinuz')
> +        kernel_hash = '3fe04abfc852b66653b8c3c897a59a689270bc77'
> +        kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
> +
> +        self.vm.set_console()
> +        kernel_command_line = self.KERNEL_COMMON_COMMAND_LINE + 'console=hvc0'
> +        self.vm.add_args('-kernel', kernel_path,
> +                         '-append', kernel_command_line)
> +        self.vm.launch()
> +        console_pattern = 'Kernel command line: %s' % kernel_command_line
> +        wait_for_console_pattern(self, console_pattern, self.panic_message)



^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine
  2021-01-12 16:40 ` [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine Thomas Huth
@ 2021-01-12 18:51   ` Wainer dos Santos Moschetta
  2021-01-13  1:14   ` David Gibson
  2021-01-15 17:19   ` Willian Rampazzo
  2 siblings, 0 replies; 19+ messages in thread
From: Wainer dos Santos Moschetta @ 2021-01-12 18:51 UTC (permalink / raw)
  To: Thomas Huth, qemu-devel, David Gibson, Greg Kurz
  Cc: Philippe Mathieu-Daudé, qemu-ppc, Cleber Rosa


On 1/12/21 1:40 PM, Thomas Huth wrote:
> We can use the "Stupid creek" image to test the mpc8544ds ppc machine.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   tests/acceptance/machine_ppc.py | 17 +++++++++++++++++
>   1 file changed, 17 insertions(+)


Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>


>
> diff --git a/tests/acceptance/machine_ppc.py b/tests/acceptance/machine_ppc.py
> index 51bbfd411c..71025d296c 100644
> --- a/tests/acceptance/machine_ppc.py
> +++ b/tests/acceptance/machine_ppc.py
> @@ -5,6 +5,7 @@
>   # This work is licensed under the terms of the GNU GPL, version 2 or
>   # later.  See the COPYING file in the top-level directory.
>   
> +from avocado.utils import archive
>   from avocado_qemu import Test
>   from avocado_qemu import wait_for_console_pattern
>   
> @@ -32,3 +33,19 @@ class PpcMachine(Test):
>           self.vm.launch()
>           console_pattern = 'Kernel command line: %s' % kernel_command_line
>           wait_for_console_pattern(self, console_pattern, self.panic_message)
> +
> +    def test_ppc_mpc8544ds(self):
> +        """
> +        :avocado: tags=arch:ppc
> +        :avocado: tags=machine:mpc8544ds
> +        """
> +        tar_url = ('https://www.qemu-advent-calendar.org'
> +                   '/2020/download/day17.tar.gz')
> +        tar_hash = '7a5239542a7c4257aa4d3b7f6ddf08fb6775c494'
> +        file_path = self.fetch_asset(tar_url, asset_hash=tar_hash)
> +        archive.extract(file_path, self.workdir)
> +        self.vm.set_console()
> +        self.vm.add_args('-kernel', self.workdir + '/creek/creek.bin')
> +        self.vm.launch()
> +        wait_for_console_pattern(self, 'QEMU advent calendar 2020',
> +                                 self.panic_message)



^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine
  2021-01-12 16:40 ` [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine Thomas Huth
@ 2021-01-12 18:51   ` Wainer dos Santos Moschetta
  2021-01-13  1:15   ` David Gibson
  2021-01-15 17:20   ` Willian Rampazzo
  2 siblings, 0 replies; 19+ messages in thread
From: Wainer dos Santos Moschetta @ 2021-01-12 18:51 UTC (permalink / raw)
  To: Thomas Huth, qemu-devel, David Gibson, Greg Kurz
  Cc: qemu-ppc, Philippe Mathieu-Daudé, Cleber Rosa


On 1/12/21 1:40 PM, Thomas Huth wrote:
> The "And a hippo new year" image from the QEMU advent calendar 2020
> can be used to test the virtex-ml507 ppc machine.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   tests/acceptance/machine_ppc.py | 18 ++++++++++++++++++
>   1 file changed, 18 insertions(+)


Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>


>
> diff --git a/tests/acceptance/machine_ppc.py b/tests/acceptance/machine_ppc.py
> index 71025d296c..a836e2496f 100644
> --- a/tests/acceptance/machine_ppc.py
> +++ b/tests/acceptance/machine_ppc.py
> @@ -49,3 +49,21 @@ class PpcMachine(Test):
>           self.vm.launch()
>           wait_for_console_pattern(self, 'QEMU advent calendar 2020',
>                                    self.panic_message)
> +
> +    def test_ppc_virtex_ml507(self):
> +        """
> +        :avocado: tags=arch:ppc
> +        :avocado: tags=machine:virtex-ml507
> +        """
> +        tar_url = ('https://www.qemu-advent-calendar.org'
> +                   '/2020/download/hippo.tar.gz')
> +        tar_hash = '306b95bfe7d147f125aa176a877e266db8ef914a'
> +        file_path = self.fetch_asset(tar_url, asset_hash=tar_hash)
> +        archive.extract(file_path, self.workdir)
> +        self.vm.set_console()
> +        self.vm.add_args('-kernel', self.workdir + '/hippo/hippo.linux',
> +                         '-dtb', self.workdir + '/hippo/virtex440-ml507.dtb',
> +                         '-m', '512')
> +        self.vm.launch()
> +        wait_for_console_pattern(self, 'QEMU advent calendar 2020',
> +                                 self.panic_message)



^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file
  2021-01-12 16:40 ` [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file Thomas Huth
  2021-01-12 18:50   ` Wainer dos Santos Moschetta
@ 2021-01-12 20:36   ` Philippe Mathieu-Daudé
  2021-01-13  1:14   ` David Gibson
  2021-01-14 20:17   ` Willian Rampazzo
  3 siblings, 0 replies; 19+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-01-12 20:36 UTC (permalink / raw)
  To: Thomas Huth, qemu-devel, David Gibson, Greg Kurz
  Cc: qemu-ppc, Wainer dos Santos Moschetta, Cleber Rosa

On 1/12/21 5:40 PM, Thomas Huth wrote:
> Let's gather the POWER-related tests in a separate file.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  MAINTAINERS                            |  1 +
>  tests/acceptance/boot_linux_console.py | 19 --------------
>  tests/acceptance/machine_ppc.py        | 34 ++++++++++++++++++++++++++
>  3 files changed, 35 insertions(+), 19 deletions(-)
>  create mode 100644 tests/acceptance/machine_ppc.py

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>



^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file
  2021-01-12 16:40 ` [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file Thomas Huth
  2021-01-12 18:50   ` Wainer dos Santos Moschetta
  2021-01-12 20:36   ` Philippe Mathieu-Daudé
@ 2021-01-13  1:14   ` David Gibson
  2021-01-14 20:17   ` Willian Rampazzo
  3 siblings, 0 replies; 19+ messages in thread
From: David Gibson @ 2021-01-13  1:14 UTC (permalink / raw)
  To: Thomas Huth
  Cc: qemu-devel, Wainer dos Santos Moschetta, Greg Kurz, qemu-ppc,
	Cleber Rosa, Philippe Mathieu-Daudé

[-- Attachment #1: Type: text/plain, Size: 4168 bytes --]

On Tue, Jan 12, 2021 at 05:40:43PM +0100, Thomas Huth wrote:
> Let's gather the POWER-related tests in a separate file.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Acked-by: David Gibson <david@gibson.dropbear.id.au>

> ---
>  MAINTAINERS                            |  1 +
>  tests/acceptance/boot_linux_console.py | 19 --------------
>  tests/acceptance/machine_ppc.py        | 34 ++++++++++++++++++++++++++
>  3 files changed, 35 insertions(+), 19 deletions(-)
>  create mode 100644 tests/acceptance/machine_ppc.py
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 4be087b88e..189776a036 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -280,6 +280,7 @@ F: target/ppc/
>  F: hw/ppc/
>  F: include/hw/ppc/
>  F: disas/ppc.c
> +F: tests/acceptance/machine_ppc.py
>  
>  RISC-V TCG CPUs
>  M: Palmer Dabbelt <palmer@dabbelt.com>
> diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py
> index fb41bb7144..41d2c86e98 100644
> --- a/tests/acceptance/boot_linux_console.py
> +++ b/tests/acceptance/boot_linux_console.py
> @@ -976,25 +976,6 @@ class BootLinuxConsole(LinuxKernelTest):
>          console_pattern = 'Kernel command line: %s' % kernel_command_line
>          self.wait_for_console_pattern(console_pattern)
>  
> -    def test_ppc64_pseries(self):
> -        """
> -        :avocado: tags=arch:ppc64
> -        :avocado: tags=machine:pseries
> -        """
> -        kernel_url = ('https://archives.fedoraproject.org/pub/archive'
> -                      '/fedora-secondary/releases/29/Everything/ppc64le/os'
> -                      '/ppc/ppc64/vmlinuz')
> -        kernel_hash = '3fe04abfc852b66653b8c3c897a59a689270bc77'
> -        kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
> -
> -        self.vm.set_console()
> -        kernel_command_line = self.KERNEL_COMMON_COMMAND_LINE + 'console=hvc0'
> -        self.vm.add_args('-kernel', kernel_path,
> -                         '-append', kernel_command_line)
> -        self.vm.launch()
> -        console_pattern = 'Kernel command line: %s' % kernel_command_line
> -        self.wait_for_console_pattern(console_pattern)
> -
>      def test_m68k_q800(self):
>          """
>          :avocado: tags=arch:m68k
> diff --git a/tests/acceptance/machine_ppc.py b/tests/acceptance/machine_ppc.py
> new file mode 100644
> index 0000000000..51bbfd411c
> --- /dev/null
> +++ b/tests/acceptance/machine_ppc.py
> @@ -0,0 +1,34 @@
> +# Test that Linux kernel boots on ppc machines and check the console
> +#
> +# Copyright (c) 2018, 2020 Red Hat, Inc.
> +#
> +# This work is licensed under the terms of the GNU GPL, version 2 or
> +# later.  See the COPYING file in the top-level directory.
> +
> +from avocado_qemu import Test
> +from avocado_qemu import wait_for_console_pattern
> +
> +class PpcMachine(Test):
> +
> +    timeout = 90
> +    KERNEL_COMMON_COMMAND_LINE = 'printk.time=0 '
> +    panic_message = 'Kernel panic - not syncing'
> +
> +    def test_ppc64_pseries(self):
> +        """
> +        :avocado: tags=arch:ppc64
> +        :avocado: tags=machine:pseries
> +        """
> +        kernel_url = ('https://archives.fedoraproject.org/pub/archive'
> +                      '/fedora-secondary/releases/29/Everything/ppc64le/os'
> +                      '/ppc/ppc64/vmlinuz')
> +        kernel_hash = '3fe04abfc852b66653b8c3c897a59a689270bc77'
> +        kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
> +
> +        self.vm.set_console()
> +        kernel_command_line = self.KERNEL_COMMON_COMMAND_LINE + 'console=hvc0'
> +        self.vm.add_args('-kernel', kernel_path,
> +                         '-append', kernel_command_line)
> +        self.vm.launch()
> +        console_pattern = 'Kernel command line: %s' % kernel_command_line
> +        wait_for_console_pattern(self, console_pattern, self.panic_message)

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine
  2021-01-12 16:40 ` [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine Thomas Huth
  2021-01-12 18:51   ` Wainer dos Santos Moschetta
@ 2021-01-13  1:14   ` David Gibson
  2021-01-15 17:19   ` Willian Rampazzo
  2 siblings, 0 replies; 19+ messages in thread
From: David Gibson @ 2021-01-13  1:14 UTC (permalink / raw)
  To: Thomas Huth, @yekko.fritz.box
  Cc: qemu-devel, Wainer dos Santos Moschetta, Greg Kurz, qemu-ppc,
	Cleber Rosa, Philippe Mathieu-Daudé

[-- Attachment #1: Type: text/plain, Size: 2006 bytes --]

On Tue, Jan 12, 2021 at 05:40:44PM +0100, Thomas Huth wrote:
> We can use the "Stupid creek" image to test the mpc8544ds ppc machine.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Acked-by: David Gibson <david@gibson.dropbear.id.au>

> ---
>  tests/acceptance/machine_ppc.py | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/tests/acceptance/machine_ppc.py b/tests/acceptance/machine_ppc.py
> index 51bbfd411c..71025d296c 100644
> --- a/tests/acceptance/machine_ppc.py
> +++ b/tests/acceptance/machine_ppc.py
> @@ -5,6 +5,7 @@
>  # This work is licensed under the terms of the GNU GPL, version 2 or
>  # later.  See the COPYING file in the top-level directory.
>  
> +from avocado.utils import archive
>  from avocado_qemu import Test
>  from avocado_qemu import wait_for_console_pattern
>  
> @@ -32,3 +33,19 @@ class PpcMachine(Test):
>          self.vm.launch()
>          console_pattern = 'Kernel command line: %s' % kernel_command_line
>          wait_for_console_pattern(self, console_pattern, self.panic_message)
> +
> +    def test_ppc_mpc8544ds(self):
> +        """
> +        :avocado: tags=arch:ppc
> +        :avocado: tags=machine:mpc8544ds
> +        """
> +        tar_url = ('https://www.qemu-advent-calendar.org'
> +                   '/2020/download/day17.tar.gz')
> +        tar_hash = '7a5239542a7c4257aa4d3b7f6ddf08fb6775c494'
> +        file_path = self.fetch_asset(tar_url, asset_hash=tar_hash)
> +        archive.extract(file_path, self.workdir)
> +        self.vm.set_console()
> +        self.vm.add_args('-kernel', self.workdir + '/creek/creek.bin')
> +        self.vm.launch()
> +        wait_for_console_pattern(self, 'QEMU advent calendar 2020',
> +                                 self.panic_message)

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine
  2021-01-12 16:40 ` [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine Thomas Huth
  2021-01-12 18:51   ` Wainer dos Santos Moschetta
@ 2021-01-13  1:15   ` David Gibson
  2021-01-15 17:20   ` Willian Rampazzo
  2 siblings, 0 replies; 19+ messages in thread
From: David Gibson @ 2021-01-13  1:15 UTC (permalink / raw)
  To: Thomas Huth
  Cc: qemu-devel, Wainer dos Santos Moschetta, Greg Kurz, qemu-ppc,
	Cleber Rosa, Philippe Mathieu-Daudé

[-- Attachment #1: Type: text/plain, Size: 1869 bytes --]

On Tue, Jan 12, 2021 at 05:40:45PM +0100, Thomas Huth wrote:
> The "And a hippo new year" image from the QEMU advent calendar 2020
> can be used to test the virtex-ml507 ppc machine.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Acked-by: David Gibson <david@gibson.dropbear.id.au>

> ---
>  tests/acceptance/machine_ppc.py | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
> 
> diff --git a/tests/acceptance/machine_ppc.py b/tests/acceptance/machine_ppc.py
> index 71025d296c..a836e2496f 100644
> --- a/tests/acceptance/machine_ppc.py
> +++ b/tests/acceptance/machine_ppc.py
> @@ -49,3 +49,21 @@ class PpcMachine(Test):
>          self.vm.launch()
>          wait_for_console_pattern(self, 'QEMU advent calendar 2020',
>                                   self.panic_message)
> +
> +    def test_ppc_virtex_ml507(self):
> +        """
> +        :avocado: tags=arch:ppc
> +        :avocado: tags=machine:virtex-ml507
> +        """
> +        tar_url = ('https://www.qemu-advent-calendar.org'
> +                   '/2020/download/hippo.tar.gz')
> +        tar_hash = '306b95bfe7d147f125aa176a877e266db8ef914a'
> +        file_path = self.fetch_asset(tar_url, asset_hash=tar_hash)
> +        archive.extract(file_path, self.workdir)
> +        self.vm.set_console()
> +        self.vm.add_args('-kernel', self.workdir + '/hippo/hippo.linux',
> +                         '-dtb', self.workdir + '/hippo/virtex440-ml507.dtb',
> +                         '-m', '512')
> +        self.vm.launch()
> +        wait_for_console_pattern(self, 'QEMU advent calendar 2020',
> +                                 self.panic_message)

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file
  2021-01-12 18:50   ` Wainer dos Santos Moschetta
@ 2021-01-13  5:30     ` Thomas Huth
  2021-02-04  8:52       ` Philippe Mathieu-Daudé
  0 siblings, 1 reply; 19+ messages in thread
From: Thomas Huth @ 2021-01-13  5:30 UTC (permalink / raw)
  To: Wainer dos Santos Moschetta, qemu-devel, David Gibson, Greg Kurz,
	Philippe Mathieu-Daudé
  Cc: Philippe Mathieu-Daudé, qemu-ppc, Cleber Rosa

On 12/01/2021 19.50, Wainer dos Santos Moschetta wrote:
> Hi,
> 
> On 1/12/21 1:40 PM, Thomas Huth wrote:
>> Let's gather the POWER-related tests in a separate file.
> 
> 
> Did you consider having others ppc/ppc64 boot tests together too?
> 
> Some candidates:
> 
> tests/acceptance/boot_linux.py:BootLinuxPPC64.test_pseries_tcg
> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc64_e500
> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_g3beige
> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_mac99

The e500, g3beige and mac99 tests are depending on the do_test_advcal_2018() 
function in that file, so I think they should rather stay there.

> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_factory_firmware_and_netbsd 
> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_192m
> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_and_netbsd

That's a good point, I did not notice that file when writing my patches. 
Philippe, since you've created this ppc_prep_40p.py file, what do you think, 
should it be merged with the other ppc tests, or shall we rather keep this 
separate?

  Thomas



^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file
  2021-01-12 16:40 ` [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file Thomas Huth
                     ` (2 preceding siblings ...)
  2021-01-13  1:14   ` David Gibson
@ 2021-01-14 20:17   ` Willian Rampazzo
  3 siblings, 0 replies; 19+ messages in thread
From: Willian Rampazzo @ 2021-01-14 20:17 UTC (permalink / raw)
  To: Thomas Huth
  Cc: Greg Kurz, Wainer dos Santos Moschetta, qemu-devel, qemu-ppc,
	Cleber Rosa, Philippe Mathieu-Daudé,
	David Gibson

On Tue, Jan 12, 2021 at 1:42 PM Thomas Huth <thuth@redhat.com> wrote:
>
> Let's gather the POWER-related tests in a separate file.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  MAINTAINERS                            |  1 +
>  tests/acceptance/boot_linux_console.py | 19 --------------
>  tests/acceptance/machine_ppc.py        | 34 ++++++++++++++++++++++++++
>  3 files changed, 35 insertions(+), 19 deletions(-)
>  create mode 100644 tests/acceptance/machine_ppc.py
>

We can have other tests moved to this file later. For now,

Reviewed-by: Willian Rampazzo <willianr@redhat.com>



^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine
  2021-01-12 16:40 ` [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine Thomas Huth
  2021-01-12 18:51   ` Wainer dos Santos Moschetta
  2021-01-13  1:14   ` David Gibson
@ 2021-01-15 17:19   ` Willian Rampazzo
  2 siblings, 0 replies; 19+ messages in thread
From: Willian Rampazzo @ 2021-01-15 17:19 UTC (permalink / raw)
  To: Thomas Huth
  Cc: Greg Kurz, Wainer dos Santos Moschetta, qemu-devel, qemu-ppc,
	Cleber Rosa, Philippe Mathieu-Daudé,
	David Gibson

On Tue, Jan 12, 2021 at 1:44 PM Thomas Huth <thuth@redhat.com> wrote:
>
> We can use the "Stupid creek" image to test the mpc8544ds ppc machine.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  tests/acceptance/machine_ppc.py | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)

Reviewed-by: Willian Rampazzo <willianr@redhat.com>



^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine
  2021-01-12 16:40 ` [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine Thomas Huth
  2021-01-12 18:51   ` Wainer dos Santos Moschetta
  2021-01-13  1:15   ` David Gibson
@ 2021-01-15 17:20   ` Willian Rampazzo
  2 siblings, 0 replies; 19+ messages in thread
From: Willian Rampazzo @ 2021-01-15 17:20 UTC (permalink / raw)
  To: Thomas Huth
  Cc: Greg Kurz, Wainer dos Santos Moschetta, qemu-devel, qemu-ppc,
	Cleber Rosa, Philippe Mathieu-Daudé,
	David Gibson

On Tue, Jan 12, 2021 at 1:45 PM Thomas Huth <thuth@redhat.com> wrote:
>
> The "And a hippo new year" image from the QEMU advent calendar 2020
> can be used to test the virtex-ml507 ppc machine.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  tests/acceptance/machine_ppc.py | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)

Reviewed-by: Willian Rampazzo <willianr@redhat.com>



^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file
  2021-01-13  5:30     ` Thomas Huth
@ 2021-02-04  8:52       ` Philippe Mathieu-Daudé
  2021-02-04 12:59         ` Thomas Huth
  2021-02-08 19:57         ` Willian Rampazzo
  0 siblings, 2 replies; 19+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-02-04  8:52 UTC (permalink / raw)
  To: Thomas Huth, Wainer dos Santos Moschetta, qemu-devel,
	David Gibson, Greg Kurz
  Cc: Philippe Mathieu-Daudé, qemu-ppc, Willian Rampazzo, Cleber Rosa

Hi Thomas,

On 1/13/21 6:30 AM, Thomas Huth wrote:
> On 12/01/2021 19.50, Wainer dos Santos Moschetta wrote:
>> Hi,
>>
>> On 1/12/21 1:40 PM, Thomas Huth wrote:
>>> Let's gather the POWER-related tests in a separate file.
>>
>>
>> Did you consider having others ppc/ppc64 boot tests together too?
>>
>> Some candidates:
>>
>> tests/acceptance/boot_linux.py:BootLinuxPPC64.test_pseries_tcg
>> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc64_e500
>> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_g3beige
>> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_mac99
> 
> The e500, g3beige and mac99 tests are depending on the
> do_test_advcal_2018() function in that file, so I think they should
> rather stay there.
> 
>> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_factory_firmware_and_netbsd
>> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_192m
>> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_and_netbsd
>>
> 
> That's a good point, I did not notice that file when writing my patches.
> Philippe, since you've created this ppc_prep_40p.py file, what do you
> think, should it be merged with the other ppc tests, or shall we rather
> keep this separate?

The choice was deliberate: the PReP machine has a different set of
maintainers. If possible when we have a particular section in
MAINTAINERS I'd like to use it as separation, to let the maintainers
track changes in tests.

In this example, Hervé is interested to look for PReP related files,
but doesn't have bandwidth to look at all PPC patches.

If this doesn't scale, I suggested (was it on the list or directly
to Willian?) to add a Python script to map Avocado test tags to
MAINTAINERS entry, so 1/ maintainers could run all tests linked to
their subsystem by naming the subsystem, and 2/ when a test fails
we know which maintainer to contact.

Regards,

Phil.


^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file
  2021-02-04  8:52       ` Philippe Mathieu-Daudé
@ 2021-02-04 12:59         ` Thomas Huth
  2021-02-08 19:57         ` Willian Rampazzo
  1 sibling, 0 replies; 19+ messages in thread
From: Thomas Huth @ 2021-02-04 12:59 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé,
	Wainer dos Santos Moschetta, qemu-devel, David Gibson, Greg Kurz
  Cc: qemu-ppc, Philippe Mathieu-Daudé, Willian Rampazzo, Cleber Rosa

On 04/02/2021 09.52, Philippe Mathieu-Daudé wrote:
> Hi Thomas,
> 
> On 1/13/21 6:30 AM, Thomas Huth wrote:
>> On 12/01/2021 19.50, Wainer dos Santos Moschetta wrote:
>>> Hi,
>>>
>>> On 1/12/21 1:40 PM, Thomas Huth wrote:
>>>> Let's gather the POWER-related tests in a separate file.
>>>
>>>
>>> Did you consider having others ppc/ppc64 boot tests together too?
>>>
>>> Some candidates:
>>>
>>> tests/acceptance/boot_linux.py:BootLinuxPPC64.test_pseries_tcg
>>> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc64_e500
>>> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_g3beige
>>> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_mac99
>>
>> The e500, g3beige and mac99 tests are depending on the
>> do_test_advcal_2018() function in that file, so I think they should
>> rather stay there.
>>
>>> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_factory_firmware_and_netbsd
>>> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_192m
>>> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_and_netbsd
>>>
>>
>> That's a good point, I did not notice that file when writing my patches.
>> Philippe, since you've created this ppc_prep_40p.py file, what do you
>> think, should it be merged with the other ppc tests, or shall we rather
>> keep this separate?
> 
> The choice was deliberate: the PReP machine has a different set of
> maintainers. If possible when we have a particular section in
> MAINTAINERS I'd like to use it as separation, to let the maintainers
> track changes in tests.

Ok, fair point, let's keep it separate then!

  Thomas



^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines
  2021-01-12 16:40 [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines Thomas Huth
                   ` (2 preceding siblings ...)
  2021-01-12 16:40 ` [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine Thomas Huth
@ 2021-02-04 14:08 ` Philippe Mathieu-Daudé
  3 siblings, 0 replies; 19+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-02-04 14:08 UTC (permalink / raw)
  To: Thomas Huth, qemu-devel, David Gibson, Greg Kurz
  Cc: qemu-ppc, Wainer dos Santos Moschetta, Cleber Rosa

On 1/12/21 5:40 PM, Thomas Huth wrote:
> Let's use the two new images from the QEMU advent calendar 2020 to test
> the mpc8544ds & virtex-ml507 ppc machines.
> 
> Thomas Huth (3):
>   tests/acceptance: Move the pseries test to a separate file
>   tests/acceptance: Test the mpc8544ds machine
>   tests/acceptance: Add a test for the virtex-ml507 ppc machine
> 
>  MAINTAINERS                            |  1 +
>  tests/acceptance/boot_linux_console.py | 19 -------
>  tests/acceptance/machine_ppc.py        | 69 ++++++++++++++++++++++++++
>  3 files changed, 70 insertions(+), 19 deletions(-)
>  create mode 100644 tests/acceptance/machine_ppc.py

Thanks, series applied to my acceptance-testing tree.


^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file
  2021-02-04  8:52       ` Philippe Mathieu-Daudé
  2021-02-04 12:59         ` Thomas Huth
@ 2021-02-08 19:57         ` Willian Rampazzo
  1 sibling, 0 replies; 19+ messages in thread
From: Willian Rampazzo @ 2021-02-08 19:57 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé,
	Thomas Huth, Wainer dos Santos Moschetta, qemu-devel,
	David Gibson, Greg Kurz
  Cc: Philippe Mathieu-Daudé, qemu-ppc, Cleber Rosa



On 2/4/21 5:52 AM, Philippe Mathieu-Daudé wrote:
> Hi Thomas,
> 
> On 1/13/21 6:30 AM, Thomas Huth wrote:
>> On 12/01/2021 19.50, Wainer dos Santos Moschetta wrote:
>>> Hi,
>>>
>>> On 1/12/21 1:40 PM, Thomas Huth wrote:
>>>> Let's gather the POWER-related tests in a separate file.
>>>
>>>
>>> Did you consider having others ppc/ppc64 boot tests together too?
>>>
>>> Some candidates:
>>>
>>> tests/acceptance/boot_linux.py:BootLinuxPPC64.test_pseries_tcg
>>> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc64_e500
>>> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_g3beige
>>> tests/acceptance/boot_linux_console.py:BootLinuxConsole.test_ppc_mac99
>>
>> The e500, g3beige and mac99 tests are depending on the
>> do_test_advcal_2018() function in that file, so I think they should
>> rather stay there.
>>
>>> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_factory_firmware_and_netbsd
>>> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_192m
>>> tests/acceptance/ppc_prep_40p.py:IbmPrep40pMachine.test_openbios_and_netbsd
>>>
>>
>> That's a good point, I did not notice that file when writing my patches.
>> Philippe, since you've created this ppc_prep_40p.py file, what do you
>> think, should it be merged with the other ppc tests, or shall we rather
>> keep this separate?
> 
> The choice was deliberate: the PReP machine has a different set of
> maintainers. If possible when we have a particular section in
> MAINTAINERS I'd like to use it as separation, to let the maintainers
> track changes in tests.
> 
> In this example, Hervé is interested to look for PReP related files,
> but doesn't have bandwidth to look at all PPC patches.
> 
> If this doesn't scale, I suggested (was it on the list or directly
> to Willian?) 

I remember we had this conversation some time ago, so, I'm sure you 
suggested to me, not sure if you mentioned that to the list.

> to add a Python script to map Avocado test tags to
> MAINTAINERS entry, so 1/ maintainers could run all tests linked to
> their subsystem by naming the subsystem, and 2/ when a test fails
> we know which maintainer to contact.

And I agree that is an interesting idea.

> 
> Regards,
> 
> Phil.
> 



^ permalink raw reply	[flat|nested] 19+ messages in thread

end of thread, other threads:[~2021-02-08 22:47 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-12 16:40 [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines Thomas Huth
2021-01-12 16:40 ` [PATCH 1/3] tests/acceptance: Move the pseries test to a separate file Thomas Huth
2021-01-12 18:50   ` Wainer dos Santos Moschetta
2021-01-13  5:30     ` Thomas Huth
2021-02-04  8:52       ` Philippe Mathieu-Daudé
2021-02-04 12:59         ` Thomas Huth
2021-02-08 19:57         ` Willian Rampazzo
2021-01-12 20:36   ` Philippe Mathieu-Daudé
2021-01-13  1:14   ` David Gibson
2021-01-14 20:17   ` Willian Rampazzo
2021-01-12 16:40 ` [PATCH 2/3] tests/acceptance: Test the mpc8544ds machine Thomas Huth
2021-01-12 18:51   ` Wainer dos Santos Moschetta
2021-01-13  1:14   ` David Gibson
2021-01-15 17:19   ` Willian Rampazzo
2021-01-12 16:40 ` [PATCH 3/3] tests/acceptance: Add a test for the virtex-ml507 ppc machine Thomas Huth
2021-01-12 18:51   ` Wainer dos Santos Moschetta
2021-01-13  1:15   ` David Gibson
2021-01-15 17:20   ` Willian Rampazzo
2021-02-04 14:08 ` [PATCH 0/3] Acceptance tests for mpc8544ds & virtex-ml507 ppc machines Philippe Mathieu-Daudé

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).