[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 04/26] tests/acceptance: Send <carriage return> on serial lin
From: |
Cleber Rosa |
Subject: |
Re: [PATCH 04/26] tests/acceptance: Send <carriage return> on serial lines |
Date: |
Mon, 28 Oct 2019 11:36:40 -0400 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
On Mon, Oct 28, 2019 at 08:34:19AM +0100, Philippe Mathieu-Daudé wrote:
> From: Philippe Mathieu-Daudé <address@hidden>
>
> Some firmwares don't parse the <Newline> control character and
> expect a <carriage return>.
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> since previous: do not send NewLine
> ---
> 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 8897e0c253..f9b77af359 100644
> --- a/tests/acceptance/boot_linux_console.py
> +++ b/tests/acceptance/boot_linux_console.py
> @@ -34,7 +34,7 @@ class BootLinuxConsole(Test):
> failure_message='Kernel panic - not
> syncing')
>
> def exec_command_and_wait_for_pattern(self, command, success_message):
> - command += '\n'
> + command += '\r'
> self.vm.console_socket.sendall(command.encode())
> wait_for_console_pattern(self, success_message)
>
> --
> 2.21.0
>
Given the discussion on
<address@hidden>, this LGTM.
Reviewed-by: Cleber Rosa <address@hidden>
Tested-by: Cleber Rosa <address@hidden>
Queuing on my python-next branch (and sending shortly in a PR).
Thanks,
- Cleber.
- Re: [PATCH 01/26] python/qemu/machine: Allow to use other serial consoles than default, (continued)
- [PATCH 02/26] Acceptance tests: refactor wait_for_console_pattern, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 03/26] tests/acceptance: Fixe wait_for_console_pattern() hangs, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 04/26] tests/acceptance: Send <carriage return> on serial lines, Philippe Mathieu-Daudé, 2019/10/28
- Re: [PATCH 04/26] tests/acceptance: Send <carriage return> on serial lines,
Cleber Rosa <=
- [PATCH 05/26] tests/acceptance: Refactor exec_command_and_wait_for_pattern(), Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 06/26] tests/acceptance: Rename avocado_qemu.Test as MachineTest, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 07/26] tests/acceptance: Make pick_default_qemu_bin() more generic, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 08/26] tests/acceptance: Introduce LinuxUserTest base class, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 09/26] tests/acceptance: Add bFLT loader linux-user test, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 10/26] tests/acceptance: Add test that boots the HelenOS microkernel on Leon3, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 11/26] tests/acceptance: Add test that boots Linux up to BusyBox on Leon3, Philippe Mathieu-Daudé, 2019/10/28