From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from AM0PR83CU005.outbound.protection.outlook.com (mail-westeuropeazon11020112.outbound.protection.outlook.com [52.101.69.112]) by lore.virtuozzo.com (Postfix) with ESMTPS id 4A90E802A5 for ; Fri, 9 Oct 2026 14:56:32 +0000 (UTC) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=XX7NA+Pdn7ihb7TjaJCNl3IASLo0uzf19//KgHd6oCnw7WkfXA/su8KqTzli3+SiSc7T7Fcxg76eReKNpwQGEXC4dhbsR5z30BbmiO5mvZkFVr3mKEcfeE+xcWcU+VElub4l0nO1E30cM1dM3giUPoKLyinAVEr60KDHkdgpENgZ8Fjh/3uH7Hz8A6zN8P7jVu5ISqJB3wZ76GRCGcsqScIs/r9NWjc24uOa2hLClEi0NFuSVV6yWDqS/wV+EdUWhIzi2ApOOsIqFWwWkCIRKLNOJDN1dzsQnIqMrEkg3moxqjLAQ2kLMBXMVS3v5y3il43Mkf2u2eLox9t633dvvw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=HOCPSiuOtfXURWTpZBtSzFrqss+O8pvaRrDB34eGAxo=; b=jwqWa/G4XGdHlr3NoHzsCHoyUsEl3za4kIeYUk5FmpBxG4HAx588bcGDHKMhbB8J2CSfDqaQfZn2lbw/bgyfmCuvM7BrWuH2C3DCtEDzrFIbvpULGWyI+ejGX47WQN0cf2vG8jNuPv3FD7tu9xp2eU2m2kn9p+hvxLbP4C5K+8Dojm9JpjupMVZjBTAcCDhkjZ/oaw8+qkGNkywjEmz6TO5wgGVuRRtPcVIL/VJVEKgh3ii4hohbTISkSw/TWnzUG7i9dTzoV1rgkX6Q3zA315PBPhCZcUBDBSzivxyd8tH3YxDo13VWhi6MGGkalDTvUPaDrVPlf69g5VYnBpfqvg== 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=HOCPSiuOtfXURWTpZBtSzFrqss+O8pvaRrDB34eGAxo=; b=YccenlmopSyGOtZQuCUMDaForqJlphovbvIEaHL96R+4bKD4pYV8HvwrjNXTiVpUFWwT69FJq3sJJmzMFYvH+Ie6WJBlg/BXEfrsXKNCvbuK1cVQ5FaxJn+Ys3V32O8FvGBLOqxkHbRTVl/gXbx1f5Rt0WEs5415moTRmwEFSO5/VAGkuO8getGqhqSsBbmKTzxqfpkc4BJE17H3q/iFqQFPa7xkQ/gIkeJGwD6l1xmVwIt++N602FT15ccHQ2Df5HNzRmNSaYOxnvD6wezil2yz/gHj4Yl3SG6sXzyLMQ6Enlor3BncNgrXcOsqmw2/PJOMTODlpxp3uy2bdhvjng== Authentication-Results: mx.microsoft.com 1; dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=virtuozzo.com; Received: from VI0PR08MB10656.eurprd08.prod.outlook.com (2603:10a6:800:20a::12) by DB3PR08MB9110.eurprd08.prod.outlook.com (2603:10a6:10:428::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.496.17; Fri, 9 Oct 2026 14:56:28 +0000 Received: from VI0PR08MB10656.eurprd08.prod.outlook.com ([fe80::4e37:b189:ddcd:3dd8]) by VI0PR08MB10656.eurprd08.prod.outlook.com ([fe80::4e37:b189:ddcd:3dd8%7]) with mapi id 15.21.0496.010; Fri, 9 Oct 2026 14:56:28 +0000 From: Andrey Drobyshev Date: Fri, 09 Oct 2026 17:56:18 +0300 Subject: [PATCH QEMU HCI-8.1 19/21] VZ: migration/cpr-exec: abort cpr-exec if memory can't be preserved Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20261009-vfio_v1-v1-19-2ae91d5b8048@virtuozzo.com> References: <20261009-vfio_v1-v1-0-2ae91d5b8048@virtuozzo.com> In-Reply-To: <20261009-vfio_v1-v1-0-2ae91d5b8048@virtuozzo.com> To: svt-core@virtuozzo.com Cc: Andrey Drobyshev X-Mailer: b4 0.16.0 X-ClientProxiedBy: VIUP296CA0132.AUTP296.PROD.OUTLOOK.COM (2603:10a6:800:34a::14) To VI0PR08MB10656.eurprd08.prod.outlook.com (2603:10a6:800:20a::12) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VI0PR08MB10656:EE_|DB3PR08MB9110:EE_ X-MS-Office365-Filtering-Correlation-Id: e9f860e8-1159-4d21-aa96-08df2615761b X-LD-Processed: 0bc7f26d-0264-416e-a6fc-8352af79c58f,ExtAddr List-Id: svt-core@virtuozzo.com X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|366016|1800799024|52116014|23010399003|56012099006|38350700014|10067099003|22082099003|5113699003|18002099003; X-Microsoft-Antispam-Message-Info: 5/j5RPNl0Wsea/mtVPYBuaeFZmPn6979KW4HYuHW0CKCSLjDuC64AKjGCgPxzH8fPTznvmthdcTMwiAtc/UD/dnV8125nV0Ph0bfSAPV4Ax/NTkMfNt7SDQZG3/YZ8aY+vMbIlUrbgNu5PlLoi22eSs0Ah++Pg1Ty9mnshXKHpoR9q7XTWhfL21ChsAh336HpFVBu2iUjOxzdG+LsDM/D+muFVUa1rKk01pKKCDDx7isi013euStG7b4RUmSk6TMTNgmfCwRnAHx9x+AnMMzxb4ktP0d++nyv6e2PXtDM0i4mi1uyE9dH2pjnHcuf08oMr0ohwqrKXwk42geU0ZOWeILU5ylnMFk/fJcAOI62aUvzr31s19dchzNi8uQ9ouUqcZOsnOiTLgGt+QdHGh2atYF8/6PiAePl0yYaBoERBTOXbXzbTPEQTfKc9yP1PzHkWmDibybDnPUEII0G1h85WtENiGZTNJlrEdeWXvMCOdBFhcUxT/Q6ZPzPTa7s83kYARZPZmS4cXHnY2Y1QjS3ZL55rxg3/e43F488PfU1DHL8/SjsH6rNwzvL60bTrsmos72IxXXYZnLrbOLAo9JwPP44ub04r1JAN0t/CzoGR9PjTB0WRpPO/kkM+VvVtG4LEew3HLx/1yZNvL96RyTZJV+mTp54cLU0E4OMjwGcGx14oxzxREt6kW2sHvPFdce1IFYjOb9Ehhond0+QRjaZuF3wm/0gLBTg1iSB+i3wKE= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI0PR08MB10656.eurprd08.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(366016)(1800799024)(52116014)(23010399003)(56012099006)(38350700014)(10067099003)(22082099003)(5113699003)(18002099003);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?LzNldjA4eXFwSUpiM05OdGMyOVRaZ21ndS9FcUxUL05IK0UweVR3SjlTbXhH?= =?utf-8?B?TE4rTkxhMW5VS2hQUVZNek9RdGJLRHlEWFlMWmZTVVNxc2c3QkJYYXFzM0xK?= =?utf-8?B?SzNzOGZhUnN4OVNPYWR4SllCbjR5NlVSMXhJVU4rS0JuQkg3OW1GbERJSUh6?= =?utf-8?B?cEhzTTZsdXVsM1U4M0xuYU1VVFdwR3dqUGxPMmRZbFcyMUNBeElBWWNXck83?= =?utf-8?B?c1V2L0grOEdKRStLTm8rN1dGemRCVGl0Q1M5NXdaem9Qd2dVTWl6eHc4cVQv?= =?utf-8?B?WU1idVlpcW5rSzVDWnBNdnQ5S2phcGNybDBTZFlmUVZPVU5ESEEwY3RPeGtH?= =?utf-8?B?bEJmZXZZVElWVkJtaHRDckdjdHZVRHg3endhZkhUeklMUWdpVG1JdlUycFFr?= =?utf-8?B?YWhsN1h6NUFxVU9CRVRQSFg4cEtWSmJjL0VoMmpxYXRPU1R0bmY1QTg1dU1y?= =?utf-8?B?ZkQ3cTVNcE8veGNaSUhNbk1yUURWdFJjZmZIRGxocmphbEtVc3Y0SHg2cVNv?= =?utf-8?B?WXJwc1I2NUNLa3hISDFSKzczdytIT3BZcVRoNGFyTjZLOFNCclpubGdRQ09Y?= =?utf-8?B?M3ZBSnlWN1BGQlFEMTdyc3RDU1o2eXkvOWlZNDNGa215dUtHY0lMVmViMGhK?= =?utf-8?B?OUVCeHltVjJ6NHhOd1J4SkRmMUdKZGdna0IrQUQ2aEZvaHpYc2JLQ1U0cmY4?= =?utf-8?B?dXNHbnBEUVJ5ZE5EZU1COWFycWJBOGJsZHBjT1VFTDVnb0pqL3NGSW1lL2tt?= =?utf-8?B?YXVkU1VBRjBraU44Ykd6UE93amsyMjcwa2ZUSHVDdmxGakVRbWJPZXFjRFRZ?= =?utf-8?B?aWwxQnFNVEVhTjBROHVNZnlSd0pWTUMzL21qL2xxRXFXZDVUL1g4NTV4Skcr?= =?utf-8?B?WVBDSE5pbXc1Z1pMY1hQZDREd3RjWnovY2Zwc01sUkZscXhvTVJJMEROWjFC?= =?utf-8?B?cXlKMnltTnI5bnlmcmlPdzJidmswcTN1ZUJmSWVONXNyajhUVldoNm5XSWtl?= =?utf-8?B?VUNGVHJGUnZIOFJ2VnVscEswWkZXY1lxZGhsU3MzLzhlNE1nQkVReEE5M0dE?= =?utf-8?B?cUhaWWFkSytIWUExcklTRTNvKzNnQ3lCOFJrUWhpS1JXYnI5anBXTTRHYUND?= =?utf-8?B?UWhRV2tDdFJnVW9vd2ZwYkdMS3d5YjZsMllOV3EybDBmUmY3elJzWndvUDZi?= =?utf-8?B?N2RrV09vZDBVQm00OWRlTmJGNXVaWWk3K3lKQ08zek1GWmlsWXhwanAvb0dZ?= =?utf-8?B?YU8rZFVHRnN4UEc2UU9xVFM2dWthUzF6S1UyNzVBSWkyQmZsemxuUTFjeURG?= =?utf-8?B?RjlTdjRHY0xJcy9ZbllsSktLS2c0bnlLUU4xTlJ5WDFoQ1VmVkZ2VVVyZXFG?= =?utf-8?B?a3NxK2ROcklUVU9NOVg0R3c2cVJpU1duZElvcFoyaHpnTE1yU0ZDeTFHMlg3?= =?utf-8?B?SGdsK0V1c0s5UjJMM3VyazNHSWdhQ05PdjV6UUd4YkN1OEFrbTJaa0tmQ2Zj?= =?utf-8?B?dEpKYzVLTDdXeFBoZURMd3gzTERMbkdvSlRjaXpsSERHNXFMb1dtMnVpWHVE?= =?utf-8?B?YTFOWjJ5RGpPYnNLcnM2WWZhZndvQy84T01lZnU5M1NRMHJuODJ3N1FBM0U4?= =?utf-8?B?YmNrUEJJUHIzUm5yR2wycXBZa0l5R3JTc3ZuZkVwRFBZQWxRS0lEVUVUUTNp?= =?utf-8?B?aTlGTlhtem96YWNLVkJnME8zRk9kanl0YkptQlJScmVaREN2S2tjcUY3TFZV?= =?utf-8?B?RjdyUXhNeUhVUitGNXZUNmZIMnNPdE1aaGJhSVRiaStNalB5K1dsOTdZRXBI?= =?utf-8?B?Q2ZFeTF0bEsrZHRwWUtySXBDTUhzYi9ReHpRMUpCajR5Y2MrRGZSMFhxMlFo?= =?utf-8?B?VWpYWE1KSFNha3NNRzJkZTd5aUhtbE9abnh6SFMycUxIK01saEZIWU0vZmdp?= =?utf-8?B?UTFMSGhHVzJ6cHQxdm5BeXg4Zzk0bWxiTkN5THpZK0lIUU5uUGZpdnBQVzZL?= =?utf-8?B?eWJzSXVZOWVuT3ZBa3N6VU9xZmJqU1ZuZkRIZ05hRUhqUVkzUTB1Tnp3OHdH?= =?utf-8?B?ZllFNlJ2T1dXL0MzZmtadlU1SlpyZEdBQy9MR2gwUHFZT0VuczlyQThjNitx?= =?utf-8?B?aThtSlZNa3YxcFlMeDdBQ3VOQ1JUVVprTC93SHQzV2ZwSHk4T05uWnRFaWEv?= =?utf-8?B?N3JGS3dMRS9uRldMeTN2UythazVLYlBHQnluckZWa1lIekhWQmFlT1pMRE1Z?= =?utf-8?B?U1RGZW4rU095dDVJTDhBY3hCY3pBRnBaMXJXelRYZDZqTlRubkp6QzRhSGll?= =?utf-8?B?bE5Dc1ZTeHk4bVZUVm9SWkRNRDJwcC9hZTdKVit0NzJlTjZMNXRLdURRNEc5?= =?utf-8?Q?vRzgmsfr5hXMjwtE=3D?= X-Auto-Response-Suppress: DR, OOF, AutoReply X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: e9f860e8-1159-4d21-aa96-08df2615761b X-MS-Exchange-CrossTenant-AuthSource: VI0PR08MB10656.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Oct 2026 14:56:12.8556 (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: xMyENPZJ7a4+itdnE2uG24q8AGHQudZtBD2u7vBOQX+LSw7sfAZnhqSJKkhoYvHQmt0d80ZKk4J7uqElLpq9xQ/Y1B3HXsd0Z9MnMdTTSFA= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB3PR08MB9110 cpr_exec_cb() marks all saved ramblocks with MADV_DOEXEC before exec, but ignores the result. If madvise() fails for some block, the walk stops there and the remaining blocks are not marked, yet execvp() still runs. The new QEMU then reuses addresses of guest RAM that wasn't preserved, and the guest is lost. Treat that as a failed cpr-exec: go to the same failure path as when execvp() fails, so the VM keeps running in the old QEMU. On that path also clear MADV_DOEXEC from all saved ramblocks again, just like the preserved fds get close-on-exec back. Signed-off-by: Andrey Drobyshev --- migration/cpr-exec.c | 26 ++++++++++++++++++++++---- 1 file changed, 22 insertions(+), 4 deletions(-) diff --git a/migration/cpr-exec.c b/migration/cpr-exec.c index 7b7609e19cd..cb962ce7252 100644 --- a/migration/cpr-exec.c +++ b/migration/cpr-exec.c @@ -154,6 +154,19 @@ static bool preserve_ram(cpr_ramblock_t ram, char *name) return true; } +static bool unpreserve_ram(cpr_ramblock_t ram, char *name) +{ +#ifdef MADV_DONTEXEC + qemu_madvise((void *) ram.addr, ram.length, MADV_DONTEXEC); +#endif + return true; +} + +static void cpr_exec_unpreserve_ram(void) +{ + cpr_walk_ramblock(unpreserve_ram); +} + static bool mark_ram_rdonly(cpr_ramblock_t ram, char *name) { /* @@ -202,7 +215,10 @@ static void cpr_exec_cb(void *opaque) */ cpr_exec_preserve_fds(); - cpr_walk_ramblock(preserve_ram); + if (!cpr_walk_ramblock(preserve_ram)) { + /* preserve_ram() has already reported the error and failed migration */ + goto fail; + } if (cpr_ram_rdonly) { cpr_walk_ramblock(mark_ram_rdonly); } @@ -221,15 +237,17 @@ static void cpr_exec_cb(void *opaque) * or the system is very short on resources. */ error_setg_errno(&err, errno, "execvp %s failed", argv[0]); - g_clear_pointer(&argv, g_strfreev); - cpr_exec_unpreserve_fds(); - error_report_err(error_copy(err)); migrate_set_state(&s->state, s->state, MIGRATION_STATUS_FAILED); migrate_set_error(s, err); error_free(err); err = NULL; +fail: + g_clear_pointer(&argv, g_strfreev); + cpr_exec_unpreserve_fds(); + cpr_exec_unpreserve_ram(); + /* Note, we can go from state COMPLETED to FAILED */ migration_call_notifiers(MIG_EVENT_PRECOPY_FAILED, NULL); -- 2.47.1