All of lore.kernel.org
 help / color / mirror / Atom feed
* RE: [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
  0 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2013-12-11  8:31 UTC (permalink / raw)
  To: pbonzini, gleb, hpa, kvm, qemu-devel, linux-kernel

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] 20+ messages in thread

* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2013-12-11  8:31   ` Liu, Jinsong
  0 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2013-12-11  8:31 UTC (permalink / raw)
  To: pbonzini, gleb, hpa, kvm, qemu-devel, linux-kernel

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] 20+ messages in thread

* Re: [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
  2013-12-11  8:31   ` [Qemu-devel] " Liu, Jinsong
@ 2013-12-11  9:53     ` Paolo Bonzini
  -1 siblings, 0 replies; 20+ messages in thread
From: Paolo Bonzini @ 2013-12-11  9:53 UTC (permalink / raw)
  To: Liu, Jinsong; +Cc: gleb, hpa, kvm, qemu-devel, linux-kernel

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] 20+ messages in thread

* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2013-12-11  9:53     ` Paolo Bonzini
  0 siblings, 0 replies; 20+ messages in thread
From: Paolo Bonzini @ 2013-12-11  9:53 UTC (permalink / raw)
  To: Liu, Jinsong; +Cc: qemu-devel, linux-kernel, kvm, gleb, hpa

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] 20+ messages in thread

* RE: [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
  2013-12-11  9:53     ` [Qemu-devel] " Paolo Bonzini
  (?)
@ 2013-12-12  5:47       ` Liu, Jinsong
  -1 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2013-12-12  5:47 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gleb, hpa, kvm, qemu-devel, linux-kernel, 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] 20+ messages in thread

* Re: [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2013-12-12  5:47       ` Liu, Jinsong
  0 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2013-12-12  5:47 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gleb, kvm, qemu-devel, linux-kernel, hpa, 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] 20+ messages in thread

* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2013-12-12  5:47       ` Liu, Jinsong
  0 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2013-12-12  5:47 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gleb, kvm, qemu-devel, linux-kernel, hpa, 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] 20+ messages in thread

* Re: [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
  -1 siblings, 0 replies; 20+ messages in thread
From: Paolo Bonzini @ 2013-12-12  9:43 UTC (permalink / raw)
  To: Liu, Jinsong; +Cc: gleb, hpa, kvm, qemu-devel, linux-kernel, 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] 20+ messages in thread

* Re: [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2013-12-12  9:43         ` Paolo Bonzini
  0 siblings, 0 replies; 20+ messages in thread
From: Paolo Bonzini @ 2013-12-12  9:43 UTC (permalink / raw)
  To: Liu, Jinsong; +Cc: gleb, kvm, qemu-devel, linux-kernel, hpa, 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] 20+ messages in thread

* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2013-12-12  9:43         ` Paolo Bonzini
  0 siblings, 0 replies; 20+ messages in thread
From: Paolo Bonzini @ 2013-12-12  9:43 UTC (permalink / raw)
  To: Liu, Jinsong; +Cc: gleb, kvm, qemu-devel, linux-kernel, hpa, 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] 20+ messages in thread

* RE: [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
  -1 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2013-12-12 11:09 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gleb, hpa, kvm, qemu-devel, linux-kernel, 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] 20+ messages in thread

* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2013-12-12 11:09           ` Liu, Jinsong
  0 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2013-12-12 11:09 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gleb, kvm, qemu-devel, linux-kernel, hpa, 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] 20+ messages in thread

* Re: [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
  2013-12-12 11:09           ` [Qemu-devel] " Liu, Jinsong
@ 2014-01-20 14:35             ` Paolo Bonzini
  -1 siblings, 0 replies; 20+ messages in thread
From: Paolo Bonzini @ 2014-01-20 14:35 UTC (permalink / raw)
  To: Liu, Jinsong; +Cc: gleb, hpa, kvm, qemu-devel, linux-kernel, 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] 20+ messages in thread

* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2014-01-20 14:35             ` Paolo Bonzini
  0 siblings, 0 replies; 20+ messages in thread
