From: Sean Christopherson <firstname.lastname@example.org> To: Paolo Bonzini <email@example.com> Cc: Vitaly Kuznetsov <firstname.lastname@example.org>, email@example.com, Jim Mattson <firstname.lastname@example.org>, email@example.com, Liran Alon <firstname.lastname@example.org>, Roman Kagan <email@example.com> Subject: Re: [PATCH RFC 2/3] x86/kvm/hyper-v: move VMX controls sanitization out of nested_enable_evmcs() Date: Tue, 21 Jan 2020 21:47:24 -0800 Message-ID: <20200122054724.GD18513@linux.intel.com> (raw) In-Reply-To: <firstname.lastname@example.org> On Sun, Jan 19, 2020 at 09:54:44AM +0100, Paolo Bonzini wrote: > On 15/01/20 18:10, Vitaly Kuznetsov wrote: > > With fine grained VMX feature enablement QEMU>=4.2 tries to do KVM_SET_MSRS > > with default (matching CPU model) values and in case eVMCS is also enabled, > > fails. > > > > It would be possible to drop VMX feature filtering completely and make > > this a guest's responsibility: if it decides to use eVMCS it should know > > which fields are available and which are not. Hyper-V mostly complies to > > this, however, there is at least one problematic control: > > SECONDARY_EXEC_VIRTUALIZE_APIC_ACCESSES > > which Hyper-V enables. As there is no 'apic_addr_field' in eVMCS, we > > fail to handle this properly in KVM. It is unclear how this is supposed > > to work, genuine Hyper-V doesn't expose the control so it is possible that > > this is just a bug (in Hyper-V). > > Yes, it most likely is and it would be nice if Microsoft fixed it, but I > guess we're stuck with it for existing Windows versions. Well, for one > we found a bug in Hyper-V and not the converse. :) > > There is a problem with this approach, in that we're stuck with it > forever due to live migration. But I guess if in the future eVMCS v2 > adds an apic_address field we can limit the hack to eVMCS v1. Another > possibility is to use the quirks mechanism but it's overkill for now. > > Unless there are objections, I plan to apply these patches. Doesn't applying this patch contradict your earlier opinion? This patch would still hide the affected controls from the guest because the host controls enlightened_vmcs_enabled. On Sat, Jan 18, 2020 at 10:42:31PM +0100, Paolo Bonzini wrote: > IMHO the features should stay available in case the guest chooses not to > use eVMCS. A guest that uses eVMCS should know which features it cannot > use and not enable them. Makes sense, wasn't thinking about the scenario where the guest doesn't support eVMCS or doesn't want to use it for whatever reason. Rather than update vmx->nested.msrs or filter vmx_get_msr(), what about manually adding eVMCS consistency checks on the disallowed bits and handle SECONDARY_EXEC_VIRTUALIZE_APIC_ACCESSES as a one-off case by simply clearing it from the eVMCS? Or alternatively, squashing all the disallowed bits.
next prev parent reply index Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-01-15 17:10 [PATCH RFC 0/3] x86/kvm/hyper-v: fix enlightened VMCS & QEMU4.2 Vitaly Kuznetsov 2020-01-15 17:10 ` [PATCH RFC 1/3] x86/kvm/hyper-v: remove stale evmcs_already_enabled check from nested_enable_evmcs() Vitaly Kuznetsov 2020-01-15 22:50 ` Liran Alon 2020-01-15 17:10 ` [PATCH RFC 2/3] x86/kvm/hyper-v: move VMX controls sanitization out of nested_enable_evmcs() Vitaly Kuznetsov 2020-01-15 22:49 ` Liran Alon 2020-01-16 8:37 ` Vitaly Kuznetsov 2020-02-03 15:11 ` Vitaly Kuznetsov 2020-01-15 23:27 ` Sean Christopherson 2020-01-15 23:30 ` Liran Alon 2020-01-16 8:51 ` Vitaly Kuznetsov 2020-01-16 16:19 ` Sean Christopherson 2020-01-16 16:57 ` Vitaly Kuznetsov 2020-01-17 6:31 ` Sean Christopherson 2020-01-18 21:42 ` Paolo Bonzini 2020-01-19 8:54 ` Paolo Bonzini 2020-01-22 5:47 ` Sean Christopherson [this message] 2020-01-22 9:37 ` Vitaly Kuznetsov 2020-01-22 14:33 ` Paolo Bonzini 2020-01-22 15:08 ` Vitaly Kuznetsov 2020-01-22 15:51 ` Sean Christopherson 2020-01-22 16:29 ` Vitaly Kuznetsov 2020-01-22 16:40 ` Paolo Bonzini 2020-01-23 9:15 ` Vitaly Kuznetsov 2020-01-23 19:09 ` Vitaly Kuznetsov 2020-01-24 17:25 ` Sean Christopherson 2020-01-27 15:38 ` Vitaly Kuznetsov 2020-01-27 17:53 ` Paolo Bonzini 2020-01-27 21:52 ` Vitaly Kuznetsov 2020-01-27 18:17 ` Sean Christopherson 2020-01-15 17:10 ` [PATCH RFC 3/3] x86/kvm/hyper-v: don't allow to turn on unsupported VMX controls for nested guests Vitaly Kuznetsov 2020-01-15 22:59 ` Liran Alon 2020-01-16 8:55 ` Vitaly Kuznetsov 2020-01-16 16:21 ` Sean Christopherson 2020-01-19 8:57 ` Paolo Bonzini
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=20200122054724.GD18513@linux.intel.com \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.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
KVM Archive on lore.kernel.org Archives are clonable: git clone --mirror https://lore.kernel.org/kvm/0 kvm/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 kvm kvm/ https://lore.kernel.org/kvm \ email@example.com public-inbox-index kvm Example config snippet for mirrors Newsgroup available over NNTP: nntp://nntp.lore.kernel.org/org.kernel.vger.kvm AGPL code for this site: git clone https://public-inbox.org/public-inbox.git