[tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-06 Thread tip-bot for Andi Kleen
Commit-ID: 7f47d8cc039f8746e0038fe05f1ddcb15a2e27f0 Gitweb: http://git.kernel.org/tip/7f47d8cc039f8746e0038fe05f1ddcb15a2e27f0 Author: Andi Kleen AuthorDate: Tue, 1 Dec 2015 17:00:59 -0800 Committer: Ingo Molnar CommitDate: Sun, 6 Dec 2015 12:56:10 +0100 x86, tracing, perf: Add trace p

Re: [tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-04 Thread H. Peter Anvin
On December 4, 2015 2:48:37 PM PST, Borislav Petkov wrote: >On Fri, Dec 04, 2015 at 02:35:38PM -0800, H. Peter Anvin wrote: >> How about this: it is easy too easy to hard code MSR accesses, and >the >> last things we need is alien non-GPL drivers doing their own >low-level >> hacks bypassing these

Re: [tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-04 Thread Borislav Petkov
On Fri, Dec 04, 2015 at 02:35:38PM -0800, H. Peter Anvin wrote: > How about this: it is easy too easy to hard code MSR accesses, and the > last things we need is alien non-GPL drivers doing their own low-level > hacks bypassing these facilities. Yeah, that doesn't stop alien, non-GPL drivers from

Re: [tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-04 Thread H. Peter Anvin
On December 4, 2015 10:30:17 AM PST, Borislav Petkov wrote: >On Fri, Dec 04, 2015 at 10:28:02AM -0800, Andi Kleen wrote: >> Because making them GPL would prevent any non GPL driver from >> using MSRs with tracing compiled in, which doesn't make any sense. > >I know what EXPORT_SYMBOL_GPL means - I

Re: [tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-04 Thread Borislav Petkov
On Fri, Dec 04, 2015 at 02:18:56PM -0800, Andi Kleen wrote: > On Fri, Dec 04, 2015 at 07:30:17PM +0100, Borislav Petkov wrote: > > On Fri, Dec 04, 2015 at 10:28:02AM -0800, Andi Kleen wrote: > > > Because making them GPL would prevent any non GPL driver from > > > using MSRs with tracing compiled i

Re: [tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-04 Thread Andi Kleen
On Fri, Dec 04, 2015 at 07:30:17PM +0100, Borislav Petkov wrote: > On Fri, Dec 04, 2015 at 10:28:02AM -0800, Andi Kleen wrote: > > Because making them GPL would prevent any non GPL driver from > > using MSRs with tracing compiled in, which doesn't make any sense. > > I know what EXPORT_SYMBOL_GPL

Re: [tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-04 Thread Borislav Petkov
On Fri, Dec 04, 2015 at 10:28:02AM -0800, Andi Kleen wrote: > Because making them GPL would prevent any non GPL driver from > using MSRs with tracing compiled in, which doesn't make any sense. I know what EXPORT_SYMBOL_GPL means - I'm questioning the need of making them available to non-GPL driver

Re: [tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-04 Thread Andi Kleen
> Any particular reason why those are EXPORT_SYMBOL and not > EXPORT_SYMBOL_GPL? Because making them GPL would prevent any non GPL driver from using MSRs with tracing compiled in, which doesn't make any sense. -Andi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the

Re: [tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-04 Thread Borislav Petkov
On Fri, Dec 04, 2015 at 03:59:42AM -0800, tip-bot for Andi Kleen wrote: > Commit-ID: 07b41f5cf37b34ef3de4f40fbaf8547abe690abb > Gitweb: http://git.kernel.org/tip/07b41f5cf37b34ef3de4f40fbaf8547abe690abb > Author: Andi Kleen > AuthorDate: Tue, 1 Dec 2015 17:00:59 -0800 > Committer: Ingo M

[tip:perf/core] x86, tracing, perf: Add trace point for MSR accesses

2015-12-04 Thread tip-bot for Andi Kleen
Commit-ID: 07b41f5cf37b34ef3de4f40fbaf8547abe690abb Gitweb: http://git.kernel.org/tip/07b41f5cf37b34ef3de4f40fbaf8547abe690abb Author: Andi Kleen AuthorDate: Tue, 1 Dec 2015 17:00:59 -0800 Committer: Ingo Molnar CommitDate: Fri, 4 Dec 2015 10:58:34 +0100 x86, tracing, perf: Add trace p