From: Paolo Bonzini @ 2014-01-20 14:35 UTC (permalink / raw)
  To: Liu, Jinsong; +Cc: gleb, kvm, qemu-devel, linux-kernel, hpa, 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] 20+ messages in thread

* RE: [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
  2014-01-20 14:35             ` [Qemu-devel] " Paolo Bonzini
@ 2014-01-21 15:25               ` Liu, Jinsong
  -1 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2014-01-21 15:25 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gleb, hpa, kvm, qemu-devel, linux-kernel, 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] 20+ messages in thread

* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2014-01-21 15:25               ` Liu, Jinsong
  0 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2014-01-21 15:25 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gleb, kvm, qemu-devel, linux-kernel, hpa, 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] 20+ messages in thread

* Re: [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
  2014-01-21 15:25               ` [Qemu-devel] " Liu, Jinsong
@ 2014-01-21 16:33                 ` Paolo Bonzini
  -1 siblings, 0 replies; 20+ messages in thread
From: Paolo Bonzini @ 2014-01-21 16:33 UTC (permalink / raw)
  To: Liu, Jinsong; +Cc: gleb, hpa, kvm, qemu-devel, linux-kernel, 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] 20+ messages in thread

* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2014-01-21 16:33                 ` Paolo Bonzini
  0 siblings, 0 replies; 20+ messages in thread
From: Paolo Bonzini @ 2014-01-21 16:33 UTC (permalink / raw)
  To: Liu, Jinsong; +Cc: gleb, kvm, qemu-devel, linux-kernel, hpa, 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] 20+ messages in thread

* RE: [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
  2014-01-21 16:33                 ` [Qemu-devel] " Paolo Bonzini
@ 2014-01-21 17:26                   ` Liu, Jinsong
  -1 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2014-01-21 17:26 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gleb, hpa, kvm, qemu-devel, linux-kernel, 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] 20+ messages in thread

* Re: [Qemu-devel] [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM
@ 2014-01-21 17:26                   ` Liu, Jinsong
  0 siblings, 0 replies; 20+ messages in thread
From: Liu, Jinsong @ 2014-01-21 17:26 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: gleb, kvm, qemu-devel, linux-kernel, hpa, 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] 20+ messages in thread

end of thread, other threads:[~2014-01-21 17:26 UTC | newest]

Thread overview: 20+ 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 ` [PATCH v2 0/4] X86/KVM: enable Intel MPX for KVM Liu, Jinsong
2013-12-11  8:31   ` [Qemu-devel] " Liu, Jinsong
2013-12-11  9:53   ` Paolo Bonzini
2013-12-11  9:53     ` [Qemu-devel] " Paolo Bonzini
2013-12-12  5:47     ` Liu, Jinsong
2013-12-12  5:47       ` [Qemu-devel] " Liu, Jinsong
2013-12-12  5:47       ` Liu, Jinsong
2013-12-12  9:43       ` Paolo Bonzini
2013-12-12  9:43         ` [Qemu-devel] " Paolo Bonzini
2013-12-12  9:43         ` Paolo Bonzini
2013-12-12 11:09         ` Liu, Jinsong
2013-12-12 11:09           ` [Qemu-devel] " Liu, Jinsong
2014-01-20 14:35           ` Paolo Bonzini
2014-01-20 14:35             ` [Qemu-devel] " Paolo Bonzini
2014-01-21 15:25             ` Liu, Jinsong
2014-01-21 15:25               ` [Qemu-devel] " Liu, Jinsong
2014-01-21 16:33               ` Paolo Bonzini
2014-01-21 16:33                 ` [Qemu-devel] " Paolo Bonzini
2014-01-21 17:26                 ` Liu, Jinsong
2014-01-21 17:26                   ` [Qemu-devel] " Liu, Jinsong

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.