RE: [PATCH] KVM: emulate lapic tsc deadline timer for guest

2011-10-05 Thread Liu, Jinsong
Marcelo Tosatti wrote:
> On Sun, Sep 25, 2011 at 10:47:46PM +0800, Liu, Jinsong wrote:
>> Marcelo Tosatti wrote:
>>> On Fri, Sep 23, 2011 at 04:25:51PM +0800, Liu, Jinsong wrote:
 Marcelo Tosatti wrote:
> On Thu, Sep 22, 2011 at 04:55:52PM +0800, Liu, Jinsong wrote:
>>> From 4d5b83aba40ce0d421add9a41a6c591a8590a32e Mon Sep 17
>>> 00:00:00 2001
>> From: Liu, Jinsong 
>> Date: Thu, 22 Sep 2011 14:00:08 +0800
>> Subject: [PATCH 2/2] KVM: emulate lapic tsc deadline timer for
>> guest 
>> 
>> This patch emulate lapic tsc deadline timer for guest:
>> Enumerate tsc deadline timer capability by CPUID;
>> Enable tsc deadline timer mode by lapic MMIO;
>> Start tsc deadline timer by WRMSR;
>> 
>> Signed-off-by: Liu, Jinsong  ---
>>  arch/x86/include/asm/kvm_host.h |2 +
>>  arch/x86/kvm/kvm_timer.h|2 +
>>  arch/x86/kvm/lapic.c|  123
>>  --- arch/x86/kvm/lapic.h
>>  |3 + arch/x86/kvm/x86.c  |   16 +-
>>  5 files changed, 123 insertions(+), 23 deletions(-)
> 
> Looks good, please rebase against branch master of
> 
> git://github.com/avikivity/kvm.git
 
 Rebased as attached.
 
 Thanks,
 Jinsong
>>> 
>>> Please write a simple test case to arm a lapic timer via wrmsr (see
>>> https://github.com/avikivity/kvm-unit-tests).
>>> 
>>> Kernel patches have been applied, thanks.
>> 
>> Marcelo,
>> 
>> I'm not quite clear the purpose and usage of test case of the
>> kvm-unit-tests. Can you give me some hint?
> 
> The purpose is to add unit tests for new features (such as lapic
> deadline timer). There are examples that make it relatively easy to
> construct new test case (or modify existing ones to accomodate new
> tests).
> 
> Please add a new test case for lapic deadline timer, thanks.


Thanks Marcelo. I will add the test case. Sorry for slow email reply because of 
holiday.

Regards,
Jinsong
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] KVM: emulate lapic tsc deadline timer for guest

2011-10-03 Thread Marcelo Tosatti
On Sun, Sep 25, 2011 at 10:47:46PM +0800, Liu, Jinsong wrote:
> Marcelo Tosatti wrote:
> > On Fri, Sep 23, 2011 at 04:25:51PM +0800, Liu, Jinsong wrote:
> >> Marcelo Tosatti wrote:
> >>> On Thu, Sep 22, 2011 at 04:55:52PM +0800, Liu, Jinsong wrote:
> > From 4d5b83aba40ce0d421add9a41a6c591a8590a32e Mon Sep 17 00:00:00
> > 2001
>  From: Liu, Jinsong 
>  Date: Thu, 22 Sep 2011 14:00:08 +0800
>  Subject: [PATCH 2/2] KVM: emulate lapic tsc deadline timer for
>  guest 
>  
>  This patch emulate lapic tsc deadline timer for guest:
>  Enumerate tsc deadline timer capability by CPUID;
>  Enable tsc deadline timer mode by lapic MMIO;
>  Start tsc deadline timer by WRMSR;
>  
>  Signed-off-by: Liu, Jinsong  ---
>   arch/x86/include/asm/kvm_host.h |2 +
>   arch/x86/kvm/kvm_timer.h|2 +
>   arch/x86/kvm/lapic.c|  123
>   --- arch/x86/kvm/lapic.h
>   |3 + arch/x86/kvm/x86.c  |   16 +-
>   5 files changed, 123 insertions(+), 23 deletions(-)
> >>> 
> >>> Looks good, please rebase against branch master of
> >>> 
> >>> git://github.com/avikivity/kvm.git
> >> 
> >> Rebased as attached.
> >> 
> >> Thanks,
> >> Jinsong
> > 
> > Please write a simple test case to arm a lapic timer via wrmsr (see
> > https://github.com/avikivity/kvm-unit-tests).
> > 
> > Kernel patches have been applied, thanks.
> 
> Marcelo,
> 
> I'm not quite clear the purpose and usage of test case of the kvm-unit-tests.
> Can you give me some hint?

The purpose is to add unit tests for new features (such as lapic
deadline timer). There are examples that make it relatively easy to
construct new test case (or modify existing ones to accomodate new
tests).

Please add a new test case for lapic deadline timer, thanks.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH] KVM: emulate lapic tsc deadline timer for guest

