diff options
author | Andrew Cooper <andrew.cooper3@citrix.com> | 2019-09-13 14:45:40 +0100 |
---|---|---|
committer | Andrew Cooper <andrew.cooper3@citrix.com> | 2019-09-26 13:40:18 +0100 |
commit | ff78e500844429ca1b0c2f8220e4fc263499b1ed (patch) | |
tree | abe007f35d4749d047caae2d21f5baa5fa3a0130 /xen/arch/x86/msr.c | |
parent | 433387661ec789962dffd8ec929f59fdee78194c (diff) |
x86/msr: Offer CPUID Faulting to PVH control domains
The control domain exclusion for CPUID Faulting predates dom0 PVH, but the
reason for the exclusion (to allow the domain builder to see host CPUID
values) isn't applicable.
The domain builder *is* broken in PVH control domains, and restricting the use
of CPUID Faulting doesn't make it any less broken. Tweak the logic to only
exclude PV control domains.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
Release-acked-by: Juergen Gross <jgross@suse.com>
Diffstat (limited to 'xen/arch/x86/msr.c')
-rw-r--r-- | xen/arch/x86/msr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xen/arch/x86/msr.c b/xen/arch/x86/msr.c index e65961fccb..a6c8cc7627 100644 --- a/xen/arch/x86/msr.c +++ b/xen/arch/x86/msr.c @@ -91,8 +91,8 @@ int init_domain_msr_policy(struct domain *d) if ( !mp ) return -ENOMEM; - /* See comment in intel_ctxt_switch_levelling() */ - if ( is_control_domain(d) ) + /* See comment in ctxt_switch_levelling() */ + if ( is_control_domain(d) && is_pv_domain(d) ) mp->platform_info.cpuid_faulting = false; d->arch.msr = mp; |