* [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine
@ 2019-10-10 12:21 Philippe Mathieu-Daudé
2019-10-10 12:21 ` [PATCH 1/2] tests/boot_console: Send <carriage return> on serial lines Philippe Mathieu-Daudé
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-10 12:21 UTC (permalink / raw)
To: qemu-devel
Cc: Eduardo Habkost, Helge Deller, Philippe Mathieu-Daudé,
Cleber Rosa, Sven Schnelle, Richard Henderson
This tests boot a HP-UX firmware CD-ROM which allow serial
console interaction. This exercise the PCI LSI53C895A SCSI
controller.
I'm not adding it to the Travis-CI list because I'm not sure
how to split/rename the current job, see:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg644753.html
Philippe Mathieu-Daudé (2):
tests/boot_console: Send <carriage return> on serial lines
tests/boot_console: Test booting HP-UX firmware upgrade
tests/acceptance/boot_linux_console.py | 27 +++++++++++++++++++++++++-
1 file changed, 26 insertions(+), 1 deletion(-)
--
2.21.0
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] tests/boot_console: Send <carriage return> on serial lines
2019-10-10 12:21 [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine Philippe Mathieu-Daudé
@ 2019-10-10 12:21 ` Philippe Mathieu-Daudé
2020-01-29 18:20 ` Wainer dos Santos Moschetta
2019-10-10 12:21 ` [PATCH 2/2] tests/boot_console: Test booting HP-UX firmware upgrade Philippe Mathieu-Daudé
2019-10-18 14:23 ` [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine Philippe Mathieu-Daudé
2 siblings, 1 reply; 7+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-10 12:21 UTC (permalink / raw)
To: qemu-devel
Cc: Eduardo Habkost, Helge Deller, Philippe Mathieu-Daudé,
Cleber Rosa, Sven Schnelle, Richard Henderson
Some firmwares don't parse the <Newline> control character and
expect a <carriage return>.
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
tests/acceptance/boot_linux_console.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py
index 8a9a314ab4..f05452824e 100644
--- a/tests/acceptance/boot_linux_console.py
+++ b/tests/acceptance/boot_linux_console.py
@@ -51,7 +51,7 @@ class BootLinuxConsole(Test):
self.fail(fail)
def exec_command_and_wait_for_pattern(self, command, success_message):
- command += '\n'
+ command += '\r\n'
self.vm.console_socket.sendall(command.encode())
self.wait_for_console_pattern(success_message)
--
2.21.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] tests/boot_console: Test booting HP-UX firmware upgrade
2019-10-10 12:21 [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine Philippe Mathieu-Daudé
2019-10-10 12:21 ` [PATCH 1/2] tests/boot_console: Send <carriage return> on serial lines Philippe Mathieu-Daudé
@ 2019-10-10 12:21 ` Philippe Mathieu-Daudé
2020-01-29 18:28 ` Wainer dos Santos Moschetta
2019-10-18 14:23 ` [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine Philippe Mathieu-Daudé
2 siblings, 1 reply; 7+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-10 12:21 UTC (permalink / raw)
To: qemu-devel
Cc: Eduardo Habkost, Helge Deller, Philippe Mathieu-Daudé,
Cleber Rosa, Sven Schnelle, Richard Henderson
Add a test which boots a HP-UX firmware upgrade CD-ROM.
It exercise the PCI LSI53C895A SCSI controller.
The ISO image comes from:
https://web.archive.org/web/20101204061612/http://ftp.parisc-linux.org/kernels/712/PF_C7120023
This test is very quick, less than 3s:
$ avocado --show=app,console run -t arch:hppa tests/acceptance/boot_linux_console.py
console: Firmware Version 6.1
console: Duplex Console IO Dependent Code (IODC) revision 1
console: Memory Test/Initialization Completed
console: ------------------------------------------------------------------------------
console: (c) Copyright 2017-2018 Helge Deller <deller@gmx.de> and SeaBIOS developers.
console: ------------------------------------------------------------------------------
console: Processor Speed State Coprocessor State Cache Size
console: --------- -------- --------------------- ----------------- ----------
console: 0 250 MHz Active Functional 0 KB
console: Available memory: 512 MB
console: Good memory required: 16 MB
console: Primary boot path: FWSCSI.0.0
console: Alternate boot path: FWSCSI.2.0
console: Console path: SERIAL_1.9600.8.none
console: Keyboard path: PS2
console: Available boot devices:
console: 1. DVD/CD [lsi 00:00.0 2:0 Drive QEMU QEMU CD-ROM 2.5+]
console: Booting from DVD/CD [lsi 00:00.0 2:0 Drive QEMU QEMU CD-ROM 2.5+]
console: Booting...
console: Boot IO Dependent Code (IODC) revision 153
console: HARD Booted.
console: ISL Revision A.00.25 November 18, 1992
console: ISL booting ODE UPDATE ; stable off ; RUN
console: Loading...
console: ***************************************************************************
console: ****** ******
console: ****** Offline Diagnostic Environment ******
console: ****** ******
console: ****** (C) Copyright Hewlett-Packard Co 1993 ******
console: ****** All Rights Reserved ******
console: ****** ******
console: ****** HP shall not be liable for any damages resulting from the ******
console: ****** use of this program. ******
console: ****** ******
console: ****** TC Version A.00.15 ******
console: ****** SysLib Version A.00.44 ******
console: ****** ******
console: ***************************************************************************
console: Type HELP for command information.
console: ISL_CMD> UPDATE ; stable off ; RUN
console: ***************************************************************************
console: ****** ******
console: ****** PDC UPDATER ******
console: ****** ******
console: ****** Copyright (C) 1993, 1994, 1995 by Hewlett-Packard Company ******
console: ****** Version A.00.19 ******
console: ****** ******
console: ***************************************************************************
console: Type HELP for command information.
console: Stable Storage will NOT be updated
console: STARTING EXECUTION OF UPDATE
console: Unrecognized MODEL TYPE = 502
console: ERROR 0001
console: UPDATE PAUSED> exit
console: UPDATE>
console: UPDATE> ls
console: Modules on this boot media are:
console: filename type size created description
console: -----------------------------------------------------------------------------
console: DAGGER DATA 118 96/02/15 ?
console: IMAGE1A DATA 512 96/02/15 ?
console: IMAGE1B DATA 388 96/02/15 ?
console: UPDATE> exit
console: THIS UTILITY WILL NOW RESET THE SYSTEM.....
PASS (2.39 s)
JOB TIME : 2.54 s
Suggested-by: Sven Schnelle <svens@stackframe.org>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
I know this is not a 'Linux' test, but all our console functions
reside in this file and I don't want to duplicate again.
Maybe we could rename this file as 'boot_console_tests.py' or
extract the console related functions.
---
tests/acceptance/boot_linux_console.py | 25 +++++++++++++++++++++++++
1 file changed, 25 insertions(+)
diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py
index f05452824e..8bfcd0625a 100644
--- a/tests/acceptance/boot_linux_console.py
+++ b/tests/acceptance/boot_linux_console.py
@@ -378,3 +378,28 @@ class BootLinuxConsole(Test):
self.vm.launch()
console_pattern = 'Kernel command line: %s' % kernel_command_line
self.wait_for_console_pattern(console_pattern)
+
+ def test_hppa_fwupdate(self):
+ """
+ :avocado: tags=arch:hppa
+ :avocado: tags=device:lsi53c895a
+ """
+ cdrom_url = ('https://github.com/philmd/qemu-testing-blob/raw/ec1b741/'
+ 'hppa/hp9000/712/C7120023.frm')
+ cdrom_hash = '17944dee46f768791953009bcda551be5ab9fac9'
+ cdrom_path = self.fetch_asset(cdrom_url, asset_hash=cdrom_hash)
+
+ self.vm.set_console()
+ self.vm.add_args('-cdrom', cdrom_path,
+ '-boot', 'd',
+ '-no-reboot')
+ self.vm.launch()
+ self.wait_for_console_pattern('Unrecognized MODEL TYPE = 502')
+
+ self.exec_command_and_wait_for_pattern('exit',
+ 'UPDATE>')
+ self.exec_command_and_wait_for_pattern('ls',
+ 'IMAGE1B')
+ self.exec_command_and_wait_for_pattern('exit',
+ 'THIS UTILITY WILL NOW '
+ 'RESET THE SYSTEM.....')
--
2.21.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine
2019-10-10 12:21 [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine Philippe Mathieu-Daudé
2019-10-10 12:21 ` [PATCH 1/2] tests/boot_console: Send <carriage return> on serial lines Philippe Mathieu-Daudé
2019-10-10 12:21 ` [PATCH 2/2] tests/boot_console: Test booting HP-UX firmware upgrade Philippe Mathieu-Daudé
@ 2019-10-18 14:23 ` Philippe Mathieu-Daudé
2019-10-18 14:40 ` Sven Schnelle
2 siblings, 1 reply; 7+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-18 14:23 UTC (permalink / raw)
To: qemu-devel@nongnu.org Developers, Sven Schnelle, Cleber Rosa
Cc: Helge Deller, Eduardo Habkost, Richard Henderson
ping?
Sven since you tested this series, can I add your Tested-by tag?
On Thu, Oct 10, 2019 at 2:21 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> This tests boot a HP-UX firmware CD-ROM which allow serial
> console interaction. This exercise the PCI LSI53C895A SCSI
> controller.
>
> I'm not adding it to the Travis-CI list because I'm not sure
> how to split/rename the current job, see:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg644753.html
>
> Philippe Mathieu-Daudé (2):
> tests/boot_console: Send <carriage return> on serial lines
> tests/boot_console: Test booting HP-UX firmware upgrade
>
> tests/acceptance/boot_linux_console.py | 27 +++++++++++++++++++++++++-
> 1 file changed, 26 insertions(+), 1 deletion(-)
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine
2019-10-18 14:23 ` [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine Philippe Mathieu-Daudé
@ 2019-10-18 14:40 ` Sven Schnelle
0 siblings, 0 replies; 7+ messages in thread
From: Sven Schnelle @ 2019-10-18 14:40 UTC (permalink / raw)
To: Philippe Mathieu-Daudé
Cc: Helge Deller, Richard Henderson,
qemu-devel@nongnu.org Developers, Eduardo Habkost, Cleber Rosa
Hi Philippe,
On Fri, Oct 18, 2019 at 04:23:10PM +0200, Philippe Mathieu-Daudé wrote:
> Sven since you tested this series, can I add your Tested-by tag?
Sure, please do.
> On Thu, Oct 10, 2019 at 2:21 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
> >
> > This tests boot a HP-UX firmware CD-ROM which allow serial
> > console interaction. This exercise the PCI LSI53C895A SCSI
> > controller.
> >
> > I'm not adding it to the Travis-CI list because I'm not sure
> > how to split/rename the current job, see:
> > https://www.mail-archive.com/qemu-devel@nongnu.org/msg644753.html
> >
> > Philippe Mathieu-Daudé (2):
> > tests/boot_console: Send <carriage return> on serial lines
> > tests/boot_console: Test booting HP-UX firmware upgrade
> >
> > tests/acceptance/boot_linux_console.py | 27 +++++++++++++++++++++++++-
> > 1 file changed, 26 insertions(+), 1 deletion(-)
Best Regards
Sven
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] tests/boot_console: Send <carriage return> on serial lines
2019-10-10 12:21 ` [PATCH 1/2] tests/boot_console: Send <carriage return> on serial lines Philippe Mathieu-Daudé
@ 2020-01-29 18:20 ` Wainer dos Santos Moschetta
0 siblings, 0 replies; 7+ messages in thread
From: Wainer dos Santos Moschetta @ 2020-01-29 18:20 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel
Cc: Helge Deller, Richard Henderson, Sven Schnelle, Eduardo Habkost,
Cleber Rosa
On 10/10/19 9:21 AM, Philippe Mathieu-Daudé wrote:
> Some firmwares don't parse the <Newline> control character and
> expect a <carriage return>.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> tests/acceptance/boot_linux_console.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py
> index 8a9a314ab4..f05452824e 100644
> --- a/tests/acceptance/boot_linux_console.py
> +++ b/tests/acceptance/boot_linux_console.py
> @@ -51,7 +51,7 @@ class BootLinuxConsole(Test):
> self.fail(fail)
>
> def exec_command_and_wait_for_pattern(self, command, success_message):
> - command += '\n'
> + command += '\r\n'
> self.vm.console_socket.sendall(command.encode())
> self.wait_for_console_pattern(success_message)
>
This doesn't apply anymore since exec_command_and_wait_for_pattern() is
now part of the avocado_qemu module.
I'm little worried about this breaking the current tests that use the
method. Could you please run those tests before putting it on the pull
request, Philippe?
Besides that, this change looks good to me:
Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] tests/boot_console: Test booting HP-UX firmware upgrade
2019-10-10 12:21 ` [PATCH 2/2] tests/boot_console: Test booting HP-UX firmware upgrade Philippe Mathieu-Daudé
@ 2020-01-29 18:28 ` Wainer dos Santos Moschetta
0 siblings, 0 replies; 7+ messages in thread
From: Wainer dos Santos Moschetta @ 2020-01-29 18:28 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, qemu-devel
Cc: Helge Deller, Richard Henderson, Sven Schnelle, Eduardo Habkost,
Cleber Rosa
On 10/10/19 9:21 AM, Philippe Mathieu-Daudé wrote:
> Add a test which boots a HP-UX firmware upgrade CD-ROM.
> It exercise the PCI LSI53C895A SCSI controller.
>
> The ISO image comes from:
> https://web.archive.org/web/20101204061612/http://ftp.parisc-linux.org/kernels/712/PF_C7120023
>
> This test is very quick, less than 3s:
>
> $ avocado --show=app,console run -t arch:hppa tests/acceptance/boot_linux_console.py
> console: Firmware Version 6.1
> console: Duplex Console IO Dependent Code (IODC) revision 1
> console: Memory Test/Initialization Completed
> console: ------------------------------------------------------------------------------
> console: (c) Copyright 2017-2018 Helge Deller <deller@gmx.de> and SeaBIOS developers.
> console: ------------------------------------------------------------------------------
> console: Processor Speed State Coprocessor State Cache Size
> console: --------- -------- --------------------- ----------------- ----------
> console: 0 250 MHz Active Functional 0 KB
> console: Available memory: 512 MB
> console: Good memory required: 16 MB
> console: Primary boot path: FWSCSI.0.0
> console: Alternate boot path: FWSCSI.2.0
> console: Console path: SERIAL_1.9600.8.none
> console: Keyboard path: PS2
> console: Available boot devices:
> console: 1. DVD/CD [lsi 00:00.0 2:0 Drive QEMU QEMU CD-ROM 2.5+]
> console: Booting from DVD/CD [lsi 00:00.0 2:0 Drive QEMU QEMU CD-ROM 2.5+]
> console: Booting...
> console: Boot IO Dependent Code (IODC) revision 153
> console: HARD Booted.
> console: ISL Revision A.00.25 November 18, 1992
> console: ISL booting ODE UPDATE ; stable off ; RUN
> console: Loading...
> console: ***************************************************************************
> console: ****** ******
> console: ****** Offline Diagnostic Environment ******
> console: ****** ******
> console: ****** (C) Copyright Hewlett-Packard Co 1993 ******
> console: ****** All Rights Reserved ******
> console: ****** ******
> console: ****** HP shall not be liable for any damages resulting from the ******
> console: ****** use of this program. ******
> console: ****** ******
> console: ****** TC Version A.00.15 ******
> console: ****** SysLib Version A.00.44 ******
> console: ****** ******
> console: ***************************************************************************
> console: Type HELP for command information.
> console: ISL_CMD> UPDATE ; stable off ; RUN
> console: ***************************************************************************
> console: ****** ******
> console: ****** PDC UPDATER ******
> console: ****** ******
> console: ****** Copyright (C) 1993, 1994, 1995 by Hewlett-Packard Company ******
> console: ****** Version A.00.19 ******
> console: ****** ******
> console: ***************************************************************************
> console: Type HELP for command information.
> console: Stable Storage will NOT be updated
> console: STARTING EXECUTION OF UPDATE
> console: Unrecognized MODEL TYPE = 502
> console: ERROR 0001
> console: UPDATE PAUSED> exit
> console: UPDATE>
> console: UPDATE> ls
> console: Modules on this boot media are:
> console: filename type size created description
> console: -----------------------------------------------------------------------------
> console: DAGGER DATA 118 96/02/15 ?
> console: IMAGE1A DATA 512 96/02/15 ?
> console: IMAGE1B DATA 388 96/02/15 ?
> console: UPDATE> exit
> console: THIS UTILITY WILL NOW RESET THE SYSTEM.....
> PASS (2.39 s)
> JOB TIME : 2.54 s
>
> Suggested-by: Sven Schnelle <svens@stackframe.org>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> I know this is not a 'Linux' test, but all our console functions
> reside in this file and I don't want to duplicate again.
> Maybe we could rename this file as 'boot_console_tests.py' or
> extract the console related functions.
Yeah, there is some discussion about the fact that boot_linux_console
contain more than just Linux boot tests. Also the 'boot something'
acceptance tests list is growing fast, thus IMO soon we gonna need to
re-organize the code. So I'm ok to merge this in boot_linux_console,
then later refactor it all.
I could not test it because the changes no longer applies. The code
looks good though, so:
Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
- Wainer
> ---
> tests/acceptance/boot_linux_console.py | 25 +++++++++++++++++++++++++
> 1 file changed, 25 insertions(+)
>
> diff --git a/tests/acceptance/boot_linux_console.py b/tests/acceptance/boot_linux_console.py
> index f05452824e..8bfcd0625a 100644
> --- a/tests/acceptance/boot_linux_console.py
> +++ b/tests/acceptance/boot_linux_console.py
> @@ -378,3 +378,28 @@ class BootLinuxConsole(Test):
> self.vm.launch()
> console_pattern = 'Kernel command line: %s' % kernel_command_line
> self.wait_for_console_pattern(console_pattern)
> +
> + def test_hppa_fwupdate(self):
> + """
> + :avocado: tags=arch:hppa
> + :avocado: tags=device:lsi53c895a
> + """
> + cdrom_url = ('https://github.com/philmd/qemu-testing-blob/raw/ec1b741/'
> + 'hppa/hp9000/712/C7120023.frm')
> + cdrom_hash = '17944dee46f768791953009bcda551be5ab9fac9'
> + cdrom_path = self.fetch_asset(cdrom_url, asset_hash=cdrom_hash)
> +
> + self.vm.set_console()
> + self.vm.add_args('-cdrom', cdrom_path,
> + '-boot', 'd',
> + '-no-reboot')
> + self.vm.launch()
> + self.wait_for_console_pattern('Unrecognized MODEL TYPE = 502')
> +
> + self.exec_command_and_wait_for_pattern('exit',
> + 'UPDATE>')
> + self.exec_command_and_wait_for_pattern('ls',
> + 'IMAGE1B')
> + self.exec_command_and_wait_for_pattern('exit',
> + 'THIS UTILITY WILL NOW '
> + 'RESET THE SYSTEM.....')
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-01-29 18:30 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-10 12:21 [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine Philippe Mathieu-Daudé
2019-10-10 12:21 ` [PATCH 1/2] tests/boot_console: Send <carriage return> on serial lines Philippe Mathieu-Daudé
2020-01-29 18:20 ` Wainer dos Santos Moschetta
2019-10-10 12:21 ` [PATCH 2/2] tests/boot_console: Test booting HP-UX firmware upgrade Philippe Mathieu-Daudé
2020-01-29 18:28 ` Wainer dos Santos Moschetta
2019-10-18 14:23 ` [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine Philippe Mathieu-Daudé
2019-10-18 14:40 ` Sven Schnelle
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).