From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [PATCH 1/2] KVM: x86: disable MPX if host did not enable MPX XSAVE features Date: Thu, 24 Mar 2016 14:07:55 +0100 Message-ID: <56F3E6AB.5070902@redhat.com> References: <1457437481-65784-1-git-send-email-pbonzini@redhat.com> <1457437481-65784-2-git-send-email-pbonzini@redhat.com> <56E2B58B.8020605@redhat.com> <56F3E653.6040901@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: linux-kernel@vger.kernel.org, kvm@vger.kernel.org, Xiao Guangrong To: Yang Zhang Return-path: Received: from mx1.redhat.com ([209.132.183.28]:57008 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750972AbcCXNIA (ORCPT ); Thu, 24 Mar 2016 09:08:00 -0400 In-Reply-To: <56F3E653.6040901@gmail.com> Sender: kvm-owner@vger.kernel.org List-ID: On 24/03/2016 14:06, Yang Zhang wrote: > > I mean why not keep the old way that only activate the eager_fpu while > guest sees the MPX bit in CPUID, like: > > vcpu->arch.eager_fpu = use_eager_fpu() && guest_cpuid_has_mpx(vcpu); If the host uses eager FPU you can assume that it's faster than lazy FPU. Paolo > Besides, vmx_fpu_activate is called when do vcpu_reset. So it seems no > need to call fpu_activate() here again.