* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM [not found] <1386394315-48984-1-git-send-email-jinsong.liu@intel.com> @ 2013-12-11 8:31 ` Liu, Jinsong 2013-12-11 9:53 ` Paolo Bonzini 0 siblings, 1 reply; 9+ messages in thread From: Liu, Jinsong @ 2013-12-11 8:31 UTC (permalink / raw) To: pbonzini@redhat.com, gleb@redhat.com, hpa@zytor.com, kvm@vger.kernel.org, qemu-devel@nongnu.org, linux-kernel@vger.kernel.org Paolo, comments for version 2? Thanks, Jinsong Liu, Jinsong wrote: > These patches are version 2 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, kernel side has add the > definiation > * add MSR_IA32_BNDCFGS to msrs_to_save > > Thanks, > Jinsong > > Liu Jinsong (4): > KVM/X86: Fix xsave cpuid exposing bug > KVM/X86: Intel MPX vmx and msr handle > KVM/X86: add MSR_IA32_BNDCFGS to msrs_to_save > KVM/X86: Enable Intel MPX for guest. > > arch/x86/include/asm/vmx.h | 4 ++++ > arch/x86/include/asm/xsave.h | 2 ++ > arch/x86/include/uapi/asm/msr-index.h | 1 + > arch/x86/kvm/cpuid.c | 8 ++++---- > arch/x86/kvm/vmx.c | 18 ++++++++++++++++-- > arch/x86/kvm/x86.c | 12 +++++++++--- > arch/x86/kvm/x86.h | 3 ++- > 7 files changed, 38 insertions(+), 10 deletions(-) ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM 2013-12-11 8:31 ` [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM Liu, Jinsong @ 2013-12-11 9:53 ` Paolo Bonzini 2013-12-12 5:47 ` Liu, Jinsong 0 siblings, 1 reply; 9+ messages in thread From: Paolo Bonzini @ 2013-12-11 9:53 UTC (permalink / raw) To: Liu, Jinsong Cc: qemu-devel@nongnu.org, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, gleb@redhat.com, hpa@zytor.com Il 11/12/2013 09:31, Liu, Jinsong ha scritto: > Paolo, comments for version 2? I think I commented that it's fine, I'm just waiting for a rebase on top of the generic patches. Paolo > Thanks, > Jinsong > > Liu, Jinsong wrote: >> These patches are version 2 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, kernel side has add the >> definiation >> * add MSR_IA32_BNDCFGS to msrs_to_save >> >> Thanks, >> Jinsong >> >> Liu Jinsong (4): >> KVM/X86: Fix xsave cpuid exposing bug >> KVM/X86: Intel MPX vmx and msr handle >> KVM/X86: add MSR_IA32_BNDCFGS to msrs_to_save >> KVM/X86: Enable Intel MPX for guest. >> >> arch/x86/include/asm/vmx.h | 4 ++++ >> arch/x86/include/asm/xsave.h | 2 ++ >> arch/x86/include/uapi/asm/msr-index.h | 1 + >> arch/x86/kvm/cpuid.c | 8 ++++---- >> arch/x86/kvm/vmx.c | 18 ++++++++++++++++-- >> arch/x86/kvm/x86.c | 12 +++++++++--- >> arch/x86/kvm/x86.h | 3 ++- >> 7 files changed, 38 insertions(+), 10 deletions(-) > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM 2013-12-11 9:53 ` Paolo Bonzini @ 2013-12-12 5:47 ` Liu, Jinsong 2013-12-12 9:43 ` Paolo Bonzini 0 siblings, 1 reply; 9+ messages in thread From: Liu, Jinsong @ 2013-12-12 5:47 UTC (permalink / raw) 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 Paolo Bonzini wrote: > Il 11/12/2013 09:31, Liu, Jinsong ha scritto: >> Paolo, comments for version 2? > > I think I commented that it's fine, I'm just waiting for a rebase on > top of the generic patches. > > Paolo > Thanks! common MPX definiation patches have been checked in tip tree (both Qiaowei and I use that definiations): http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=191f57c137bcce0e3e9313acb77b2f114d15afbb http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=e7d820a5e549b3eb6c3f9467507566565646a669 Jinsong >> >> Liu, Jinsong wrote: >>> These patches are version 2 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, kernel side has add the >>> definiation >>> * add MSR_IA32_BNDCFGS to msrs_to_save >>> >>> Thanks, >>> Jinsong >>> >>> Liu Jinsong (4): >>> KVM/X86: Fix xsave cpuid exposing bug >>> KVM/X86: Intel MPX vmx and msr handle >>> KVM/X86: add MSR_IA32_BNDCFGS to msrs_to_save >>> KVM/X86: Enable Intel MPX for guest. >>> >>> arch/x86/include/asm/vmx.h | 4 ++++ >>> arch/x86/include/asm/xsave.h | 2 ++ >>> arch/x86/include/uapi/asm/msr-index.h | 1 + >>> arch/x86/kvm/cpuid.c | 8 ++++---- >>> arch/x86/kvm/vmx.c | 18 ++++++++++++++++-- >>> arch/x86/kvm/x86.c | 12 +++++++++--- >>> arch/x86/kvm/x86.h | 3 ++- >>> 7 files changed, 38 insertions(+), 10 deletions(-) ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM 2013-12-12 5:47 ` Liu, Jinsong @ 2013-12-12 9:43 ` Paolo Bonzini 2013-12-12 11:09 ` Liu, Jinsong 0 siblings, 1 reply; 9+ messages in thread From: Paolo Bonzini @ 2013-12-12 9:43 UTC (permalink / raw) To: Liu, Jinsong Cc: gleb@redhat.com, kvm@vger.kernel.org, qemu-devel@nongnu.org, linux-kernel@vger.kernel.org, hpa@zytor.com, Ren, Qiaowei Il 12/12/2013 06:47, Liu, Jinsong ha scritto: > Paolo Bonzini wrote: >> Il 11/12/2013 09:31, Liu, Jinsong ha scritto: >>> Paolo, comments for version 2? >> >> I think I commented that it's fine, I'm just waiting for a rebase on >> top of the generic patches. >> >> Paolo >> > > Thanks! common MPX definiation patches have been checked in tip tree (both Qiaowei and I use that definiations): > http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=191f57c137bcce0e3e9313acb77b2f114d15afbb > http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=e7d820a5e549b3eb6c3f9467507566565646a669 Ok, can you rebase and resend? Paolo ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM 2013-12-12 9:43 ` Paolo Bonzini @ 2013-12-12 11:09 ` Liu, Jinsong 2014-01-20 14:35 ` Paolo Bonzini 0 siblings, 1 reply; 9+ messages in thread From: Liu, Jinsong @ 2013-12-12 11:09 UTC (permalink / raw) 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 Paolo Bonzini wrote: > Il 12/12/2013 06:47, Liu, Jinsong ha scritto: >> Paolo Bonzini wrote: >>> Il 11/12/2013 09:31, Liu, Jinsong ha scritto: >>>> Paolo, comments for version 2? >>> >>> I think I commented that it's fine, I'm just waiting for a rebase >>> on top of the generic patches. >>> >>> Paolo >>> >> >> Thanks! common MPX definiation patches have been checked in tip tree >> (both Qiaowei and I use that definiations): >> http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=191f57c137bcce0e3e9313acb77b2f114d15afbb >> http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=e7d820a5e549b3eb6c3f9467507566565646a669 > > Ok, can you rebase and resend? > > Paolo Sure, I have pulled and rebased on git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git No confliction, patches v3 will send out later. Thanks, Jinsong ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM 2013-12-12 11:09 ` Liu, Jinsong @ 2014-01-20 14:35 ` Paolo Bonzini 2014-01-21 15:25 ` Liu, Jinsong 0 siblings, 1 reply; 9+ messages in thread From: Paolo Bonzini @ 2014-01-20 14:35 UTC (permalink / raw) To: Liu, Jinsong Cc: gleb@redhat.com, kvm@vger.kernel.org, qemu-devel@nongnu.org, linux-kernel@vger.kernel.org, hpa@zytor.com, Ren, Qiaowei Il 12/12/2013 12:09, Liu, Jinsong ha scritto: > Paolo Bonzini wrote: >> Il 12/12/2013 06:47, Liu, Jinsong ha scritto: >>> Paolo Bonzini wrote: >>>> Il 11/12/2013 09:31, Liu, Jinsong ha scritto: >>>>> Paolo, comments for version 2? >>>> >>>> I think I commented that it's fine, I'm just waiting for a rebase >>>> on top of the generic patches. >>>> >>>> Paolo >>>> >>> >>> Thanks! common MPX definiation patches have been checked in tip tree >>> (both Qiaowei and I use that definiations): >>> http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=191f57c137bcce0e3e9313acb77b2f114d15afbb >>> http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=e7d820a5e549b3eb6c3f9467507566565646a669 >> >> Ok, can you rebase and resend? >> >> Paolo > > Sure, I have pulled and rebased on git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git > No confliction, patches v3 will send out later. Didn't see this... You still have a couple of days to send it out and address the review remarks. Paolo ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM 2014-01-20 14:35 ` Paolo Bonzini @ 2014-01-21 15:25 ` Liu, Jinsong 2014-01-21 16:33 ` Paolo Bonzini 0 siblings, 1 reply; 9+ messages in thread From: Liu, Jinsong @ 2014-01-21 15:25 UTC (permalink / raw) 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 Paolo Bonzini wrote: > Il 12/12/2013 12:09, Liu, Jinsong ha scritto: >> Paolo Bonzini wrote: >>> Il 12/12/2013 06:47, Liu, Jinsong ha scritto: >>>> Paolo Bonzini wrote: >>>>> Il 11/12/2013 09:31, Liu, Jinsong ha scritto: >>>>>> Paolo, comments for version 2? >>>>> >>>>> I think I commented that it's fine, I'm just waiting for a rebase >>>>> on top of the generic patches. >>>>> >>>>> Paolo >>>>> >>>> >>>> Thanks! common MPX definiation patches have been checked in tip >>>> tree (both Qiaowei and I use that definiations): >>>> http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=191f57c137bcce0e3e9313acb77b2f114d15afbb >>>> http://git.kernel.org/cgit/linux/kernel/git/tip/tip.git/commit/?id=e7d820a5e549b3eb6c3f9467507566565646a669 >>> >>> Ok, can you rebase and resend? >>> >>> Paolo >> >> Sure, I have pulled and rebased on >> git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git No >> confliction, patches v3 will send out later. > > Didn't see this... You still have a couple of days to send it out and > address the review remarks. > > Paolo Hmm? I remember I have sent out the rebased patches v3 last month ... If you didn't receive them I'm OK to rebase and resend them. BTW, what's the review remarks? I remember you commented that the patches are fine. Any misunderstanding please point out to me. Thanks, Jinsong ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM 2014-01-21 15:25 ` Liu, Jinsong @ 2014-01-21 16:33 ` Paolo Bonzini 2014-01-21 17:26 ` Liu, Jinsong 0 siblings, 1 reply; 9+ messages in thread From: Paolo Bonzini @ 2014-01-21 16:33 UTC (permalink / raw) To: Liu, Jinsong Cc: gleb@redhat.com, kvm@vger.kernel.org, qemu-devel@nongnu.org, linux-kernel@vger.kernel.org, hpa@zytor.com, Ren, Qiaowei Il 21/01/2014 16:25, Liu, Jinsong ha scritto: > Hmm? I remember I have sent out the rebased patches v3 last month > If you didn't receive them I'm OK to rebase and resend them. BTW, what's > the review remarks? I remember you commented that the patches are fine. > Any misunderstanding please point out to me. You sent v3 of QEMU, but not of KVM. I don't see any mail from you after December 12th on kvm@vger.kernel.org. I can see my comment that the patches were fine (apart from needing a rebase), but the threading was wrong and I cannot find anymore _which_ patches they were. I did find a comment that BNDCFGS must be added to msrs_to_save, but I don't know if that was v1 or v2 because you didn't add the version number when sending v2. Paolo ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM 2014-01-21 16:33 ` Paolo Bonzini @ 2014-01-21 17:26 ` Liu, Jinsong 0 siblings, 0 replies; 9+ messages in thread From: Liu, Jinsong @ 2014-01-21 17:26 UTC (permalink / raw) 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 Paolo Bonzini wrote: > Il 21/01/2014 16:25, Liu, Jinsong ha scritto: >> Hmm? I remember I have sent out the rebased patches v3 last month >> If you didn't receive them I'm OK to rebase and resend them. BTW, >> what's the review remarks? I remember you commented that the patches >> are fine. Any misunderstanding please point out to me. > > You sent v3 of QEMU, but not of KVM. I don't see any mail from you > after December 12th on kvm@vger.kernel.org. > > I can see my comment that the patches were fine (apart from needing a > rebase), but the threading was wrong and I cannot find anymore _which_ > patches they were. I did find a comment that BNDCFGS must be added to > msrs_to_save, but I don't know if that was v1 or v2 because you didn't > add the version number when sending v2. > > Paolo So I don't need resend qemu patches, just rebase and resend KVM MPX patches, is that right? Thanks, Jinsong ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2014-01-21 17:26 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1386394315-48984-1-git-send-email-jinsong.liu@intel.com>
2013-12-11 8:31 ` [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM Liu, Jinsong
2013-12-11 9:53 ` Paolo Bonzini
2013-12-12 5:47 ` Liu, Jinsong
2013-12-12 9:43 ` Paolo Bonzini
2013-12-12 11:09 ` Liu, Jinsong
2014-01-20 14:35 ` Paolo Bonzini
2014-01-21 15:25 ` Liu, Jinsong
2014-01-21 16:33 ` Paolo Bonzini
2014-01-21 17:26 ` Liu, Jinsong
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).