Re: [Xen-devel] [PATCH 1/2] xenoprof: fix up ability to disable it

2016-02-09 Thread Boris Ostrovsky
On 02/08/2016 11:05 PM, Doug Goldstein wrote: On 2/8/16 10:24 AM, Boris Ostrovsky wrote: While at it --- I think we should put VPMU code under CONFIG option too, especially given its support status. I can do that (unless Doug wants to). I forgot to answer the VPMU part. If you have the band

Re: [Xen-devel] [PATCH 1/2] xenoprof: fix up ability to disable it

2016-02-08 Thread Doug Goldstein
On 2/8/16 10:24 AM, Boris Ostrovsky wrote: > > > On 02/08/2016 11:04 AM, Doug Goldstein wrote: >> Allow Xenoprof to be fully disabled when toggling the option off. >> >> CC: Keir Fraser >> CC: Jan Beulich >> CC: Andrew Cooper >> CC: Boris Ostrovsky >> CC: Suravee Suthikulpanit >> CC: Aravind

Re: [Xen-devel] [PATCH 1/2] xenoprof: fix up ability to disable it

2016-02-08 Thread Doug Goldstein
On 2/8/16 10:24 AM, Boris Ostrovsky wrote: > > > On 02/08/2016 11:04 AM, Doug Goldstein wrote: >> Allow Xenoprof to be fully disabled when toggling the option off. >> >> CC: Keir Fraser >> CC: Jan Beulich >> CC: Andrew Cooper >> CC: Boris Ostrovsky >> CC: Suravee Suthikulpanit >> CC: Aravind

Re: [Xen-devel] [PATCH 1/2] xenoprof: fix up ability to disable it

2016-02-08 Thread Andrew Cooper
On 08/02/16 16:04, Doug Goldstein wrote: > diff --git a/xen/arch/x86/cpu/vpmu_amd.c b/xen/arch/x86/cpu/vpmu_amd.c > index 990e6f3..c7b5bd5 100644 > --- a/xen/arch/x86/cpu/vpmu_amd.c > +++ b/xen/arch/x86/cpu/vpmu_amd.c > @@ -22,6 +22,7 @@ > */ > > #include > +#include > #include > #include

Re: [Xen-devel] [PATCH 1/2] xenoprof: fix up ability to disable it

2016-02-08 Thread Boris Ostrovsky
On 02/08/2016 11:04 AM, Doug Goldstein wrote: Allow Xenoprof to be fully disabled when toggling the option off. CC: Keir Fraser CC: Jan Beulich CC: Andrew Cooper CC: Boris Ostrovsky CC: Suravee Suthikulpanit CC: Aravind Gopalakrishnan CC: Jun Nakajima CC: Kevin Tian Signed-off-by: Doug

[Xen-devel] [PATCH 1/2] xenoprof: fix up ability to disable it

2016-02-08 Thread Doug Goldstein
Allow Xenoprof to be fully disabled when toggling the option off. CC: Keir Fraser CC: Jan Beulich CC: Andrew Cooper CC: Boris Ostrovsky CC: Suravee Suthikulpanit CC: Aravind Gopalakrishnan CC: Jun Nakajima CC: Kevin Tian Signed-off-by: Doug Goldstein --- xen/arch/x86/Makefile