@samitouri / QOSamiQemu / commits / 182d1afcb9

tests/qtest/pnv: drop unnecessary -serial mon:stdio

The pnv-xive2-test and pnv-host-i2c-test pass "-serial mon:stdio" but never use HMP commands. The "mon:" chardev prefix creates an HMP monitor mux, which fails when HMP is disabled. Other pnv tests (e.g. pnv-spi-seeprom-test) don't specify -serial at all. Drop the unnecessary -serial mon:stdio option. Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Message-ID: <20260828-qemu-no-hmp-v5-12-9227de146347@redhat.com>

Marc-André Lureau committed Aug 28, 2026 at 16:04 UTC 182d1afcb95f8c17ca03e5efc098ab47cd92e522
2 files changed +2 -2
tests/qtest/pnv-host-i2c-test.c
+1 -1
@@ -414,7 +414,7 @@ static void test_host_i2c(const void *data)
414 }
415
416 qts = qtest_initf("-M %s -smp %d,cores=1,threads=%d -nographic "
417 - "-nodefaults -serial mon:stdio -S "
417 + "-nodefaults -S "
418 "-d guest_errors",
419 machine, SMT, SMT);
420
tests/qtest/pnv-xive2-test.c
+1 -1
@@ -552,7 +552,7 @@ static void test_xive(const void *data)
552 QTestState *qts;
553
554 qts = qtest_initf("-M %s -smp %d,cores=1,threads=%d -nographic "
555 - "-nodefaults -serial mon:stdio -S "
555 + "-nodefaults -S "
556 "-d guest_errors -trace '*xive*'",
557 machine, SMT, SMT);
558 init_xive(qts);