On Tue Jul 2, 2024 at 8:33 PM AEST, Thomas Huth wrote: > From: Nicholas Piggin <npig...@gmail.com> > > s390x with TCG is more stable now. Enable it.
Ah, you did a more complete version of my flic fix that migrates all the state. I didn't see that go by but yeah I suspect that was probably the correct thing to do. Thanks for that. Should the s390x flic migrate fix could be got to stable, perhaps? There's some kvm-unit-tests s390x migration tests that can be enabled after the fix too don't forget. Thanks, Nick > > Signed-off-by: Nicholas Piggin <npig...@gmail.com> > Message-Id: <20240525131241.378473-3-npig...@gmail.com> > Reviewed-by: Prasad Pandit <p...@fedoraproject.org> > [thuth: Added "with TCG" to the commit message] > Signed-off-by: Thomas Huth <th...@redhat.com> > --- > tests/qtest/migration-test.c | 12 ------------ > 1 file changed, 12 deletions(-) > > diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c > index 571fc1334c..70b606b888 100644 > --- a/tests/qtest/migration-test.c > +++ b/tests/qtest/migration-test.c > @@ -3823,16 +3823,6 @@ int main(int argc, char **argv) > test_vmstate_checker_script); > #endif > > - /* > - * On s390x with TCG, migration is observed to hang due to the 'pending' > - * state of the flic interrupt controller not being migrated or > - * reconstructed post-migration. Disable it until the problem is > resolved. > - */ > - if (g_str_equal(arch, "s390x") && !has_kvm) { > - g_test_message("Skipping tests: s390x host with KVM is required"); > - goto test_add_done; > - } > - > if (is_x86) { > migration_test_add("/migration/precopy/unix/suspend/live", > test_precopy_unix_suspend_live); > @@ -4036,8 +4026,6 @@ int main(int argc, char **argv) > test_vcpu_dirty_limit); > } > > -test_add_done: > - > ret = g_test_run(); > > g_assert_cmpint(ret, ==, 0);