2011-09-25 Thread Liu, Jinsong
Marcelo Tosatti wrote:
> On Fri, Sep 23, 2011 at 04:25:51PM +0800, Liu, Jinsong wrote:
>> Marcelo Tosatti wrote:
>>> On Thu, Sep 22, 2011 at 04:55:52PM +0800, Liu, Jinsong wrote:
> From 4d5b83aba40ce0d421add9a41a6c591a8590a32e Mon Sep 17 00:00:00
> 2001
 From: Liu, Jinsong 
 Date: Thu, 22 Sep 2011 14:00:08 +0800
 Subject: [PATCH 2/2] KVM: emulate lapic tsc deadline timer for
 guest 
 
 This patch emulate lapic tsc deadline timer for guest:
 Enumerate tsc deadline timer capability by CPUID;
 Enable tsc deadline timer mode by lapic MMIO;
 Start tsc deadline timer by WRMSR;
 
 Signed-off-by: Liu, Jinsong  ---
  arch/x86/include/asm/kvm_host.h |2 +
  arch/x86/kvm/kvm_timer.h|2 +
  arch/x86/kvm/lapic.c|  123
  --- arch/x86/kvm/lapic.h
  |3 + arch/x86/kvm/x86.c  |   16 +-
  5 files changed, 123 insertions(+), 23 deletions(-)
>>> 
>>> Looks good, please rebase against branch master of
>>> 
>>> git://github.com/avikivity/kvm.git
>> 
>> Rebased as attached.
>> 
>> Thanks,
>> Jinsong
> 
> Please write a simple test case to arm a lapic timer via wrmsr (see
> https://github.com/avikivity/kvm-unit-tests).
> 
> Kernel patches have been applied, thanks.

Marcelo,

I'm not quite clear the purpose and usage of test case of the kvm-unit-tests.
Can you give me some hint?

Thanks,
Jinsong--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] KVM: emulate lapic tsc deadline timer for guest

2011-09-23 Thread Marcelo Tosatti
On Fri, Sep 23, 2011 at 04:25:51PM +0800, Liu, Jinsong wrote:
> Marcelo Tosatti wrote:
> > On Thu, Sep 22, 2011 at 04:55:52PM +0800, Liu, Jinsong wrote:
> >>> From 4d5b83aba40ce0d421add9a41a6c591a8590a32e Mon Sep 17 00:00:00
> >>> 2001 
> >> From: Liu, Jinsong 
> >> Date: Thu, 22 Sep 2011 14:00:08 +0800
> >> Subject: [PATCH 2/2] KVM: emulate lapic tsc deadline timer for guest
> >> 
> >> This patch emulate lapic tsc deadline timer for guest:
> >> Enumerate tsc deadline timer capability by CPUID;
> >> Enable tsc deadline timer mode by lapic MMIO;
> >> Start tsc deadline timer by WRMSR;
> >> 
> >> Signed-off-by: Liu, Jinsong  ---
> >>  arch/x86/include/asm/kvm_host.h |2 +
> >>  arch/x86/kvm/kvm_timer.h|2 +
> >>  arch/x86/kvm/lapic.c|  123
> >>  --- arch/x86/kvm/lapic.h   
> >>  |3 + arch/x86/kvm/x86.c  |   16 +-
> >>  5 files changed, 123 insertions(+), 23 deletions(-)
> > 
> > Looks good, please rebase against branch master of
> > 
> > git://github.com/avikivity/kvm.git
> 
> Rebased as attached.
> 
> Thanks,
> Jinsong

Please write a simple test case to arm a lapic timer via wrmsr (see
https://github.com/avikivity/kvm-unit-tests).

Kernel patches have been applied, thanks.

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH] KVM: emulate lapic tsc deadline timer for guest

2011-09-23 Thread Liu, Jinsong
Marcelo Tosatti wrote:
> On Thu, Sep 22, 2011 at 04:55:52PM +0800, Liu, Jinsong wrote:
>>> From 4d5b83aba40ce0d421add9a41a6c591a8590a32e Mon Sep 17 00:00:00
>>> 2001 
>> From: Liu, Jinsong 
>> Date: Thu, 22 Sep 2011 14:00:08 +0800
>> Subject: [PATCH 2/2] KVM: emulate lapic tsc deadline timer for guest
>> 
>> This patch emulate lapic tsc deadline timer for guest:
>> Enumerate tsc deadline timer capability by CPUID;
>> Enable tsc deadline timer mode by lapic MMIO;
>> Start tsc deadline timer by WRMSR;
>> 
>> Signed-off-by: Liu, Jinsong  ---
>>  arch/x86/include/asm/kvm_host.h |2 +
>>  arch/x86/kvm/kvm_timer.h|2 +
>>  arch/x86/kvm/lapic.c|  123
>>  --- arch/x86/kvm/lapic.h   
>>  |3 + arch/x86/kvm/x86.c  |   16 +-
>>  5 files changed, 123 insertions(+), 23 deletions(-)
> 
> Looks good, please rebase against branch master of
> 
> git://github.com/avikivity/kvm.git

