Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754725AbaBRJcF (ORCPT ); Tue, 18 Feb 2014 04:32:05 -0500 Received: from mga11.intel.com ([192.55.52.93]:50950 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754472AbaBRJcD convert rfc822-to-8bit (ORCPT ); Tue, 18 Feb 2014 04:32:03 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,500,1389772800"; d="scan'208";a="483315846" From: "Liu, Jinsong" To: Paolo Bonzini CC: "gleb@redhat.com" , "kvm@vger.kernel.org" , "qemu-devel@nongnu.org" , "linux-kernel@vger.kernel.org" , "hpa@zytor.com" , "Ren, Qiaowei" Subject: RE: [PATCH v3 0/4] X86/KVM: enable Intel MPX for KVM Thread-Topic: [PATCH v3 0/4] X86/KVM: enable Intel MPX for KVM Thread-Index: AQHPLIqDEXGTQoHbzECIeeZtPHIgl5q6vFiQ Date: Tue, 18 Feb 2014 09:32:00 +0000 Message-ID: References: <52DFB382.5080002@redhat.com> <53032581.7050406@redhat.com> In-Reply-To: <53032581.7050406@redhat.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Paolo Bonzini wrote: > Il 22/01/2014 13:03, Paolo Bonzini ha scritto: >> Il 22/01/2014 06:29, Liu, Jinsong ha scritto: >>> These patches are version 3 to enalbe Intel MPX for KVM. >>> >>> Version 1: >>> * Add some Intel MPX definiation >>> * Fix a cpuid(0x0d, 0) exposing bug, dynamic per XCR0 features >>> enable/disable >>> * vmx and msr handle for MPX support at KVM >>> * enalbe MPX feature for guest >>> >>> Version 2: >>> * remove generic MPX definiation, Qiaowei's patch has add the >>> definiation at kernel side >>> * add MSR_IA32_BNDCFGS to msrs_to_save >>> >>> Version 3: >>> * rebase on latest kernel, which include Qiaowei's MPX common >>> definiation pulled from HPA's tree >> >> I am afraid there is still some work to do on these patches, so they >> need to be delayed to 3.15. >> >> Patch 1: >> this seems mostly separate from the rest of the MPX work. I >> commented on the missing "ULL" suffix, but I would also like to >> understand why you put this patch in this series. >> >> Patch 2: >> As remarked in the reply to this patch: >> - the vmx_disable_intercept_for_msr has to be unconditional >> - you need a new kvm_x86_ops member mpx_supported, to disable >> MPX whenever the two VMX controls are not available. >> >> Patch 3: >> this patch needs to be rebased. Apart from that it is fine, >> but please move the VMX bits together with patch 2, and the >> other bits together with patch 4. >> >> Patch 4: >> this patch needs to be rebased and to use the new mpx_supported >> member >> >> If you also want to look at nested VMX support for MPX, that would be >> nice. It should not be hard. Otherwise we can take care of that >> later. >> >> Thanks for your work, >> >> Paolo > > Are you going to send v4? > > Paolo Yes, I just return from long Chinese Spring Festival, I will send V4 later. Thanks, Jinsong-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/