From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] intel_txt: enable VMXON check with SMX in KVM Date: Sun, 02 May 2010 16:49:30 +0300 Message-ID: <4BDD82EA.7060308@redhat.com> References: <1272557341-3777-1-git-send-email-shane.wang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: hpa@zytor.com, mingo@elte.hu, joseph.cihula@intel.com, linux-kernel@vger.kernel.org, kvm@vger.kernel.org To: Shane Wang Return-path: In-Reply-To: <1272557341-3777-1-git-send-email-shane.wang@intel.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 04/29/2010 07:09 PM, Shane Wang wrote: > Per document, for feature control MSR > Bit 1 enables VMXON in SMX operation. If the bit is clear, execution of VMXON in SMX operation causes a general-protection exception. > Bit 2 enables VMXON outside SMX operation. If the bit is clear, execution of VMXON outside SMX operation causes a general-protection exception. > > This patch is to enable this kind of check with SMX for VMXON in KVM. > > Looks good. > @@ -1176,9 +1177,16 @@ static __init int vmx_disabled_by_bios(v > u64 msr; > > rdmsrl(MSR_IA32_FEATURE_CONTROL, msr); > - return (msr& (FEATURE_CONTROL_LOCKED | > - FEATURE_CONTROL_VMXON_ENABLED)) > - == FEATURE_CONTROL_LOCKED; > + if (!!(msr& FEATURE_CONTROL_LOCKED)) { > !! not really necessary here (no need to resend, we can fix while applying). -- error compiling committee.c: too many arguments to function