[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE s
From: |
Haozhong Zhang |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support |
Date: |
Mon, 13 Jun 2016 15:55:50 +0800 |
User-agent: |
Mutt/1.6.1-neo (2016-05-02) |
On 06/08/16 13:32, Paolo Bonzini wrote:
>
>
> On 03/06/2016 17:57, Radim Krčmář wrote:
> >> > + cenv->msr_ia32_feature_control |=
> >> > + MSR_IA32_FEATURE_CONTROL_LMCE |
> >> > + MSR_IA32_FEATURE_CONTROL_LOCKED;
> > Locking right from the start breaks nested KVM, because nested relies on
> > setting VMXON feature from inside of the guest.
> >
> > Do we keep it unlocked, or move everything into QEMU?
> >
> > (The latter seems simpler.)
>
> I think it should be moved into the firmware, with QEMU publishing the
> desired setting via fw_cfg. The same as what is done in real hardware,
> that's the KVM mantra. :)
>
> For v4 it's okay to just remove this.
>
> Paolo
Currently, only VMX bits (bit 1 & 2), LMCE bit (bit 20) as well as
lock bit (bit 0) in MSR_IA32_FEATURE_CONTROL are used for guest. The
availability of features indicated by those bits (except the lock bit)
can be discovered from cpuid and other MSR, so it looks not necessary
to publish them via fw_cfg. Or do you have other concerns?
Thanks,
Haozhong
- [Qemu-devel] [PATCH v3 0/2] Add QEMU support for Intel local MCE, Haozhong Zhang, 2016/06/03
- [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Haozhong Zhang, 2016/06/03
- Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Radim Krčmář, 2016/06/03
- Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Haozhong Zhang, 2016/06/05
- Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Paolo Bonzini, 2016/06/08
- Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support,
Haozhong Zhang <=
- Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Paolo Bonzini, 2016/06/13
- Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Haozhong Zhang, 2016/06/13
- Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Paolo Bonzini, 2016/06/13
- Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Haozhong Zhang, 2016/06/13
Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Boris Petkov, 2016/06/04
Re: [Qemu-devel] [PATCH v3 1/2] target-i386: KVM: add basic Intel LMCE support, Boris Petkov, 2016/06/04