diff mbox series

tests/avocado: Remove non-working sparc leon3 test

Message ID 20240710111755.60584-1-thuth@redhat.com (mailing list archive)
State New, archived
Headers show
Series tests/avocado: Remove non-working sparc leon3 test | expand

Commit Message

Thomas Huth July 10, 2024, 11:17 a.m. UTC
The test has been marked as broken more than 4 years ago, and
so far nobody ever cared to fix it. Thus let's simply remove it
now ... if somebody ever needs it again, they can restore the
file from an older version of QEMU.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 tests/avocado/machine_sparc_leon3.py | 37 ----------------------------
 1 file changed, 37 deletions(-)
 delete mode 100644 tests/avocado/machine_sparc_leon3.py

Comments

Alex Bennée July 10, 2024, 3:25 p.m. UTC | #1
Thomas Huth <thuth@redhat.com> writes:

> The test has been marked as broken more than 4 years ago, and
> so far nobody ever cared to fix it. Thus let's simply remove it
> now ... if somebody ever needs it again, they can restore the
> file from an older version of QEMU.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Acked-by: Alex Bennée <alex.bennee@linaro.org>
Philippe Mathieu-Daudé July 10, 2024, 5:01 p.m. UTC | #2
+ Clément & Frederic as Leon3 maintainers and SPARC team

On 10/7/24 13:17, Thomas Huth wrote:
> The test has been marked as broken more than 4 years ago, and
> so far nobody ever cared to fix it. Thus let's simply remove it
> now ... if somebody ever needs it again, they can restore the
> file from an older version of QEMU.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   tests/avocado/machine_sparc_leon3.py | 37 ----------------------------
>   1 file changed, 37 deletions(-)
>   delete mode 100644 tests/avocado/machine_sparc_leon3.py
> 
> diff --git a/tests/avocado/machine_sparc_leon3.py b/tests/avocado/machine_sparc_leon3.py
> deleted file mode 100644
> index e61b223185..0000000000
> --- a/tests/avocado/machine_sparc_leon3.py
> +++ /dev/null
> @@ -1,37 +0,0 @@
> -# Functional test that boots a Leon3 machine and checks its serial console.
> -#
> -# Copyright (c) Philippe Mathieu-Daudé <f4bug@amsat.org>
> -#
> -# 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 QemuSystemTest
> -from avocado_qemu import wait_for_console_pattern
> -from avocado import skip
> -
> -
> -class Leon3Machine(QemuSystemTest):
> -
> -    timeout = 60
> -
> -    @skip("Test currently broken")
> -    # A Window Underflow exception occurs before booting the kernel,
> -    # and QEMU exit calling cpu_abort(), which makes this test to fail.

       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

> -    def test_leon3_helenos_uimage(self):
> -        """
> -        :avocado: tags=arch:sparc
> -        :avocado: tags=machine:leon3_generic
> -        :avocado: tags=binfmt:uimage
> -        """
> -        kernel_url = ('http://www.helenos.org/releases/'
> -                      'HelenOS-0.6.0-sparc32-leon3.bin')
> -        kernel_hash = 'a88c9cfdb8430c66650e5290a08765f9bf049a30'
> -        kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
> -
> -        self.vm.set_console()
> -        self.vm.add_args('-kernel', kernel_path)
> -
> -        self.vm.launch()
> -
> -        wait_for_console_pattern(self, 'Copyright (c) 2001-2014 HelenOS project')
> -        wait_for_console_pattern(self, 'Booting the kernel ...')
Clément Chigot July 11, 2024, 6:50 a.m. UTC | #3
On Wed, Jul 10, 2024 at 5:25 PM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> Thomas Huth <thuth@redhat.com> writes:
>
> > The test has been marked as broken more than 4 years ago, and
> > so far nobody ever cared to fix it. Thus let's simply remove it
> > now ... if somebody ever needs it again, they can restore the
> > file from an older version of QEMU.
> >
> > Signed-off-by: Thomas Huth <thuth@redhat.com>
>
> Acked-by: Alex Bennée <alex.bennee@linaro.org>

Yeah, it's been on my todo list for far too long but I still didn't
find time to work on it.

Reviewed-by: Clément Chigot <chigot@adacore.com>
Alex Bennée July 11, 2024, 1:47 p.m. UTC | #4
Thomas Huth <thuth@redhat.com> writes:

> The test has been marked as broken more than 4 years ago, and
> so far nobody ever cared to fix it. Thus let's simply remove it
> now ... if somebody ever needs it again, they can restore the
> file from an older version of QEMU.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Queued to testing/next, thanks.
diff mbox series

Patch

diff --git a/tests/avocado/machine_sparc_leon3.py b/tests/avocado/machine_sparc_leon3.py
deleted file mode 100644
index e61b223185..0000000000
--- a/tests/avocado/machine_sparc_leon3.py
+++ /dev/null
@@ -1,37 +0,0 @@ 
-# Functional test that boots a Leon3 machine and checks its serial console.
-#
-# Copyright (c) Philippe Mathieu-Daudé <f4bug@amsat.org>
-#
-# 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 QemuSystemTest
-from avocado_qemu import wait_for_console_pattern
-from avocado import skip
-
-
-class Leon3Machine(QemuSystemTest):
-
-    timeout = 60
-
-    @skip("Test currently broken")
-    # A Window Underflow exception occurs before booting the kernel,
-    # and QEMU exit calling cpu_abort(), which makes this test to fail.
-    def test_leon3_helenos_uimage(self):
-        """
-        :avocado: tags=arch:sparc
-        :avocado: tags=machine:leon3_generic
-        :avocado: tags=binfmt:uimage
-        """
-        kernel_url = ('http://www.helenos.org/releases/'
-                      'HelenOS-0.6.0-sparc32-leon3.bin')
-        kernel_hash = 'a88c9cfdb8430c66650e5290a08765f9bf049a30'
-        kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
-
-        self.vm.set_console()
-        self.vm.add_args('-kernel', kernel_path)
-
-        self.vm.launch()
-
-        wait_for_console_pattern(self, 'Copyright (c) 2001-2014 HelenOS project')
-        wait_for_console_pattern(self, 'Booting the kernel ...')