[PATCH] KVM: PPC: Make Performance Counters work

[PATCH] KVM: PPC: Make Performance Counters work

[Alexander Graf]

When we get a performance counter interrupt we need to route it on to the
Linux handler after we got out of the guest context. We also need to tell
our handling code that this particular interrupt doesn't need treatment.

So let's add those two bits in, making perf work while having a KVM guest
running.

Signed-off-by: Alexander Graf <agraf@suse.de>
---
 arch/powerpc/kvm/book3s.c            |    3 +++
 arch/powerpc/kvm/book3s_interrupts.S |    2 ++
 2 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/arch/powerpc/kvm/book3s.c b/arch/powerpc/kvm/book3s.c
index a7de709..a03163b 100644
--- a/arch/powerpc/kvm/book3s.c
+++ b/arch/powerpc/kvm/book3s.c
@@ -872,6 +872,9 @@ int kvmppc_handle_exit(struct kvm_run *run, struct kvm_vcpu *vcpu,
 		vcpu->stat.ext_intr_exits++;
 		r = RESUME_GUEST;
 		break;
+	case BOOK3S_INTERRUPT_PERFMON:
+		r = RESUME_GUEST;
+		break;
 	case BOOK3S_INTERRUPT_PROGRAM:
 	{
 		enum emulation_result er;
diff --git a/arch/powerpc/kvm/book3s_interrupts.S b/arch/powerpc/kvm/book3s_interrupts.S
index f5b3358..e486193 100644
--- a/arch/powerpc/kvm/book3s_interrupts.S
+++ b/arch/powerpc/kvm/book3s_interrupts.S
@@ -228,6 +228,8 @@ no_dcbz32_off:
 	beq	call_linux_handler
 	cmpwi	r12, BOOK3S_INTERRUPT_DECREMENTER
 	beq	call_linux_handler
+	cmpwi	r12, BOOK3S_INTERRUPT_PERFMON
+	beq	call_linux_handler
 
 	/* Back to EE=1 */
 	mtmsr	r6
-- 
1.6.0.2

[PATCH] KVM: PPC: Make Performance Counters work

[Alexander Graf]

When we get a performance counter interrupt we need to route it on to the
Linux handler after we got out of the guest context. We also need to tell
our handling code that this particular interrupt doesn't need treatment.

So let's add those two bits in, making perf work while having a KVM guest
running.

Signed-off-by: Alexander Graf <agraf-l3A5Bk7waGM@public.gmane.org>
---
 arch/powerpc/kvm/book3s.c            |    3 +++
 arch/powerpc/kvm/book3s_interrupts.S |    2 ++
 2 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/arch/powerpc/kvm/book3s.c b/arch/powerpc/kvm/book3s.c
index a7de709..a03163b 100644
--- a/arch/powerpc/kvm/book3s.c
+++ b/arch/powerpc/kvm/book3s.c
@@ -872,6 +872,9 @@ int kvmppc_handle_exit(struct kvm_run *run, struct kvm_vcpu *vcpu,
 		vcpu->stat.ext_intr_exits++;
 		r = RESUME_GUEST;
 		break;
+	case BOOK3S_INTERRUPT_PERFMON:
+		r = RESUME_GUEST;
+		break;
 	case BOOK3S_INTERRUPT_PROGRAM:
 	{
 		enum emulation_result er;
diff --git a/arch/powerpc/kvm/book3s_interrupts.S b/arch/powerpc/kvm/book3s_interrupts.S
index f5b3358..e486193 100644
--- a/arch/powerpc/kvm/book3s_interrupts.S
+++ b/arch/powerpc/kvm/book3s_interrupts.S
@@ -228,6 +228,8 @@ no_dcbz32_off:
 	beq	call_linux_handler
 	cmpwi	r12, BOOK3S_INTERRUPT_DECREMENTER
 	beq	call_linux_handler
+	cmpwi	r12, BOOK3S_INTERRUPT_PERFMON
+	beq	call_linux_handler
 
 	/* Back to EE=1 */
 	mtmsr	r6
-- 
1.6.0.2

Re: [PATCH] KVM: PPC: Make Performance Counters work

[Avi Kivity]

On 04/17/2010 01:22 AM, Alexander Graf wrote:
> When we get a performance counter interrupt we need to route it on to the
> Linux handler after we got out of the guest context. We also need to tell
> our handling code that this particular interrupt doesn't need treatment.
>
> So let's add those two bits in, making perf work while having a KVM guest
> running.
>
>    

Doesn't apply - does it depend on another patchset?

-- 
error compiling committee.c: too many arguments to function

Re: [PATCH] KVM: PPC: Make Performance Counters work

[Avi Kivity]

On 04/17/2010 01:22 AM, Alexander Graf wrote:
> When we get a performance counter interrupt we need to route it on to the
> Linux handler after we got out of the guest context. We also need to tell
> our handling code that this particular interrupt doesn't need treatment.
>
> So let's add those two bits in, making perf work while having a KVM guest
> running.
>
>    

Doesn't apply - does it depend on another patchset?

-- 
error compiling committee.c: too many arguments to function

Re: [PATCH] KVM: PPC: Make Performance Counters work

[Alexander Graf]

On 19.04.2010, at 12:34, Avi Kivity wrote:

> On 04/17/2010 01:22 AM, Alexander Graf wrote:
>> When we get a performance counter interrupt we need to route it on to the
>> Linux handler after we got out of the guest context. We also need to tell
>> our handling code that this particular interrupt doesn't need treatment.
>> 
>> So let's add those two bits in, making perf work while having a KVM guest
>> running.
>> 
>>   
> 
> Doesn't apply - does it depend on another patchset?

This patch is deprecated by the "Post-PPC32" patchset. I only noted that in the cover letter of the Post-PPC32 patchset, not in a reply to this patch. Sorry.

Alex

Re: [PATCH] KVM: PPC: Make Performance Counters work

[Alexander Graf]

On 19.04.2010, at 12:34, Avi Kivity wrote:

> On 04/17/2010 01:22 AM, Alexander Graf wrote:
>> When we get a performance counter interrupt we need to route it on to the
>> Linux handler after we got out of the guest context. We also need to tell
>> our handling code that this particular interrupt doesn't need treatment.
>> 
>> So let's add those two bits in, making perf work while having a KVM guest
>> running.
>> 
>>   
> 
> Doesn't apply - does it depend on another patchset?

This patch is deprecated by the "Post-PPC32" patchset. I only noted that in the cover letter of the Post-PPC32 patchset, not in a reply to this patch. Sorry.

Alex