From nobody Sat Feb 7 09:04:18 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1657017665; cv=none; d=zohomail.com; s=zohoarc; b=ixsj9RtNBE1W5Mn9coau0sSbQIt1IKSefJzvHmNUbifpmgG95K5haSFrIkfRgBqNdThDv9CPs0wPBoa4VIJWhnqY2nDKVL9XseKYnYivtBa1yCOSRdBb+QUPqriB8da8b5TV0A6TOkkVQWPEc0KyEK5kAtq0rsPPLq9wDIItmOQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1657017665; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=jI/vFeFSjJ9wECviU/vfiJz+Koz2JKyulvHHp6jyt0Q=; b=W3CMZDasatQSES6QCFfCm02GqDd/ct2KZAeROiljjco/Sod4LUjp0/2iijjU1/tpOrufdZMMgb/I1C+OubPRr966QPa4TORT67+/PmoCQVtnhhvO5GQZ8sABLS1m8fKu6acBkUrVdEL8sByAoqaFCwqNFO/ApN6XZHdGNYIy0nY= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1657017665580288.8411808131575; Tue, 5 Jul 2022 03:41:05 -0700 (PDT) Received: from localhost ([::1]:49326 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1o8fzM-00022O-Cf for importer@patchew.org; Tue, 05 Jul 2022 06:41:04 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:36322) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1o8fwz-0006S0-By for qemu-devel@nongnu.org; Tue, 05 Jul 2022 06:38:37 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]:34560) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1o8fwx-0005zd-Sc for qemu-devel@nongnu.org; Tue, 05 Jul 2022 06:38:37 -0400 Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-108-70Wr7ghqNKaNxebyQXPNrA-1; Tue, 05 Jul 2022 06:38:31 -0400 Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 001DE80174C; Tue, 5 Jul 2022 10:38:31 +0000 (UTC) Received: from thuth.com (unknown [10.39.193.133]) by smtp.corp.redhat.com (Postfix) with ESMTP id D83DA492C3B; Tue, 5 Jul 2022 10:38:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1657017515; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=jI/vFeFSjJ9wECviU/vfiJz+Koz2JKyulvHHp6jyt0Q=; b=dkvVsJJp8CIVK+bOLNkVPVrOfEBiapSsCEXT33lZgBQKi94dkdr9brQv3VYRK2Xi/WrDwZ vgzl/dnk+DVuF+Mi6P8Rfis05CX+kM1g8wMbCRw0S5i1ZiTNxbpcJIzbp0llUbeAAAqcmG xiyXQvumTspfxnLEVcngVRn+1uyoIho= X-MC-Unique: 70Wr7ghqNKaNxebyQXPNrA-1 From: Thomas Huth To: qemu-devel@nongnu.org, Richard Henderson Cc: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , "Dr . David Alan Gilbert" , Laurent Vivier Subject: [PULL 08/14] tests: wait for migration completion before looking for STOP event Date: Tue, 5 Jul 2022 12:38:10 +0200 Message-Id: <20220705103816.608166-9-thuth@redhat.com> In-Reply-To: <20220705103816.608166-1-thuth@redhat.com> References: <20220705103816.608166-1-thuth@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 2.85 on 10.11.54.9 Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=170.10.133.124; envelope-from=thuth@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -21 X-Spam_score: -2.2 X-Spam_bar: -- X-Spam_report: (-2.2 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.082, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1657017666458100001 From: Daniel P. Berrang=C3=A9 When moving into the convergance phase, the precopy tests will first look for a STOP event and once found will look for migration completion status. If the test VM is not converging, the test suite will be waiting for the STOP event forever. If we wait for the migration completion status first, then we will trigger the previously added timeout and prevent the test hanging forever. Signed-off-by: Daniel P. Berrang=C3=A9 Reviewed-by: Dr. David Alan Gilbert Reviewed-by: Laurent Vivier Message-Id: <20220628105434.295905-3-berrange@redhat.com> Signed-off-by: Thomas Huth --- tests/qtest/migration-test.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c index d33e8060f9..ac9e303b1f 100644 --- a/tests/qtest/migration-test.c +++ b/tests/qtest/migration-test.c @@ -1232,6 +1232,10 @@ static void test_precopy_common(MigrateCommon *args) =20 migrate_set_parameter_int(from, "downtime-limit", CONVERGE_DOWNTIM= E); =20 + /* We do this first, as it has a timeout to stop us + * hanging forever if migration didn't converge */ + wait_for_migration_complete(from); + if (!got_stop) { qtest_qmp_eventwait(from, "STOP"); } @@ -1239,7 +1243,6 @@ static void test_precopy_common(MigrateCommon *args) qtest_qmp_eventwait(to, "RESUME"); =20 wait_for_serial("dest_serial"); - wait_for_migration_complete(from); } =20 if (args->finish_hook) { --=20 2.31.1