Hi On Tue, Sep 20, 2022 at 3:18 PM Bin Meng <[email protected]> wrote:
> From: Xuzhou Cheng <[email protected]> > > Make sure QEMU process "to" exited before launching another target > for migration in the test_multifd_tcp_cancel case. > > Signed-off-by: Xuzhou Cheng <[email protected]> > Signed-off-by: Bin Meng <[email protected]> > Reviewed-by: Marc-André Lureau <[email protected]> > fwiw, I didn't r-b the version with a busy wait ( https://patchew.org/QEMU/[email protected]/[email protected]/ ) --- > > Changes in v2: > - Change to a busy wait after migration is canceled > > tests/qtest/migration-test.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c > index c87afad9e8..aedd9ddb72 100644 > --- a/tests/qtest/migration-test.c > +++ b/tests/qtest/migration-test.c > @@ -2133,6 +2133,10 @@ static void test_multifd_tcp_cancel(void) > wait_for_migration_pass(from); > > migrate_cancel(from); > + /* Make sure QEMU process "to" exited */ > + while (qtest_probe_child(to)) { > + ; > + } > > args = (MigrateStart){ > .only_target = true, > -- > 2.34.1 > > > -- Marc-André Lureau
