diff options
author | Daniel P. Berrangé <berrange@redhat.com> | 2023-03-15 17:43:21 +0000 |
---|---|---|
committer | Alex Bennée <alex.bennee@linaro.org> | 2023-03-22 15:08:26 +0000 |
commit | cb845eaa88eb266c5023af06989e94d95c712871 (patch) | |
tree | 27f4291a80912df0d5fb6779b812cb54a4bc7cdb /block | |
parent | 6e5792a1f6ecc155fc977e3813e75fc8ede478ab (diff) | |
download | qemu-cb845eaa88eb266c5023af06989e94d95c712871.zip qemu-cb845eaa88eb266c5023af06989e94d95c712871.tar.gz qemu-cb845eaa88eb266c5023af06989e94d95c712871.tar.bz2 |
iotests: connect stdin to /dev/null when running tests
Currently the tests have their stdin inherited from the test harness,
meaning they are connected to a TTY. The QEMU processes spawned by
certain tests, however, modify TTY settings and if the test exits
abnormally the settings might not be restored.
The python test harness thus has some logic which will capture the
initial TTY settings and restore them once all tests are finished.
This does not, however, take into account the possibility of many
copies of the 'check' program running in parallel. With parallel
execution, a later invokation may save the TTY state that QEMU has
already modified, and thus restore bad state leaving the TTY
non-functional.
None of the I/O tests shnould actually be interactive requiring
user input and so they should not require a TTY at all. To avoid
this while TTY save/restore complexity we can connect the test
stdin to /dev/null instead.
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Acked-by: Hanna Czenczek <hreitz@redhat.com>
Tested-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20230303160727.3977246-6-berrange@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20230315174331.2959-23-alex.bennee@linaro.org>
Diffstat (limited to 'block')
0 files changed, 0 insertions, 0 deletions