From nobody Thu May 16 05:59:41 2024 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; arc=pass (i=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass(p=none dis=none) header.from=virtuozzo.com ARC-Seal: i=2; a=rsa-sha256; t=1591259699; cv=pass; d=zohomail.com; s=zohoarc; b=g3CKw3lRmuZrSWJX5rnrnmD/VuRddYaV25TIG35eA+5CCzrInSmGmE0fNd6oinXuy+Ta7N4fTBiHaWPrkPfNCWnAoCnJWpDwmD8V6AGScNK34qPibVAP3Mnlo+wRivxivfVQg3HEVkgDw2ugMIXXbbcQXlwPfNb4sRXBJAWuIQE= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1591259699; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=vig8Yz7pr2lV1wIB65UYp5XEiSnfBFahnO/GpDNSz6A=; b=LKuWp+gueC6MAKAmK6v9pN+Ky4n1MZuENd8mDM0XOkRjSr/nwMQpXKyzCLycV2iZwLdj5iZKACuW+ImNJSKDJYYyO40Fu6LW2ENN9yy4Qcymd4bC4lPeFMa8KCVTTqHmUwWDK0FGVFgPLHoBBWJ1KD6PWWQyhOE4EzqGP+Q38Wg= ARC-Authentication-Results: i=2; 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; arc=pass (i=1dmarc=pass fromdomain=virtuozzo.com); dmarc=pass header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1591259699406727.475637377008; Thu, 4 Jun 2020 01:34:59 -0700 (PDT) Received: from localhost ([::1]:42564 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jglKz-0006dD-Kl for importer@patchew.org; Thu, 04 Jun 2020 04:34:57 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:49766) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jglK6-0005qw-Nx; Thu, 04 Jun 2020 04:34:02 -0400 Received: from mail-eopbgr140098.outbound.protection.outlook.com ([40.107.14.98]:12946 helo=EUR01-VE1-obe.outbound.protection.outlook.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jglK4-0006iw-Fn; Thu, 04 Jun 2020 04:34:01 -0400 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) by AM7PR08MB5303.eurprd08.prod.outlook.com (2603:10a6:20b:102::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3066.18; Thu, 4 Jun 2020 08:33:55 +0000 Received: from AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::a408:2f0f:bc6c:d312]) by AM7PR08MB5494.eurprd08.prod.outlook.com ([fe80::a408:2f0f:bc6c:d312%3]) with mapi id 15.20.3066.018; Thu, 4 Jun 2020 08:33:55 +0000 Received: from kvm.sw.ru (185.215.60.190) by AM0PR05CA0079.eurprd05.prod.outlook.com (2603:10a6:208:136::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3066.18 via Frontend Transport; Thu, 4 Jun 2020 08:33:54 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XCf08osLK62JojPCcsPv4MKOm7CHmLrbPF4v31uV/GpdlL6ave4XYq/FMCJXlMzAlMOuHOoDft9kOLUpkvB0N3+++7CQiQRLdvwFt3wPCCJUreKs5NuLe0GfNNOBVv7zXvJs440ZpIkHKIDiBvaXFFMJKij6By4QvIm5wz4Ry54fBgcExGCOSp5Sa+SpQgPaaen5ro3tjN4I2EIcChZLWPqbP8jfzXCh2mtRjfn2nNSHVKTkYn9yKre5i6HeBBuStWr5//WINMjv8ghsI25Yb1IJek8aqAC9d8m0H30mThU92ggUAOX9uLN9d1z73atKLdyNbmc+xcHv+D/VkKrf4w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vig8Yz7pr2lV1wIB65UYp5XEiSnfBFahnO/GpDNSz6A=; b=MyezRGcBnFI98qVzKnfYBCsvgdI8/4tARVew4CoPR16W6pXKF42iwztGTanPYrPM7enOgVG3ZUTQ7xkebUxboebrMYvtiuNX7YLz0qqoo7wKBrhGPcGJYVNpzHpi2n+GvVgXio6APfAxC7xOLEPwBcHFki30e8xYmg1AqPl2P3PMyCz3yxCfrNry3uQ+AiNpvi4fZGiUonOwZaMM0zHxj8fV0L+0YVMhLcS6QzhAWQIIqGo/CHFdIqejMh0jvmIieEV0oYPSsDVgimdiphAvYjNKlWKfaANjGq62pCcIRvxJXfSebk8WZT/eZTmrJ3eCkV29WgdJ7VxVae1/R13N2w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=virtuozzo.com; dmarc=pass action=none header.from=virtuozzo.com; dkim=pass header.d=virtuozzo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=virtuozzo.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vig8Yz7pr2lV1wIB65UYp5XEiSnfBFahnO/GpDNSz6A=; b=sFOxLK+apmIVW6Z+Wit8e95L4jiFIjOtvO4MaIPRvWyylHqUZzqcKX8NWXDXE37CD6ruqTRXfcDj8URCpLD01ES/7HTO5UEm3Vr9VRf+oZJSv19r/7dbjEGZSda9W9d0YM/xdsdo8bIa5zRg3nFS4NDbGBsCzTy0iKJV303baDw= Authentication-Results: nongnu.org; dkim=none (message not signed) header.d=none;nongnu.org; dmarc=none action=none header.from=virtuozzo.com; From: Vladimir Sementsov-Ogievskiy To: qemu-block@nongnu.org Subject: [PATCH] iotests: 194: wait migration completion on target too Date: Thu, 4 Jun 2020 11:33:41 +0300 Message-Id: <20200604083341.26978-1-vsementsov@virtuozzo.com> X-Mailer: git-send-email 2.21.0 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: AM0PR05CA0079.eurprd05.prod.outlook.com (2603:10a6:208:136::19) To AM7PR08MB5494.eurprd08.prod.outlook.com (2603:10a6:20b:dc::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-Mailer: git-send-email 2.21.0 X-Originating-IP: [185.215.60.190] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 4973fa53-272d-43ad-65af-08d808620479 X-MS-TrafficTypeDiagnostic: AM7PR08MB5303: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:534; X-Forefront-PRVS: 04244E0DC5 X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: xLoJlkXd7/M1W6HVKDVHhkbJ4ZdS/FBfT6CegVOBREDqLZEeK/PhPZ9XJ87BVkETtle1vc6DdMzqTebGLH/CP+wtIXuZEYl1fOD7shYkmGuBEQCYNv7PzNTVzE2LfJzAkb2Psruokz8qlNWYNBZcBDZAMrFLzkQW5/Le1fLph1eyMo0xAsRN93pp4q16ZNsZxCFwu/mIvJpcIs6k4egRlCRTbR2Ql18iXieVnfCEUlLWwFyO4jvevjlh6Mt1qGp4HvpgJFkT0gxBOXuV2WBWahhppU3yqOYLgqGjSsWcMRveZssT+Q77q2FPJi5AdZ/KqmR2PElUr+gc+aOxF+HIBw== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM7PR08MB5494.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(39840400004)(396003)(376002)(366004)(346002)(136003)(16526019)(66556008)(66946007)(83380400001)(86362001)(26005)(8676002)(956004)(2616005)(8936002)(6512007)(66476007)(478600001)(6486002)(6916009)(6506007)(7416002)(186003)(52116002)(6666004)(1076003)(5660300002)(36756003)(2906002)(316002)(4326008); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData: ZHmiAxCK6nsbSBACjsxEm9VLAV7oWxLDYkM9mXmwdtrY8uHKm9WDTFWsNrkbPgqCNzNQzviFGQtFeIJ3xEe0zlVECpI7t+v5DEpDUIOVp6liztN3MevxyQqEJwKErB59pJqcyxfrMNZx2BDHbV9Pz2eIGapnVrQq1dzgWBCSJbGwmEmw0brBhzLY5ip8Meqw8aGnhjgF9ZfHw/QBzkQmAZbpNxtY2bBmwP5KaMFoWz9IsF1Tz2f4J1MzoBEnWapjWnD2+bQobzwBJsxXMengwgb3by33WxxWF5v80j593d/lmbP6dsRq9B40VFs/+fYq8jp3Db7n0zxAeUFn9D7is6j0SqRRLOMVB/B/UkcT8Z9jdYpkmi+lXzaiXjKqW3idIvBAMcIFJH/vhKAA14HQmsMYZ49+uk7J36YLKu3zVpnmQvNGBGjts+Q5LEUumUoN/DXZYoBFics1XdosqB2NnFNHBsOtgRJ49lobYcTvnWHOv+EVLcDuHrT7wxOgTNv5 X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4973fa53-272d-43ad-65af-08d808620479 X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Jun 2020 08:33:54.9260 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 0bc7f26d-0264-416e-a6fc-8352af79c58f X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: SKwHqYGyFmGllvMdlBQVaTjTf/2FtGuTP9XenG1vEIlwpkNZNWOGOAeQ6tx+W3bzkQXF0QFjG192Ot7LxaghXEEqXlhu5JqLvP8uL4Y6+Vc= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR08MB5303 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=40.107.14.98; envelope-from=vsementsov@virtuozzo.com; helo=EUR01-VE1-obe.outbound.protection.outlook.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/06/04 04:33:56 X-ACL-Warn: Detected OS = Windows NT kernel [generic] [fuzzy] X-Spam_score_int: 5 X-Spam_score: 0.5 X-Spam_bar: / X-Spam_report: (0.5 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, MSGID_FROM_MTA_HEADER=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H2=-0.001, RCVD_IN_SBL_CSS=3.335, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001 autolearn=_AUTOLEARN X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, fam@euphon.net, thuth@redhat.com, vsementsov@virtuozzo.com, quintela@redhat.com, alex.bennee@linaro.org, qemu-devel@nongnu.org, mreitz@redhat.com, stefanha@redhat.com, den@openvz.org, jsnow@redhat.com, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: pass (identity @virtuozzo.com) Content-Type: text/plain; charset="utf-8" It is possible, that shutdown on target occurs earlier than migration finish. In this case we crash in bdrv_release_dirty_bitmap_locked() on assertion "assert(!bdrv_dirty_bitmap_busy(bitmap));" as we do have busy bitmap, as bitmap migration is ongoing. We'll fix bitmap migration to gracefully cancel on early shutdown soon. Now let's fix iotest 194 to wait migration completion before shutdown. Note that in this test dest_vm.shutdown() is called implicitly, as vms used as context-providers, see __exit__() method of QEMUMachine class. Actually, not waiting migration finish is a wrong thing, but the test started to crash after commit ae00aa239847682 "iotests: 194: test also migration of dirty bitmap", which added dirty bitmaps here. So, Fixes: tag won't hurt. Fixes: ae00aa2398476824f0eca80461da215e7cdc1c3b Reported-by: Thomas Huth Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Eric Blake Tested-by: Thomas Huth --- tests/qemu-iotests/194 | 10 ++++++++++ tests/qemu-iotests/194.out | 5 +++++ 2 files changed, 15 insertions(+) diff --git a/tests/qemu-iotests/194 b/tests/qemu-iotests/194 index 3fad7c6c1a..6dc2bc94d7 100755 --- a/tests/qemu-iotests/194 +++ b/tests/qemu-iotests/194 @@ -87,4 +87,14 @@ with iotests.FilePath('source.img') as source_img_path, \ iotests.log(dest_vm.qmp('nbd-server-stop')) break =20 + iotests.log('Wait migration completion on target...') + migr_events =3D (('MIGRATION', {'data': {'status': 'completed'}}), + ('MIGRATION', {'data': {'status': 'failed'}})) + event =3D dest_vm.events_wait(migr_events) + iotests.log(event, filters=3D[iotests.filter_qmp_event]) + + iotests.log('Check bitmaps on source:') iotests.log(source_vm.qmp('query-block')['return'][0]['dirty-bitmaps']) + + iotests.log('Check bitmaps on target:') + iotests.log(dest_vm.qmp('query-block')['return'][0]['dirty-bitmaps']) diff --git a/tests/qemu-iotests/194.out b/tests/qemu-iotests/194.out index dd60dcc14f..f70cf7610e 100644 --- a/tests/qemu-iotests/194.out +++ b/tests/qemu-iotests/194.out @@ -21,4 +21,9 @@ Gracefully ending the `drive-mirror` job on source... {"data": {"device": "mirror-job0", "len": 1073741824, "offset": 1073741824= , "speed": 0, "type": "mirror"}, "event": "BLOCK_JOB_COMPLETED", "timestamp= ": {"microseconds": "USECS", "seconds": "SECS"}} Stopping the NBD server on destination... {"return": {}} +Wait migration completion on target... +{"data": {"status": "completed"}, "event": "MIGRATION", "timestamp": {"mic= roseconds": "USECS", "seconds": "SECS"}} +Check bitmaps on source: +[{"busy": false, "count": 0, "granularity": 65536, "name": "bitmap0", "per= sistent": false, "recording": true, "status": "active"}] +Check bitmaps on target: [{"busy": false, "count": 0, "granularity": 65536, "name": "bitmap0", "per= sistent": false, "recording": true, "status": "active"}] --=20 2.21.0