From nobody Fri Mar 29 00:37:21 2024 Delivered-To: importer@patchew.org Received-SPF: none (zoho.com: 192.237.175.120 is neither permitted nor denied by domain of lists.xenproject.org) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=none (zoho.com: 192.237.175.120 is neither permitted nor denied by domain of lists.xenproject.org) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org ARC-Seal: i=1; a=rsa-sha256; t=1557821260; cv=none; d=zoho.com; s=zohoarc; b=PjRgpTxLt3gtiuozukzcxz7uMwkYmalUTMIk5V3gWEUO6gkaEt+eJUSIZcgT4cpQ9uhgWKSSK4AGVjEjVvn2xv+lKVFEE8AjMpc5tyoypop6z15o8zRQhCozPGMYTXo3ZZls7tjp1oHRzJruGeAWPQFV37zo25OxYg9F2ojYCIw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1557821260; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To:ARC-Authentication-Results; bh=P+D8E2hTUg2nhs2GQUV6l+jz0uVCW8PgmluUgJ9d/qg=; b=P4eSX8LT1DkAy43aTTPaRsCsWUVRCdRs0M/605z4Nj9J/NrNXa915IVqgnleF8geRHYKk1GPuiIn0HFxbVBrT3DfFS6IJJTwnwfgzveIYvNDlsSe2aixRirHfGH/FnQQu9GTsLeAzTxKltdfD3wKPecNUVLvUc0S17CyWgKPOGw= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=fail; spf=none (zoho.com: 192.237.175.120 is neither permitted nor denied by domain of lists.xenproject.org) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1557821259028647.9662411408483; Tue, 14 May 2019 01:07:39 -0700 (PDT) Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1hQSRx-00007s-Rx; Tue, 14 May 2019 08:06:13 +0000 Received: from us1-rack-dfw2.inumbo.com ([104.130.134.6]) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1hQSRx-00007n-0Y for xen-devel@lists.xenproject.org; Tue, 14 May 2019 08:06:13 +0000 Received: from mo6-p01-ob.smtp.rzone.de (unknown [2a01:238:20a:202:5301::3]) by us1-rack-dfw2.inumbo.com (Halon) with ESMTPS id 220fa23f-761f-11e9-8980-bc764e045a96; Tue, 14 May 2019 08:06:11 +0000 (UTC) Received: from sender by smtp.strato.de (RZmta 44.20 DYNA|AUTH) with ESMTPSA id U080cav4E8624NR (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (curve secp521r1 with 521 ECDH bits, eq. 15360 bits RSA)) (Client did not present a certificate); Tue, 14 May 2019 10:06:02 +0200 (CEST) X-Inumbo-ID: 220fa23f-761f-11e9-8980-bc764e045a96 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1557821169; s=strato-dkim-0002; d=aepfle.de; h=Message-Id:Date:Subject:Cc:To:From:X-RZG-CLASS-ID:X-RZG-AUTH:From: Subject:Sender; bh=dMxwTJpBfGGdB+l310X9kkFmB63lDFNyAgGWNgbRUS8=; b=KtCR9QX6qG29YDKIVUzDyCVQ9qE2cx77ms0JuRkWDbUcVjaeq6owrhVOUh/sbOkm/z 8+fq8+l85RQW3LbCyV/MvRksQeGu6by5PwStLwo6/aQ2KjqiAGzkJ8x6a7Tr//XPJUu3 QfD38WHRaoC67wY/zq+ieTlsXPAmw6PyVeJMC/wHPxsZiR3PjOa+9sY8TAhKFxOWGxWZ E8Ppk1OXwUtMJ52013AIWGQKVWT2UeiysDgPpLBvFHALHfIZEkQezi7e9HI4YUGz8bJF 86RkFDYOY2Utuvc5Sr+ISDKVKx4GKksj4moIA8IO5U5HpuzMV3UtoAt4BEccCcn2JT7B HQRA== X-RZG-AUTH: ":P2EQZWCpfu+qG7CngxMFH1J+3q8wa/QXkBR9MXjAuzBW/OdlBZQ4AHSS3GpFjw==" X-RZG-CLASS-ID: mo00 From: Olaf Hering To: xen-devel@lists.xenproject.org Date: Tue, 14 May 2019 10:05:58 +0200 Message-Id: <20190514080558.14540-1-olaf@aepfle.de> X-Mailer: git-send-email 2.16.4 MIME-Version: 1.0 Subject: [Xen-devel] [PATCH v5] libxl: fix migration of PV and PVH domUs with and without qemu X-BeenThere: xen-devel@lists.xenproject.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Cc: Anthony PERARD , Wei Liu , Olaf Hering , Ian Jackson , =?UTF-8?q?Roger=20Pau=20Monn=C3=A9?= Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) If a domU has a qemu-xen instance attached, it is required to call qemus "xen-save-devices-state" method. Without it, the receiving side of a PV or PVH migration may be unable to lock the image: xen be: qdisk-51712: xen be: qdisk-51712: error: Failed to get "write" lock error: Failed to get "write" lock xen be: qdisk-51712: xen be: qdisk-51712: initialise() failed initialise() failed To fix this bug, libxl__domain_suspend_device_model() and libxl__domain_resume_device_model() have to be called not only for HVM, but also if the active device_model is QEMU_XEN. Unfortunately, libxl__domain_build_info_setdefault() used to hardcode b_info->device_model_version to QEMU_XEN if it does not know it any better. As a result libxl__device_model_version_running() will return incorrect values. This breaks domUs without a device_model. libxl__qmp_stop() would wait 10 seconds in qmp_open() for a qemu that will never appear. During this long timeframe the domU remains in state paused on the sending side. As a result network connections may be dropped. Once this bug is fixed as well, by just removing the assumption that every domU has a QEMU_XEN, there is no code to actually initialise b_info->device_model_version. There is a helper function libxl__need_xenpv_qemu(), which is used in various places to decide if a device_model has to be spawned. This function can not be used as is, just to fill device_model_version, because store_libxl_entry() was already called earlier. Introduce LIBXL_DEVICE_MODEL_VERSION_NONE for PV and PVH that have no need for a device_model to make the state explicit. Indicate this new state via LIBXL_HAVE macro in libxl.h. v05: - move initialization of device_model_version to extra commit - return error from libxl__need_xenpv_qemu - add LIBXL_HAVE_DEVICE_MODEL_VERSION_NONE v04: - make sure device_model_stubdomain is initialized v03: - rearrange code to make sure device_model_version is initialized before store_libxl_entry() is called v02: - update wording in a comment - remove stale goto in domcreate_launch_dm - initialize ret in libxl__need_xenpv_qemu Signed-off-by: Olaf Hering Cc: Roger Pau Monn=C3=A9 Cc: Anthony PERARD Reviewed-by: Roger Pau Monn=C3=A9 --- tools/libxl/libxl.h | 7 +++++++ tools/libxl/libxl_create.c | 17 ++++++++++++++--- tools/libxl/libxl_dom_suspend.c | 8 ++++++-- tools/libxl/libxl_types.idl | 1 + 4 files changed, 28 insertions(+), 5 deletions(-) diff --git a/tools/libxl/libxl.h b/tools/libxl/libxl.h index 482499a6c0..e0f6916b66 100644 --- a/tools/libxl/libxl.h +++ b/tools/libxl/libxl.h @@ -1182,6 +1182,13 @@ void libxl_mac_copy(libxl_ctx *ctx, libxl_mac *dst, = const libxl_mac *src); */ #define LIBXL_HAVE_PVCALLS 1 =20 +/* + * LIBXL_HAVE_DEVICE_MODEL_VERSION_NONE + * + * If this is defined, libxl will only run a device-model if required. + */ +#define LIBXL_HAVE_DEVICE_MODEL_VERSION_NONE 1 + typedef char **libxl_string_list; void libxl_string_list_dispose(libxl_string_list *sl); int libxl_string_list_length(const libxl_string_list *sl); diff --git a/tools/libxl/libxl_create.c b/tools/libxl/libxl_create.c index 3f0431cc84..64336b0d29 100644 --- a/tools/libxl/libxl_create.c +++ b/tools/libxl/libxl_create.c @@ -47,9 +47,20 @@ int libxl__domain_set_device_model(libxl__gc *gc, libxl_= domain_config *d_config) } break; default: - b_info->device_model_version =3D - LIBXL_DEVICE_MODEL_VERSION_QEMU_XEN_TRADITIONAL; - break; + ret =3D libxl__need_xenpv_qemu(gc, d_config); + switch (ret) { + case 1: + d_config->b_info.device_model_version =3D + LIBXL_DEVICE_MODEL_VERSION_QEMU_XEN; + break; + case 0: + d_config->b_info.device_model_version =3D + LIBXL_DEVICE_MODEL_VERSION_NONE; + break; + default: + LOGE(ERROR, "Unable to determine QEMU requisite"); + return ret; + } } =20 if (b_info->device_model_version =3D=3D LIBXL_DEVICE_MODEL_VERSION_QEM= U_XEN) { diff --git a/tools/libxl/libxl_dom_suspend.c b/tools/libxl/libxl_dom_suspen= d.c index d1af3a6573..c492fe5dd1 100644 --- a/tools/libxl/libxl_dom_suspend.c +++ b/tools/libxl/libxl_dom_suspend.c @@ -379,7 +379,9 @@ static void domain_suspend_common_guest_suspended(libxl= __egc *egc, libxl__ev_xswatch_deregister(gc, &dsps->guest_watch); libxl__ev_time_deregister(gc, &dsps->guest_timeout); =20 - if (dsps->type =3D=3D LIBXL_DOMAIN_TYPE_HVM) { + if (dsps->type =3D=3D LIBXL_DOMAIN_TYPE_HVM || + libxl__device_model_version_running(gc, dsps->domid) =3D=3D + LIBXL_DEVICE_MODEL_VERSION_QEMU_XEN) { dsps->callback_device_model_done =3D domain_suspend_common_done; libxl__domain_suspend_device_model(egc, dsps); /* must be last */ return; @@ -459,7 +461,9 @@ int libxl__domain_resume(libxl__gc *gc, uint32_t domid,= int suspend_cancel) goto out; } =20 - if (type =3D=3D LIBXL_DOMAIN_TYPE_HVM) { + if (type =3D=3D LIBXL_DOMAIN_TYPE_HVM || + libxl__device_model_version_running(gc, domid) =3D=3D + LIBXL_DEVICE_MODEL_VERSION_QEMU_XEN) { rc =3D libxl__domain_resume_device_model(gc, domid); if (rc) { LOGD(ERROR, domid, "failed to resume device model:%d", rc); diff --git a/tools/libxl/libxl_types.idl b/tools/libxl/libxl_types.idl index cb4702fd7a..75bde095bc 100644 --- a/tools/libxl/libxl_types.idl +++ b/tools/libxl/libxl_types.idl @@ -106,6 +106,7 @@ libxl_device_model_version =3D Enumeration("device_mode= l_version", [ (0, "UNKNOWN"), (1, "QEMU_XEN_TRADITIONAL"), # Historical qemu-xen device model (qemu-= dm) (2, "QEMU_XEN"), # Upstream based qemu-xen device model + (3, "NONE"), ]) =20 libxl_console_type =3D Enumeration("console_type", [ _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel