From: Roger Pau Monne <roger.pau@citrix.com>
To: xen-devel@lists.xenproject.org
Cc: Wei Liu <wei.liu2@citrix.com>,
Ian Jackson <ian.jackson@eu.citrix.com>,
Ian Campbell <ian.campbell@citrix.com>,
Roger Pau Monne <roger.pau@citrix.com>
Subject: [PATCH v4 5/6] libxl: add options to enable/disable emulated devices
Date: Thu, 21 Jan 2016 17:51:31 +0100 [thread overview]
Message-ID: <1453395092-88090-6-git-send-email-roger.pau@citrix.com> (raw)
In-Reply-To: <1453395092-88090-1-git-send-email-roger.pau@citrix.com>
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é <roger.pau@citrix.com>
---
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: Wei Liu <wei.liu2@citrix.com>
---
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<docs/misc/pci-device-reservations.txt> 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<lapic=BOOLEAN>
+
+Enables or disables the Local APIC.
+
+=item B<ioapic=BOOLEAN>
+
+Enables or disables the IO APIC.
+
+=item B<rtc=BOOLEAN>
+
+Enables or disables the RTC.
+
+=item B<power_management=BOOLEAN>
+
+Enables or disables the ACPI power management timer and control interfaces.
+
+=item B<pic=BOOLEAN>
+
+Enables or disables the PIC.
+
+=item B<pit=BOOLEAN>
+
+Enables or disables the PIT.
+
+=item B<hpet=BOOLEAN>
+
+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",
--
1.9.5 (Apple Git-50.3)
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2016-01-21 16:52 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-21 16:51 [PATCH v4 0/6] HVMlite: DomU fixes and a Dom0 preparatory patch Roger Pau Monne
2016-01-21 16:51 ` [PATCH v4 1/6] libelf: rewrite symtab/strtab loading for Dom0 Roger Pau Monne
2016-01-21 17:29 ` Ian Jackson
2016-01-21 17:55 ` Roger Pau Monné
2016-01-21 18:44 ` Ian Jackson
2016-01-22 8:11 ` Jan Beulich
2016-01-22 9:58 ` Roger Pau Monné
2016-01-21 16:51 ` [PATCH v4 2/6] libxl: introduce LIBXL_VGA_INTERFACE_TYPE_UNKNOWN Roger Pau Monne
2016-01-22 10:59 ` Ian Campbell
2016-01-21 16:51 ` [PATCH v4 3/6] libxl: initialise the build info before calling prepare_config Roger Pau Monne
2016-01-22 11:00 ` Ian Campbell
2016-01-21 16:51 ` [PATCH v4 4/6] x86/PV: allow PV guests to have an emulated PIT Roger Pau Monne
2016-01-22 10:48 ` Jan Beulich
2016-01-22 11:03 ` Ian Campbell
2016-01-21 16:51 ` Roger Pau Monne [this message]
2016-01-22 17:04 ` [PATCH v4 5/6] libxl: add options to enable/disable emulated devices Roger Pau Monné
2016-01-25 9:33 ` Ian Campbell
2016-01-21 16:51 ` [PATCH v4 6/6] x86/HVM: report the set of enabled emulated devices through CPUID Roger Pau Monne
2016-01-22 10:57 ` Jan Beulich
2016-01-22 12:43 ` Roger Pau Monné
2016-01-22 13:24 ` Jan Beulich
2016-01-22 14:41 ` Roger Pau Monné
2016-01-22 15:02 ` Jan Beulich
2016-01-22 15:41 ` Roger Pau Monné
2016-01-22 13:34 ` Andrew Cooper
2016-01-22 14:59 ` Roger Pau Monné
2016-01-22 15:31 ` Jan Beulich
2016-01-22 15:51 ` Roger Pau Monné
2016-01-25 11:23 ` Jan Beulich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1453395092-88090-6-git-send-email-roger.pau@citrix.com \
--to=roger.pau@citrix.com \
--cc=ian.campbell@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.