Re: [tip:perf/urgent] perf/x86: Set pmu->module in Intel PMU modules

2017-01-05 Thread Peter Zijlstra
On Thu, Jan 05, 2017 at 10:46:30AM -0800, Linus Torvalds wrote: > On Thu, Jan 5, 2017 at 7:52 AM, Peter Zijlstra wrote: > > > > The first time I saw this I thought about doing something like the > > below, but never got around to testing if that works and subsequently > >

Re: [tip:perf/urgent] perf/x86: Set pmu->module in Intel PMU modules

2017-01-05 Thread Peter Zijlstra
On Thu, Jan 05, 2017 at 10:46:30AM -0800, Linus Torvalds wrote: > On Thu, Jan 5, 2017 at 7:52 AM, Peter Zijlstra wrote: > > > > The first time I saw this I thought about doing something like the > > below, but never got around to testing if that works and subsequently > > forgot about things

Re: [tip:perf/urgent] perf/x86: Set pmu->module in Intel PMU modules

2017-01-05 Thread Linus Torvalds
On Thu, Jan 5, 2017 at 7:52 AM, Peter Zijlstra wrote: > > The first time I saw this I thought about doing something like the > below, but never got around to testing if that works and subsequently > forgot about things again. > > Does this make sense and or work? I'd argue

Re: [tip:perf/urgent] perf/x86: Set pmu->module in Intel PMU modules

2017-01-05 Thread Linus Torvalds
On Thu, Jan 5, 2017 at 7:52 AM, Peter Zijlstra wrote: > > The first time I saw this I thought about doing something like the > below, but never got around to testing if that works and subsequently > forgot about things again. > > Does this make sense and or work? I'd argue against it. It will

Re: [tip:perf/urgent] perf/x86: Set pmu->module in Intel PMU modules

2017-01-05 Thread Peter Zijlstra
On Thu, Jan 05, 2017 at 07:07:05AM -0800, tip-bot for David Carrillo-Cisneros wrote: > --- a/arch/x86/events/intel/cstate.c > +++ b/arch/x86/events/intel/cstate.c > @@ -434,6 +434,7 @@ static struct pmu cstate_core_pmu = { > .stop = cstate_pmu_event_stop, > .read =

Re: [tip:perf/urgent] perf/x86: Set pmu->module in Intel PMU modules

2017-01-05 Thread Peter Zijlstra
On Thu, Jan 05, 2017 at 07:07:05AM -0800, tip-bot for David Carrillo-Cisneros wrote: > --- a/arch/x86/events/intel/cstate.c > +++ b/arch/x86/events/intel/cstate.c > @@ -434,6 +434,7 @@ static struct pmu cstate_core_pmu = { > .stop = cstate_pmu_event_stop, > .read =

[tip:perf/urgent] perf/x86: Set pmu->module in Intel PMU modules

2017-01-05 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 74545f63890e38520eb4d1dbedcadaa9c0dbc824 Gitweb: http://git.kernel.org/tip/74545f63890e38520eb4d1dbedcadaa9c0dbc824 Author: David Carrillo-Cisneros AuthorDate: Thu, 22 Dec 2016 17:17:40 -0800 Committer: Ingo Molnar CommitDate: Thu, 5 Jan

[tip:perf/urgent] perf/x86: Set pmu->module in Intel PMU modules

2017-01-05 Thread tip-bot for David Carrillo-Cisneros
Commit-ID: 74545f63890e38520eb4d1dbedcadaa9c0dbc824 Gitweb: http://git.kernel.org/tip/74545f63890e38520eb4d1dbedcadaa9c0dbc824 Author: David Carrillo-Cisneros AuthorDate: Thu, 22 Dec 2016 17:17:40 -0800 Committer: Ingo Molnar CommitDate: Thu, 5 Jan 2017 09:13:55 +0100 perf/x86: Set