Skip to contentSkip to conversation

perf tools: Link libperf-jvmti.so with LDFLAGS variable

PATCH Applied Jiri Olsa 3 participants 3 messages Dec 12, 2018 – Dec 20, 2018 perf events

lore
Share & tools
lore

Jiri Olsa

author reviewer
#0 lore raw

PATCH 1/1 perf tools: Link libperf-jvmti.so with LDFLAGS variable

So we could propagate distro flags into libperf-jvmti.so library.

---
 tools/perf/Makefile.perf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
tools/perf/Makefile.perf +1 -1
index 5a90c0ef2302..6f35f4d46b0a 100644
@@ -660,7 +660,7 @@ $(LIBJVMTI_IN): FORCE
 	$(Q)$(MAKE) -f $(srctree)/tools/build/Makefile.build dir=jvmti obj=jvmti
 
 $(OUTPUT)$(LIBJVMTI): $(LIBJVMTI_IN)
-	$(QUIET_LINK)$(CC) -shared -Wl,-soname -Wl,$(LIBJVMTI) -o $@ $<
+	$(QUIET_LINK)$(CC) $(LDFLAGS) -shared -Wl,-soname -Wl,$(LIBJVMTI) -o $@ $<
 endif
 
 $(patsubst perf-%,%.o,$(PROGRAMS)): $(wildcard */*.h)
Signature · 1 line
2.17.2
maintainer Applied
#1 lore raw

Em Wed, Dec 12, 2018 at 02:29:40PM +0100, Jiri Olsa escreveu:

Quoted 1 line

So we could propagate distro flags into libperf-jvmti.so library.

Thanks, applied.

- Arnaldo

Quoted 21 lines
---
 tools/perf/Makefile.perf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
tools/perf/Makefile.perf +1 -1
index 5a90c0ef2302..6f35f4d46b0a 100644
@@ -660,7 +660,7 @@ $(LIBJVMTI_IN): FORCE
 	$(Q)$(MAKE) -f $(srctree)/tools/build/Makefile.build dir=jvmti obj=jvmti
 
 $(OUTPUT)$(LIBJVMTI): $(LIBJVMTI_IN)
-	$(QUIET_LINK)$(CC) -shared -Wl,-soname -Wl,$(LIBJVMTI) -o $@ $<
+	$(QUIET_LINK)$(CC) $(LDFLAGS) -shared -Wl,-soname -Wl,$(LIBJVMTI) -o $@ $<
 endif
 
 $(patsubst perf-%,%.o,$(PROGRAMS)): $(wildcard */*.h)
Signature · 1 line
2.17.2
Signature · 1 line
- Arnaldo
#2 lore raw
In reply to Jiri Olsa
perf tools: Link libperf-jvmti.so with LDFLAGS variable
Commit-ID:  3f643937aaab04881236b5aabec00f9802c1eda8
Gitweb:     https://git.kernel.org/tip/3f643937aaab04881236b5aabec00f9802c1eda8
Author:     Jiri Olsa <[email protected]>
AuthorDate: Wed, 12 Dec 2018 14:29:40 +0100
Committer:  Arnaldo Carvalho de Melo <[email protected]>
CommitDate: Tue, 18 Dec 2018 12:23:57 -0300

perf tools: Link libperf-jvmti.so with LDFLAGS variable

So we could propagate distro flags into libperf-jvmti.so library.

---
 tools/perf/Makefile.perf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
tools/perf/Makefile.perf +1 -1
index 22956a85e4c6..8c1350c591f2 100644
@@ -654,7 +654,7 @@ $(LIBJVMTI_IN): FORCE
 	$(Q)$(MAKE) -f $(srctree)/tools/build/Makefile.build dir=jvmti obj=jvmti
 
 $(OUTPUT)$(LIBJVMTI): $(LIBJVMTI_IN)
-	$(QUIET_LINK)$(CC) -shared -Wl,-soname -Wl,$(LIBJVMTI) -o $@ $<
+	$(QUIET_LINK)$(CC) $(LDFLAGS) -shared -Wl,-soname -Wl,$(LIBJVMTI) -o $@ $<
 endif
 
 $(patsubst perf-%,%.o,$(PROGRAMS)): $(wildcard */*.h)