On 9/30/26 8:07 AM, Akihiko Odaki wrote: > On 2026/09/30 2:23, Andrey Drobyshev wrote: >> Snapshots are refused whenever a CPR migration mode is set. Add a test >> which sets each of the CPR modes in turn and checks that 'savevm' and >> 'loadvm' fail with the corresponding error, then switches back to >> 'normal' mode and checks that the error is gone. >> >> Signed-off-by: Andrey Drobyshev <[email protected]> >> --- >> tests/qtest/migration/cpr-tests.c | 36 ++++++++++++++++++++++++++++++++++++ >> 1 file changed, 36 insertions(+) >> >> diff --git a/tests/qtest/migration/cpr-tests.c >> b/tests/qtest/migration/cpr-tests.c >> index 0bec753b4ab..2be0fc8c07e 100644 >> --- a/tests/qtest/migration/cpr-tests.c >> +++ b/tests/qtest/migration/cpr-tests.c >> @@ -290,6 +290,40 @@ static void test_mode_exec(char *name, MigrateCommon >> *args) >> test_cpr_exec(args); >> } >> >> +/* Snapshots must be refused while a CPR migration mode is set */ >> +static void test_mode_no_snapshot(char *name, MigrateCommon *args) >> +{ >> + static const char *const modes[] = { >> + "cpr-reboot", "cpr-transfer", "cpr-exec", >> + }; >> + QTestState *vm = qtest_init("-nodefaults"); > > This omits -machine and fails on e.g., AArch64: QEMU exits with “No > machine specified”, then qtest aborts waiting for its connection. > > Regards, > Akihiko Odaki Thanks for noticing this. I suppose providing '-machine none' should help here. I'll add this.
Andrey
