From patchwork Thu Jan 21 16:51:31 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Roger_Pau_Monn=C3=A9?= X-Patchwork-Id: 8083211 Return-Path: X-Original-To: patchwork-xen-devel@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 5C1329F96D for ; Thu, 21 Jan 2016 16:55:00 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 1BC60204A0 for ; Thu, 21 Jan 2016 16:54:59 +0000 (UTC) Received: from lists.xen.org (lists.xenproject.org [50.57.142.19]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id C22C9204A7 for ; Thu, 21 Jan 2016 16:54:57 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=lists.xen.org) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1aMITB-0003iB-Dn; Thu, 21 Jan 2016 16:52:25 +0000 Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1aMITA-0003hT-B5 for xen-devel@lists.xenproject.org; Thu, 21 Jan 2016 16:52:24 +0000 Received: from [85.158.137.68] by server-3.bemta-3.messagelabs.com id C8/F4-02499-7CC01A65; Thu, 21 Jan 2016 16:52:23 +0000 X-Env-Sender: prvs=8211775ec=roger.pau@citrix.com X-Msg-Ref: server-11.tower-31.messagelabs.com!1453395138!16777457!2 X-Originating-IP: [66.165.176.63] X-SpamReason: No, hits=0.0 required=7.0 tests=sa_preprocessor: VHJ1c3RlZCBJUDogNjYuMTY1LjE3Ni42MyA9PiAzMDYwNDg=\n, received_headers: No Received headers X-StarScan-Received: X-StarScan-Version: 7.35.1; banners=-,-,- X-VirusChecked: Checked Received: (qmail 5820 invoked from network); 21 Jan 2016 16:52:22 -0000 Received: from smtp02.citrix.com (HELO SMTP02.CITRIX.COM) (66.165.176.63) by server-11.tower-31.messagelabs.com with RC4-SHA encrypted SMTP; 21 Jan 2016 16:52:22 -0000 X-IronPort-AV: E=Sophos;i="5.22,326,1449532800"; d="scan'208";a="333115614" From: Roger Pau Monne To: Date: Thu, 21 Jan 2016 17:51:31 +0100 Message-ID: <1453395092-88090-6-git-send-email-roger.pau@citrix.com> X-Mailer: git-send-email 1.9.5 (Apple Git-50.3) In-Reply-To: <1453395092-88090-1-git-send-email-roger.pau@citrix.com> References: <1453395092-88090-1-git-send-email-roger.pau@citrix.com> MIME-Version: 1.0 X-DLP: MIA1 Cc: Wei Liu , Ian Jackson , Ian Campbell , Roger Pau Monne Subject: [Xen-devel] [PATCH v4 5/6] libxl: add options to enable/disable emulated devices X-BeenThere: xen-devel@lists.xen.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Allow enabling or disabling emulated devices from the libxl domain configuration file. For HVM guests with a device model all the emulated devices are enabled. For HVM guests without a device model no devices are enabled by default, although they can be enabled using the options provided. The arbiter of whether a combination is posible or not is always Xen, libxl doesn't do any kind of check. This set of options is also propagated inside of the libxl migration record as part of the contents of the libxl_domain_build_info struct, so that when the other end (restore) creates the domain the same set of devices are enabled. This is important for future compatibility, in case we decide to enable some emulated devices by default for HVMlite guests, old HVMlite guests migrated to newer versions should continue to see the same set of emulated devices. It has been discussed that it would be better to avoid having this information inside of the libxl stream, and to instead rely on which devices get their context loaded inside of Xen on resume. This of course requires more work and it also has certain issues, like the fact that some devices don't restore a context at all (like VGA). The consensus is that the solution presented in this patch is not going to prevent further developments in this direction, and can always be used as a check to make sure the Xen context and the libxl context are in sync. Signed-off-by: Roger Pau Monné --- Cc: Ian Jackson Cc: Ian Campbell Cc: Wei Liu --- Changes since v3: - Fix spelling mistake in xl.cfg man page. - Error out if a classic HVM guest tries to use some of the HVMlite options. - Modify libxl__update_domain_configuration so the fields relevant to the set of enabled devices are stored. - Expand commit message to clarify the approach taken. --- docs/man/xl.cfg.pod.5 | 43 +++++++++++++++++++++++++++++++++++++++++++ tools/libxl/libxl.h | 11 +++++++++++ tools/libxl/libxl_create.c | 26 +++++++++++++++++++++++++- tools/libxl/libxl_internal.c | 25 +++++++++++++++++++++++++ tools/libxl/libxl_types.idl | 6 ++++++ tools/libxl/libxl_x86.c | 33 ++++++++++++++++++++++++++++----- tools/libxl/xl_cmdimpl.c | 7 +++++++ 7 files changed, 145 insertions(+), 6 deletions(-) diff --git a/docs/man/xl.cfg.pod.5 b/docs/man/xl.cfg.pod.5 index 8899f75..dc47888 100644 --- a/docs/man/xl.cfg.pod.5 +++ b/docs/man/xl.cfg.pod.5 @@ -1762,6 +1762,49 @@ See F for more information. =back +=head3 HVM without a device model options + +These options can be used to change the set of emulated devices provided +to guests without a device model. Note that Xen might not support all +possible combinations. By default HVM guests without a device model +don't have any of them enabled. + +It is important to notice that these options (except the hpet one) are +not available to HVM guests with a device model, and trying to set them +will cause xl to exit with an error. + +=over 4 + +=item B + +Enables or disables the Local APIC. + +=item B + +Enables or disables the IO APIC. + +=item B + +Enables or disables the RTC. + +=item B + +Enables or disables the ACPI power management timer and control interfaces. + +=item B + +Enables or disables the PIC. + +=item B + +Enables or disables the PIT. + +=item B + +Enables or disables the HPET. + +=back + =head2 Device-Model Options The following options control the selection of the device-model. This diff --git a/tools/libxl/libxl.h b/tools/libxl/libxl.h index ab76d85..228f192 100644 --- a/tools/libxl/libxl.h +++ b/tools/libxl/libxl.h @@ -886,6 +886,17 @@ void libxl_mac_copy(libxl_ctx *ctx, libxl_mac *dst, libxl_mac *src); */ #define LIBXL_HAVE_VGA_INTERFACE_TYPE_UNKNOWN 1 +/* + * LIBXL_HAVE_EMULATED_DEVS_OPTIONS + * + * In the case that LIBXL_HAVE_EMULATED_DEVS_OPTIONS is set libxl + * allows enabling or disabling emulated devices for HVM guests + * without a device model. The following fields are added to the + * hvm structure inside of libxl_domain_build_info: lapic, ioapic, + * rtc, power_management, pic, pit. + */ +#define LIBXL_HAVE_EMULATED_DEVS_OPTIONS 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 c7700a7..7d0f570 100644 --- a/tools/libxl/libxl_create.c +++ b/tools/libxl/libxl_create.c @@ -295,13 +295,37 @@ int libxl__domain_build_info_setdefault(libxl__gc *gc, libxl_defbool_setdefault(&b_info->u.hvm.acpi_s4, true); libxl_defbool_setdefault(&b_info->u.hvm.nx, true); libxl_defbool_setdefault(&b_info->u.hvm.viridian, false); - libxl_defbool_setdefault(&b_info->u.hvm.hpet, true); libxl_defbool_setdefault(&b_info->u.hvm.vpt_align, true); libxl_defbool_setdefault(&b_info->u.hvm.nested_hvm, false); libxl_defbool_setdefault(&b_info->u.hvm.altp2m, false); libxl_defbool_setdefault(&b_info->u.hvm.usb, false); libxl_defbool_setdefault(&b_info->u.hvm.xen_platform_pci, true); + if (b_info->device_model_version == LIBXL_DEVICE_MODEL_VERSION_NONE) + { + libxl_defbool_setdefault(&b_info->u.hvm.hpet, false); + libxl_defbool_setdefault(&b_info->u.hvm.lapic, false); + libxl_defbool_setdefault(&b_info->u.hvm.ioapic, false); + libxl_defbool_setdefault(&b_info->u.hvm.rtc, false); + libxl_defbool_setdefault(&b_info->u.hvm.power_management, false); + libxl_defbool_setdefault(&b_info->u.hvm.pic, false); + libxl_defbool_setdefault(&b_info->u.hvm.pit, false); + } else { + libxl_defbool_setdefault(&b_info->u.hvm.hpet, true); + + if (!libxl_defbool_is_default(b_info->u.hvm.lapic) || + !libxl_defbool_is_default(b_info->u.hvm.ioapic) || + !libxl_defbool_is_default(b_info->u.hvm.rtc) || + !libxl_defbool_is_default(b_info->u.hvm.power_management) || + !libxl_defbool_is_default(b_info->u.hvm.pic) || + !libxl_defbool_is_default(b_info->u.hvm.pit)) { + LOG(ERROR, "the lapic, ioapic, rtc, power_management, pic and " + "pit options are only available to HVM guests " + "without a device model"); + return ERROR_INVAL; + } + } + libxl_defbool_setdefault(&b_info->u.hvm.spice.enable, false); if (!libxl_defbool_val(b_info->u.hvm.spice.enable) && (b_info->u.hvm.spice.usbredirection > 0) ){ diff --git a/tools/libxl/libxl_internal.c b/tools/libxl/libxl_internal.c index 328046b..791eae5 100644 --- a/tools/libxl/libxl_internal.c +++ b/tools/libxl/libxl_internal.c @@ -551,6 +551,31 @@ void libxl__update_domain_configuration(libxl__gc *gc, /* video ram */ dst->b_info.video_memkb = src->b_info.video_memkb; + + if (src->b_info.type == LIBXL_DOMAIN_TYPE_HVM) { +#define check_and_save_hvm_field(src, dst, field) \ + if (!libxl_defbool_is_default((src)->u.hvm.field)) \ + libxl_defbool_setdefault(&(dst)->u.hvm.field, \ + libxl_defbool_val((src)->u.hvm.field)) + + /* + * Save the status of emulated devices. + * + * For HVMlite guests all those fields are forced to a specific + * value, so they are always saved. For HVM guests only the HPET + * one is used and thus set. + */ + check_and_save_hvm_field(&src->b_info, &dst->b_info, hpet); + check_and_save_hvm_field(&src->b_info, &dst->b_info, lapic); + check_and_save_hvm_field(&src->b_info, &dst->b_info, ioapic); + check_and_save_hvm_field(&src->b_info, &dst->b_info, rtc); + check_and_save_hvm_field(&src->b_info, &dst->b_info, power_management); + check_and_save_hvm_field(&src->b_info, &dst->b_info, pic); + check_and_save_hvm_field(&src->b_info, &dst->b_info, pit); +#undef check_and_save_hvm_field + + dst->b_info.u.hvm.vga.kind = src->b_info.u.hvm.vga.kind; + } } char *libxl__device_model_xs_path(libxl__gc *gc, uint32_t dm_domid, diff --git a/tools/libxl/libxl_types.idl b/tools/libxl/libxl_types.idl index e9e0da0..fdec23c 100644 --- a/tools/libxl/libxl_types.idl +++ b/tools/libxl/libxl_types.idl @@ -519,6 +519,12 @@ libxl_domain_build_info = Struct("domain_build_info",[ ("serial_list", libxl_string_list), ("rdm", libxl_rdm_reserve), ("rdm_mem_boundary_memkb", MemKB), + ("lapic", libxl_defbool), + ("ioapic", libxl_defbool), + ("rtc", libxl_defbool), + ("power_management", libxl_defbool), + ("pic", libxl_defbool), + ("pit", libxl_defbool), ])), ("pv", Struct(None, [("kernel", string), ("slack_memkb", MemKB), diff --git a/tools/libxl/libxl_x86.c b/tools/libxl/libxl_x86.c index 46cfafb..92f25fd 100644 --- a/tools/libxl/libxl_x86.c +++ b/tools/libxl/libxl_x86.c @@ -7,15 +7,38 @@ int libxl__arch_domain_prepare_config(libxl__gc *gc, libxl_domain_config *d_config, xc_domain_configuration_t *xc_config) { + struct libxl_domain_build_info *info = &d_config->b_info; - if (d_config->c_info.type == LIBXL_DOMAIN_TYPE_HVM && - d_config->b_info.device_model_version != - LIBXL_DEVICE_MODEL_VERSION_NONE) { - /* HVM domains with a device model. */ + if (d_config->c_info.type == LIBXL_DOMAIN_TYPE_PV) { + /* PV guests. */ + xc_config->emulation_flags = XEN_X86_EMU_PIT; + } else if (info->device_model_version != LIBXL_DEVICE_MODEL_VERSION_NONE) { + /* HVM guests with a device model. */ xc_config->emulation_flags = XEN_X86_EMU_ALL; } else { - /* PV or HVM domains without a device model. */ + /* HVM guests without a device model. */ xc_config->emulation_flags = 0; + + if (libxl_defbool_val(info->u.hvm.lapic)) + xc_config->emulation_flags |= XEN_X86_EMU_LAPIC; + if (libxl_defbool_val(info->u.hvm.ioapic)) + xc_config->emulation_flags |= XEN_X86_EMU_IOAPIC; + if (libxl_defbool_val(info->u.hvm.rtc)) + xc_config->emulation_flags |= XEN_X86_EMU_RTC; + if (libxl_defbool_val(info->u.hvm.power_management)) + xc_config->emulation_flags |= XEN_X86_EMU_PM; + if (libxl_defbool_val(info->u.hvm.pic)) + xc_config->emulation_flags |= XEN_X86_EMU_PIC; + if (libxl_defbool_val(info->u.hvm.pit)) + xc_config->emulation_flags |= XEN_X86_EMU_PIT; + if (libxl_defbool_val(info->u.hvm.hpet)) + xc_config->emulation_flags |= XEN_X86_EMU_HPET; + + if (info->u.hvm.vga.kind != LIBXL_VGA_INTERFACE_TYPE_NONE) + xc_config->emulation_flags |= XEN_X86_EMU_VGA; + + if (d_config->num_pcidevs != 0) + xc_config->emulation_flags |= XEN_X86_EMU_IOMMU; } return 0; diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c index 25507c7..3654097 100644 --- a/tools/libxl/xl_cmdimpl.c +++ b/tools/libxl/xl_cmdimpl.c @@ -1514,6 +1514,13 @@ static void parse_config_data(const char *config_source, xlu_cfg_get_defbool(config, "acpi_s4", &b_info->u.hvm.acpi_s4, 0); xlu_cfg_get_defbool(config, "nx", &b_info->u.hvm.nx, 0); xlu_cfg_get_defbool(config, "hpet", &b_info->u.hvm.hpet, 0); + xlu_cfg_get_defbool(config, "lapic", &b_info->u.hvm.lapic, 0); + xlu_cfg_get_defbool(config, "ioapic", &b_info->u.hvm.ioapic, 0); + xlu_cfg_get_defbool(config, "rtc", &b_info->u.hvm.rtc, 0); + xlu_cfg_get_defbool(config, "power_management", + &b_info->u.hvm.power_management, 0); + xlu_cfg_get_defbool(config, "pic", &b_info->u.hvm.pic, 0); + xlu_cfg_get_defbool(config, "pit", &b_info->u.hvm.pit, 0); xlu_cfg_get_defbool(config, "vpt_align", &b_info->u.hvm.vpt_align, 0); switch (xlu_cfg_get_list(config, "viridian",