@@ -1829,24 +1829,6 @@ struct xc_xend_cpuid {
char *policy[4];
};
-/*
- * Make adjustments to the CPUID settings for a domain.
- *
- * This path is used in two cases. First, for fresh boots of the domain, and
- * secondly for migrate-in/restore of pre-4.14 guests (where CPUID data was
- * missing from the stream). The @restore parameter distinguishes these
- * cases, and the generated policy must be compatible with a 4.13.
- *
- * Either pass a full new @featureset (and @nr_features), or adjust individual
- * features (@pae, @itsc, @nested_virt).
- *
- * Then (optionally) apply legacy XEND overrides (@xend) to the result.
- */
-int xc_cpuid_apply_policy(xc_interface *xch,
- uint32_t domid, bool restore,
- const uint32_t *featureset,
- unsigned int nr_features, bool pae, bool itsc,
- bool nested_virt, const struct xc_xend_cpuid *xend);
int xc_mca_op(xc_interface *xch, struct xen_mc *mc);
int xc_mca_op_inject_v2(xc_interface *xch, unsigned int flags,
xc_cpumap_t cpumap, unsigned int nr_cpus);
@@ -370,139 +370,6 @@ int xc_cpu_policy_apply_cpuid(xc_interface *xch, xc_cpu_policy_t *policy,
return rc;
}
-int xc_cpuid_apply_policy(xc_interface *xch, uint32_t domid, bool restore,
- const uint32_t *featureset, unsigned int nr_features,
- bool pae, bool itsc, bool nested_virt,
- const struct xc_xend_cpuid *cpuid)
-{
- int rc;
- xc_dominfo_t di;
- unsigned int nr_leaves, nr_msrs;
- xen_cpuid_leaf_t *leaves = NULL;
- struct cpuid_policy *p = NULL;
- xc_cpu_policy_t *policy = NULL;
- xc_cpu_policy_t *host = NULL;
- uint32_t err_leaf = -1, err_subleaf = -1, err_msr = -1;
-
- if ( xc_domain_getinfo(xch, domid, 1, &di) != 1 ||
- di.domid != domid )
- {
- ERROR("Failed to obtain d%d info", domid);
- rc = -ESRCH;
- goto out;
- }
-
- rc = xc_cpu_policy_get_size(xch, &nr_leaves, &nr_msrs);
- if ( rc )
- {
- PERROR("Failed to obtain policy info size");
- rc = -errno;
- goto out;
- }
-
- rc = -ENOMEM;
- if ( (leaves = calloc(nr_leaves, sizeof(*leaves))) == NULL ||
- (p = calloc(1, sizeof(*p))) == NULL ||
- (policy = xc_cpu_policy_init()) == NULL ||
- (host = xc_cpu_policy_init()) == NULL )
- goto out;
-
- rc = xc_cpu_policy_get_system(xch, XEN_SYSCTL_cpu_policy_host, host);
- if ( rc )
- {
- PERROR("Failed to get host policy");
- rc = -errno;
- goto out;
- }
-
- /* Get the domain's default policy. */
- nr_msrs = 0;
- rc = get_system_cpu_policy(xch, di.hvm ? XEN_SYSCTL_cpu_policy_hvm_default
- : XEN_SYSCTL_cpu_policy_pv_default,
- &nr_leaves, leaves, &nr_msrs, NULL);
- if ( rc )
- {
- PERROR("Failed to obtain %s default policy", di.hvm ? "hvm" : "pv");
- rc = -errno;
- goto out;
- }
-
- rc = x86_cpuid_copy_from_buffer(p, leaves, nr_leaves,
- &err_leaf, &err_subleaf);
- if ( rc )
- {
- ERROR("Failed to deserialise CPUID (err leaf %#x, subleaf %#x) (%d = %s)",
- err_leaf, err_subleaf, -rc, strerror(-rc));
- goto out;
- }
-
- if ( restore )
- {
- policy->cpuid = *p;
- xc_cpu_policy_make_compat_4_12(xch, policy, host, di.hvm);
- *p = policy->cpuid;
- }
-
- if ( featureset )
- {
- policy.cpuid = *p;
- rc = xc_cpu_policy_apply_featureset(xch, &policy, featureset,
- nr_features);
- if ( rc )
- {
- ERROR("Failed to apply featureset to policy");
- goto out;
- }
- *p = policy.cpuid;
- }
- else
- {
- p->extd.itsc = itsc;
-
- if ( di.hvm )
- {
- p->basic.pae = pae;
- p->basic.vmx = nested_virt;
- p->extd.svm = nested_virt;
- }
- }
-
- policy->cpuid = *p;
- xc_cpu_policy_legacy_topology(xch, policy, di.hvm ? NULL : host);
- *p = policy->cpuid;
-
- rc = xc_cpu_policy_apply_cpuid(xch, policy, host, cpuid, di.hvm);
- if ( rc )
- goto out;
-
- rc = x86_cpuid_copy_to_buffer(p, leaves, &nr_leaves);
- if ( rc )
- {
- ERROR("Failed to serialise CPUID (%d = %s)", -rc, strerror(-rc));
- goto out;
- }
-
- rc = xc_set_domain_cpu_policy(xch, domid, nr_leaves, leaves, 0, NULL,
- &err_leaf, &err_subleaf, &err_msr);
- if ( rc )
- {
- PERROR("Failed to set d%d's policy (err leaf %#x, subleaf %#x, msr %#x)",
- domid, err_leaf, err_subleaf, err_msr);
- rc = -errno;
- goto out;
- }
-
- rc = 0;
-
-out:
- free(p);
- free(leaves);
- xc_cpu_policy_destroy(policy);
- xc_cpu_policy_destroy(host);
-
- return rc;
-}
-
xc_cpu_policy_t *xc_cpu_policy_init(void)
{
return calloc(1, sizeof(struct xc_cpu_policy));
@@ -441,9 +441,13 @@ int libxl__cpuid_legacy(libxl_ctx *ctx, uint32_t domid, bool restore,
libxl_domain_build_info *info)
{
GC_INIT(ctx);
+ xc_cpu_policy_t *policy = NULL;
+ xc_cpu_policy_t *host = NULL;
+ char *extra_feat;
+ bool hvm = info->type != LIBXL_DOMAIN_TYPE_PV;
bool pae = true;
bool itsc;
- int r;
+ int r, rc = 0;
/*
* Gross hack. Using libxl_defbool_val() here causes libvirt to crash in
@@ -454,6 +458,38 @@ int libxl__cpuid_legacy(libxl_ctx *ctx, uint32_t domid, bool restore,
*/
bool nested_virt = info->nested_hvm.val > 0;
+ policy = xc_cpu_policy_init();
+ host = xc_cpu_policy_init();
+ if (!policy || !host) {
+ LOGED(ERROR, domid, "Failed to init CPU policies");
+ rc = ERROR_FAIL;
+ goto out;
+ }
+
+ r = xc_cpu_policy_get_system(ctx->xch, XEN_SYSCTL_cpu_policy_host, host);
+ if (r) {
+ LOGED(ERROR, domid, "Failed to fetch host policy");
+ rc = ERROR_FAIL;
+ goto out;
+ }
+
+ r = xc_cpu_policy_get_domain(ctx->xch, domid, policy);
+ if (r) {
+ LOGED(ERROR, domid, "Failed to fetch domain CPU policy");
+ rc = ERROR_FAIL;
+ goto out;
+ }
+
+ if (restore)
+ /*
+ * Make sure the policy is compatible with pre Xen 4.13. Note that
+ * newer Xen versions will pass policy data on the restore stream, so
+ * any adjustments done here will be superseded.
+ */
+ xc_cpu_policy_make_compat_4_12(ctx->xch, policy, host, hvm);
+
+ xc_cpu_policy_legacy_topology(ctx->xch, policy, hvm ? NULL : host);
+
/*
* For PV guests, PAE is Xen-controlled (it is the 'p' that differentiates
* the xen-3.0-x86_32 and xen-3.0-x86_32p ABIs). It is mandatory as Xen
@@ -481,14 +517,39 @@ int libxl__cpuid_legacy(libxl_ctx *ctx, uint32_t domid, bool restore,
*/
itsc = (libxl_defbool_val(info->disable_migrate) ||
info->tsc_mode == LIBXL_TSC_MODE_ALWAYS_EMULATE);
+ extra_feat = GCSPRINTF("pae=%d,invtsc=%d", pae, itsc);
+
+ /* Set Nested virt CPUID bits for HVM. */
+ if (hvm)
+ extra_feat = GCSPRINTF("%s,vmx=%d,svm=%d",
+ extra_feat, nested_virt, nested_virt);
+
+ rc = libxl_cpuid_parse_config(&info->cpuid, extra_feat);
+ if (rc) {
+ LOGD(ERROR, domid, "Failed to set CPU features: %s", extra_feat);
+ rc = ERROR_FAIL;
+ goto out;
+ }
- r = xc_cpuid_apply_policy(ctx->xch, domid, restore, NULL, 0,
- pae, itsc, nested_virt, info->cpuid);
- if (r)
- LOGEVD(ERROR, -r, domid, "Failed to apply CPUID policy");
+ /* Apply the bits from info->cpuid. */
+ r = xc_cpu_policy_apply_cpuid(ctx->xch, policy, host, info->cpuid, hvm);
+ if (r) {
+ LOGEVD(ERROR, domid, -r, "Failed to apply CPUID changes");
+ rc = ERROR_FAIL;
+ goto out;
+ }
+
+ r = xc_cpu_policy_set_domain(ctx->xch, domid, policy);
+ if (r) {
+ LOGED(ERROR, domid, "Failed to set domain CPUID policy");
+ rc = ERROR_FAIL;
+ }
+ out:
+ xc_cpu_policy_destroy(policy);
+ xc_cpu_policy_destroy(host);
GC_FREE;
- return r ? ERROR_FAIL : 0;
+ return rc;
}
static const char *input_names[2] = { "leaf", "subleaf" };