All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] KVM: fix a syntax error in pmu.c
@ 2011-12-29  9:40 Ren, Yongjie
  2011-12-29  9:46 ` Gleb Natapov
  0 siblings, 1 reply; 3+ messages in thread
From: Ren, Yongjie @ 2011-12-29  9:40 UTC (permalink / raw)
  To: avi@redhat.com; +Cc: Gleb Natapov, kvm@vger.kernel.org

Fix a syntax error in arch/x86/kvm/pmu.c

Signed-off-by: Yongjie Ren <yongjie.ren@intel.com>
---
diff --git a/arch/x86/kvm/pmu.c b/arch/x86/kvm/pmu.c
index 061a03f..8f142bf 100644
--- a/arch/x86/kvm/pmu.c
+++ b/arch/x86/kvm/pmu.c
@@ -170,9 +170,9 @@ static void reprogram_counter(struct kvm_pmc *pmc, u32 type,
 		.exclude_host = 1,
 		.exclude_user = exclude_user,
 		.exclude_kernel = exclude_kernel,
-		.sample_period = (-pmc->counter) & pmc_bitmask(pmc),
 		.config = config,
 	};
+	attr.sample_period = (-pmc->counter) & pmc_bitmask(pmc),
 
 	event = perf_event_create_kernel_counter(&attr, -1, current,
 						 intr ? kvm_perf_overflow_intr :


Best Regards,
     Yongjie Ren  (Jay)



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] KVM: fix a syntax error in pmu.c
  2011-12-29  9:40 [PATCH] KVM: fix a syntax error in pmu.c Ren, Yongjie
@ 2011-12-29  9:46 ` Gleb Natapov
  2011-12-29  9:50   ` Ren, Yongjie
  0 siblings, 1 reply; 3+ messages in thread
From: Gleb Natapov @ 2011-12-29  9:46 UTC (permalink / raw)
  To: Ren, Yongjie; +Cc: avi@redhat.com, kvm@vger.kernel.org

On Thu, Dec 29, 2011 at 09:40:37AM +0000, Ren, Yongjie wrote:
> Fix a syntax error in arch/x86/kvm/pmu.c
> 
That is fixed already in kvm.git next branch.

> Signed-off-by: Yongjie Ren <yongjie.ren@intel.com>
> ---
> diff --git a/arch/x86/kvm/pmu.c b/arch/x86/kvm/pmu.c
> index 061a03f..8f142bf 100644
> --- a/arch/x86/kvm/pmu.c
> +++ b/arch/x86/kvm/pmu.c
> @@ -170,9 +170,9 @@ static void reprogram_counter(struct kvm_pmc *pmc, u32 type,
>  		.exclude_host = 1,
>  		.exclude_user = exclude_user,
>  		.exclude_kernel = exclude_kernel,
> -		.sample_period = (-pmc->counter) & pmc_bitmask(pmc),
>  		.config = config,
>  	};
> +	attr.sample_period = (-pmc->counter) & pmc_bitmask(pmc),
>  
>  	event = perf_event_create_kernel_counter(&attr, -1, current,
>  						 intr ? kvm_perf_overflow_intr :
> 
> 
> Best Regards,
>      Yongjie Ren  (Jay)
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
			Gleb.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [PATCH] KVM: fix a syntax error in pmu.c
  2011-12-29  9:46 ` Gleb Natapov
@ 2011-12-29  9:50   ` Ren, Yongjie
  0 siblings, 0 replies; 3+ messages in thread
From: Ren, Yongjie @ 2011-12-29  9:50 UTC (permalink / raw)
  To: Gleb Natapov; +Cc: avi@redhat.com, kvm@vger.kernel.org

> -----Original Message-----
> From: Gleb Natapov [mailto:gleb@redhat.com]
> Sent: Thursday, December 29, 2011 5:46 PM
> To: Ren, Yongjie
> Cc: avi@redhat.com; kvm@vger.kernel.org
> Subject: Re: [PATCH] KVM: fix a syntax error in pmu.c
> 
> On Thu, Dec 29, 2011 at 09:40:37AM +0000, Ren, Yongjie wrote:
> > Fix a syntax error in arch/x86/kvm/pmu.c
> >
> That is fixed already in kvm.git next branch.
> 
OK. Waiting for the fix in mater tree.

> > Signed-off-by: Yongjie Ren <yongjie.ren@intel.com>
> > ---
> > diff --git a/arch/x86/kvm/pmu.c b/arch/x86/kvm/pmu.c
> > index 061a03f..8f142bf 100644
> > --- a/arch/x86/kvm/pmu.c
> > +++ b/arch/x86/kvm/pmu.c
> > @@ -170,9 +170,9 @@ static void reprogram_counter(struct kvm_pmc
> *pmc, u32 type,
> >  		.exclude_host = 1,
> >  		.exclude_user = exclude_user,
> >  		.exclude_kernel = exclude_kernel,
> > -		.sample_period = (-pmc->counter) & pmc_bitmask(pmc),
> >  		.config = config,
> >  	};
> > +	attr.sample_period = (-pmc->counter) & pmc_bitmask(pmc),
> >
> >  	event = perf_event_create_kernel_counter(&attr, -1, current,
> >  						 intr ? kvm_perf_overflow_intr :
> >
> >
> > Best Regards,
> >      Yongjie Ren  (Jay)
> >
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe kvm" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> --
> 			Gleb.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-12-29  9:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-29  9:40 [PATCH] KVM: fix a syntax error in pmu.c Ren, Yongjie
2011-12-29  9:46 ` Gleb Natapov
2011-12-29  9:50   ` Ren, Yongjie

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.