Rebased as attached.

Thanks,
Jinsong

qemu-tsc-deadline-timer.patch
Description: qemu-tsc-deadline-timer.patch


Add-some-pre-defination.patch
Description: Add-some-pre-defination.patch


kvm-tsc-deadline-timer.patch
Description: kvm-tsc-deadline-timer.patch


Re: [PATCH] KVM: emulate lapic tsc deadline timer for guest

2011-09-22 Thread Marcelo Tosatti
On Thu, Sep 22, 2011 at 11:22:02PM +0800, Liu, Jinsong wrote:
> Marcelo Tosatti wrote:
> > On Thu, Sep 22, 2011 at 04:55:52PM +0800, Liu, Jinsong wrote:
> >>> From 4d5b83aba40ce0d421add9a41a6c591a8590a32e Mon Sep 17 00:00:00
> >>> 2001 
> >> From: Liu, Jinsong 
> >> Date: Thu, 22 Sep 2011 14:00:08 +0800
> >> Subject: [PATCH 2/2] KVM: emulate lapic tsc deadline timer for guest
> >> 
> >> This patch emulate lapic tsc deadline timer for guest:
> >> Enumerate tsc deadline timer capability by CPUID;
> >> Enable tsc deadline timer mode by lapic MMIO;
> >> Start tsc deadline timer by WRMSR;
> >> 
> >> Signed-off-by: Liu, Jinsong  ---
> >>  arch/x86/include/asm/kvm_host.h |2 +
> >>  arch/x86/kvm/kvm_timer.h|2 +
> >>  arch/x86/kvm/lapic.c|  123
> >>  --- arch/x86/kvm/lapic.h   
> >>  |3 + arch/x86/kvm/x86.c  |   16 +-
> >>  5 files changed, 123 insertions(+), 23 deletions(-)
> > 
> > Looks good, please rebase against branch master of
> > 
> > git://github.com/avikivity/kvm.git
> 
> Thanks!
> 
> And, for qemu patch rebase, I guess below address, which one is right?
> git://github.com/avikivity/qemu-kvm.git   or
> git://github.com/avikivity/qemu.git

qemu.git. 
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH] KVM: emulate lapic tsc deadline timer for guest

2011-09-22 Thread Liu, Jinsong
Marcelo Tosatti wrote:
> On Thu, Sep 22, 2011 at 04:55:52PM +0800, Liu, Jinsong wrote:
>>> From 4d5b83aba40ce0d421add9a41a6c591a8590a32e Mon Sep 17 00:00:00
>>> 2001 
>> From: Liu, Jinsong 
>> Date: Thu, 22 Sep 2011 14:00:08 +0800
>> Subject: [PATCH 2/2] KVM: emulate lapic tsc deadline timer for guest
>> 
>> This patch emulate lapic tsc deadline timer for guest:
>> Enumerate tsc deadline timer capability by CPUID;
>> Enable tsc deadline timer mode by lapic MMIO;
>> Start tsc deadline timer by WRMSR;
>> 
>> Signed-off-by: Liu, Jinsong  ---
>>  arch/x86/include/asm/kvm_host.h |2 +
>>  arch/x86/kvm/kvm_timer.h|2 +
>>  arch/x86/kvm/lapic.c|  123
>>  --- arch/x86/kvm/lapic.h   
>>  |3 + arch/x86/kvm/x86.c  |   16 +-
>>  5 files changed, 123 insertions(+), 23 deletions(-)
> 
> Looks good, please rebase against branch master of
> 
> git://github.com/avikivity/kvm.git

Thanks!

And, for qemu patch rebase, I guess below address, which one is right?
git://github.com/avikivity/qemu-kvm.git   or
git://github.com/avikivity/qemu.git
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] KVM: emulate lapic tsc deadline timer for guest

2011-09-22 Thread Marcelo Tosatti
On Thu, Sep 22, 2011 at 04:55:52PM +0800, Liu, Jinsong wrote:
> >From 4d5b83aba40ce0d421add9a41a6c591a8590a32e Mon Sep 17 00:00:00 2001
> From: Liu, Jinsong 
> Date: Thu, 22 Sep 2011 14:00:08 +0800
> Subject: [PATCH 2/2] KVM: emulate lapic tsc deadline timer for guest
> 
> This patch emulate lapic tsc deadline timer for guest:
> Enumerate tsc deadline timer capability by CPUID;
> Enable tsc deadline timer mode by lapic MMIO;
> Start tsc deadline timer by WRMSR;
> 
> Signed-off-by: Liu, Jinsong 
> ---
>  arch/x86/include/asm/kvm_host.h |2 +
>  arch/x86/kvm/kvm_timer.h|2 +
>  arch/x86/kvm/lapic.c|  123 
> ---
>  arch/x86/kvm/lapic.h|3 +
>  arch/x86/kvm/x86.c  |   16 +-
>  5 files changed, 123 insertions(+), 23 deletions(-)

Looks good, please rebase against branch master of

git://github.com/avikivity/kvm.git 

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html