[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 5e5432: tests/boot-serial-test: Fix problem w
From: |
GitHub |
Subject: |
[Qemu-commits] [qemu/qemu] 5e5432: tests/boot-serial-test: Fix problem with timeout d... |
Date: |
Fri, 16 Feb 2018 03:17:57 -0800 |
Branch: refs/heads/master
Home: https://github.com/qemu/qemu
Commit: 5e5432b766c424a5d1e1b81315ce6ac1dc0fa3ed
https://github.com/qemu/qemu/commit/5e5432b766c424a5d1e1b81315ce6ac1dc0fa3ed
Author: Thomas Huth <address@hidden>
Date: 2018-02-16 (Fri, 16 Feb 2018)
Changed paths:
M tests/boot-serial-test.c
Log Message:
-----------
tests/boot-serial-test: Fix problem with timeout due to dropped characters
Commit 92b540dac9fc3a5 introduce a counter to handle the timeouts in a
better way. But in case ccnt reaches 512, the current read character is
ignored - and if that character is part of the string that we are looking
for, the test fails to match the string.
Almost all of the tests look for a string within the first 512 bytes of
firmware output, so the problem never triggered there. But the hppa test
that has been added recently looks for a longer string at the very end of
a long output, thus there's a chance that we miss a character there so
that the test fails unexpectedly. Fix it by *not* reading and dropping a
character if the counter reaches 512.
Fixes: 92b540dac9fc3a572c7342edd0b073000f5a6abf
Signed-off-by: Thomas Huth <address@hidden>
Message-id: address@hidden
[PMM: added initializer for nbd to silence false-positive warning
from OpenBSD 6 compiler]
Signed-off-by: Peter Maydell <address@hidden>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-commits] [qemu/qemu] 5e5432: tests/boot-serial-test: Fix problem with timeout d...,
GitHub <=