QEMU-Devel Archive on lore.kernel.org
 help / color / Atom feed
* [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; 5+ 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] 5+ 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é
  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, 0 replies; 5+ 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	[flat|nested] 5+ 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é
  2019-10-18 14:23 ` [PATCH 0/2] tests/acceptance: Add tests for the PA-RISC machine Philippe Mathieu-Daudé
  2 siblings, 0 replies; 5+ 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	[flat|nested] 5+ 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; 5+ 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] 5+ 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; 5+ 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] 5+ messages in thread

end of thread, back to index

Thread overview: 5+ 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é
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é
2019-10-18 14:40   ` Sven Schnelle

QEMU-Devel Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/qemu-devel/0 qemu-devel/git/0.git
	git clone --mirror https://lore.kernel.org/qemu-devel/1 qemu-devel/git/1.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 qemu-devel qemu-devel/ https://lore.kernel.org/qemu-devel \
		qemu-devel@nongnu.org qemu-devel@archiver.kernel.org
	public-inbox-index qemu-devel

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.nongnu.qemu-devel


AGPL code for this site: git clone https://public-inbox.org/ public-inbox