From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vitaly Kuznetsov Subject: Re: [RFC 0/6] Enlightened VMCS support for KVM on Hyper-V Date: Tue, 16 Jan 2018 15:43:56 +0100 Message-ID: <87po69hnir.fsf@vitty.brq.redhat.com> References: <20180115173105.31845-1-vkuznets@redhat.com> <87inc2huux.fsf@vitty.brq.redhat.com> <87tvvlhplx.fsf@vitty.brq.redhat.com> Mime-Version: 1.0 Content-Type: text/plain Cc: David Hildenbrand , Wanpeng Li , kvm , the arch/x86 maintainers , Radim =?utf-8?B?S3LEjW3DocWZ?= , "K. Y. Srinivasan" , Haiyang Zhang , Stephen Hemminger , "Michael Kelley \(EOSG\)" , Mohammed Gamal , Cathy Avery , Bandan Das , linux-kernel@vger.kernel.org To: Paolo Bonzini Return-path: In-Reply-To: (Paolo Bonzini's message of "Tue, 16 Jan 2018 15:27:33 +0100") Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org Paolo Bonzini writes: > On 16/01/2018 14:58, Vitaly Kuznetsov wrote: >>> Haven't looked into the details, but we have to watch out for other >>> VCPUs trying to modify that vmcs12. >>> >>> Basically because other VCPUs could try to modify values in vmcs12 while >>> we are currently building vmcs02. Nasty races could result in us copying >>> stuff (probably unchecked) into vmcs02 and therefore running something >>> that was not intended. >>> >> I don't think we share VMCS among vCPUs, do we? > > VMCS is just memory, so who knows what a malicious L1 guest will do. > But for vmread/vmwrite we can go through hypervisor memory, for > enlightened VMCS we cannot. > True; not sure if Hyper-V actually copies the data to some internal storage, probably it does. TLFS explicitly forbids making the same enlightened VMCS active on several vCPUs simultaneously but again, this is just memory... -- Vitaly