Re: [PATCH] perf: Only include tsc file for x86

From: David Ahern
Date: Fri Feb 20 2015 - 11:41:16 EST

On 2/20/15 1:36 AM, Adrian Hunter wrote:
On 19/02/15 21:42, Jiri Olsa wrote:
On Thu, Feb 19, 2015 at 01:22:33PM -0500, David Ahern wrote:
perf_time_to_tsc and tsc_to_perf_time are only used for x86. Make
inclusion of tsc.c dependent on x86 as well.

hum, should we move it to arch/x86/util/tsc.c and remove util/tsc.c?

It is not arch-specific because you can read a file
from one architecture on another architecture.

The TSC stuff was planned for Intel PT although it might now
be done differently.

Probably you should have CONFIG_X86_TSC and then select that
with CONFIG_X86. Later it can be selected with CONFIG_INTEL_PT
as well.

For now let's leave it as CONFIG_X86. Right now only X86 code needs the functions it provides and there is no need taking any other steps until something is going to use it.

ie., patch is good as is. Agreed?


looks like it's used only by test tsc code, which is enabled
for x86 only anyway..


Signed-off-by: David Ahern <david.ahern@xxxxxxxxxx>
Cc: Adrian Hunter <adrian.hunter@xxxxxxxxx>
Cc: Jiri Olsa <jolsa@xxxxxxxxxx>
tools/perf/util/Build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/util/Build b/tools/perf/util/Build
index 32f9327b1a97..4c7095785ba0 100644
--- a/tools/perf/util/Build
+++ b/tools/perf/util/Build
@@ -71,7 +71,7 @@ libperf-y += stat.o
libperf-y += record.o
libperf-y += srcline.o
libperf-y += data.o
-libperf-y += tsc.o
+libperf-$(CONFIG_X86) += tsc.o
libperf-y += cloexec.o
libperf-y += thread-stack.o


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at