Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

tools lib traceevent: Use tools build framework

Move the libtraceevent library building under tools build framework.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Tested-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Tested-by: Will Deacon <will.deacon@arm.com>
Cc: Alexis Berlemont <alexis.berlemont@gmail.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
Link: http://lkml.kernel.org/n/tip-opvx59tcawlmm916lg4aff4h@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

authored by

Jiri Olsa and committed by
Arnaldo Carvalho de Melo
2d58ab9b 285a8f24

+65 -132
+17
tools/lib/traceevent/Build
··· 1 + libtraceevent-y += event-parse.o 2 + libtraceevent-y += event-plugin.o 3 + libtraceevent-y += trace-seq.o 4 + libtraceevent-y += parse-filter.o 5 + libtraceevent-y += parse-utils.o 6 + libtraceevent-y += kbuffer-parse.o 7 + 8 + plugin_jbd2-y += plugin_jbd2.o 9 + plugin_hrtimer-y += plugin_hrtimer.o 10 + plugin_kmem-y += plugin_kmem.o 11 + plugin_kvm-y += plugin_kvm.o 12 + plugin_mac80211-y += plugin_mac80211.o 13 + plugin_sched_switch-y += plugin_sched_switch.o 14 + plugin_function-y += plugin_function.o 15 + plugin_xen-y += plugin_xen.o 16 + plugin_scsi-y += plugin_scsi.o 17 + plugin_cfg80211-y += plugin_cfg80211.o
+45 -124
tools/lib/traceevent/Makefile
··· 67 67 PLUGIN_DIR_SQ = '$(subst ','\'',$(PLUGIN_DIR))' 68 68 endif 69 69 70 - include $(if $(BUILD_SRC),$(BUILD_SRC)/)../../scripts/Makefile.include 70 + include ../../scripts/Makefile.include 71 71 72 72 # copy a bit from Linux kbuild 73 73 ··· 78 78 VERBOSE = 0 79 79 endif 80 80 81 - ifeq ("$(origin O)", "command line") 82 - BUILD_OUTPUT := $(O) 81 + ifeq ($(srctree),) 82 + srctree := $(patsubst %/,%,$(dir $(shell pwd))) 83 + srctree := $(patsubst %/,%,$(dir $(srctree))) 84 + srctree := $(patsubst %/,%,$(dir $(srctree))) 85 + #$(info Determined 'srctree' to be $(srctree)) 83 86 endif 84 - 85 - ifeq ($(BUILD_SRC),) 86 - ifneq ($(OUTPUT),) 87 - 88 - define build_output 89 - $(if $(VERBOSE:1=),@)+$(MAKE) -C $(OUTPUT) \ 90 - BUILD_SRC=$(CURDIR)/ -f $(CURDIR)/Makefile $1 91 - endef 92 - 93 - all: sub-make 94 - 95 - $(MAKECMDGOALS): sub-make 96 - 97 - sub-make: force 98 - $(call build_output, $(MAKECMDGOALS)) 99 - 100 - 101 - # Leave processing to above invocation of make 102 - skip-makefile := 1 103 - 104 - endif # OUTPUT 105 - endif # BUILD_SRC 106 - 107 - # We process the rest of the Makefile if this is the final invocation of make 108 - ifeq ($(skip-makefile),) 109 - 110 - srctree := $(if $(BUILD_SRC),$(BUILD_SRC),$(CURDIR)) 111 - objtree := $(CURDIR) 112 - src := $(srctree) 113 - obj := $(objtree) 114 87 115 88 export prefix bindir src obj 116 89 ··· 105 132 OBJ = $@ 106 133 N = 107 134 108 - export Q VERBOSE 109 - 110 135 EVENT_PARSE_VERSION = $(EP_VERSION).$(EP_PATCHLEVEL).$(EP_EXTRAVERSION) 111 136 112 - INCLUDES = -I. -I $(srctree)/../../include $(CONFIG_INCLUDES) 137 + INCLUDES = -I. -I $(srctree)/tools/include $(CONFIG_INCLUDES) 113 138 114 - # Set compile option CFLAGS if not set elsewhere 115 - CFLAGS ?= -g -Wall 139 + # Set compile option CFLAGS 140 + ifdef EXTRA_CFLAGS 141 + CFLAGS := $(EXTRA_CFLAGS) 142 + else 143 + CFLAGS := -g -Wall 144 + endif 116 145 117 146 # Append required CFLAGS 147 + override CFLAGS += -fPIC 118 148 override CFLAGS += $(CONFIG_FLAGS) $(INCLUDES) $(PLUGIN_DIR_SQ) 119 149 override CFLAGS += $(udis86-flags) -D_GNU_SOURCE 120 150 ··· 127 151 Q = @ 128 152 endif 129 153 130 - do_compile_shared_library = \ 131 - ($(print_shared_lib_compile) \ 132 - $(CC) --shared $^ -o $@) 154 + # Disable command line variables (CFLAGS) overide from top 155 + # level Makefile (perf), otherwise build Makefile will get 156 + # the same command line setup. 157 + MAKEOVERRIDES= 133 158 134 - do_plugin_build = \ 135 - ($(print_plugin_build) \ 136 - $(CC) $(CFLAGS) -shared -nostartfiles -o $@ $<) 159 + export srctree OUTPUT CC LD CFLAGS V 160 + build := -f $(srctree)/tools/build/Makefile.build dir=. obj 137 161 138 - do_build_static_lib = \ 139 - ($(print_static_lib_build) \ 140 - $(RM) $@; $(AR) rcs $@ $^) 162 + PLUGINS = plugin_jbd2.so 163 + PLUGINS += plugin_hrtimer.so 164 + PLUGINS += plugin_kmem.so 165 + PLUGINS += plugin_kvm.so 166 + PLUGINS += plugin_mac80211.so 167 + PLUGINS += plugin_sched_switch.so 168 + PLUGINS += plugin_function.so 169 + PLUGINS += plugin_xen.so 170 + PLUGINS += plugin_scsi.so 171 + PLUGINS += plugin_cfg80211.so 141 172 173 + PLUGINS := $(addprefix $(OUTPUT),$(PLUGINS)) 174 + PLUGINS_IN := $(PLUGINS:.so=-in.o) 142 175 143 - do_compile = $(QUIET_CC)$(CC) -c $(CFLAGS) $(EXT) $< -o $(obj)/$@; 144 - 145 - $(obj)/%.o: $(src)/%.c 146 - $(call do_compile) 147 - 148 - %.o: $(src)/%.c 149 - $(call do_compile) 150 - 151 - PEVENT_LIB_OBJS = event-parse.o 152 - PEVENT_LIB_OBJS += event-plugin.o 153 - PEVENT_LIB_OBJS += trace-seq.o 154 - PEVENT_LIB_OBJS += parse-filter.o 155 - PEVENT_LIB_OBJS += parse-utils.o 156 - PEVENT_LIB_OBJS += kbuffer-parse.o 157 - 158 - PLUGIN_OBJS = plugin_jbd2.o 159 - PLUGIN_OBJS += plugin_hrtimer.o 160 - PLUGIN_OBJS += plugin_kmem.o 161 - PLUGIN_OBJS += plugin_kvm.o 162 - PLUGIN_OBJS += plugin_mac80211.o 163 - PLUGIN_OBJS += plugin_sched_switch.o 164 - PLUGIN_OBJS += plugin_function.o 165 - PLUGIN_OBJS += plugin_xen.o 166 - PLUGIN_OBJS += plugin_scsi.o 167 - PLUGIN_OBJS += plugin_cfg80211.o 168 - 169 - PLUGINS := $(PLUGIN_OBJS:.o=.so) 170 - 171 - ALL_OBJS = $(PEVENT_LIB_OBJS) $(PLUGIN_OBJS) 176 + TE_IN := $(OUTPUT)libtraceevent-in.o 177 + LIB_FILE := $(addprefix $(OUTPUT),$(LIB_FILE)) 172 178 173 179 CMD_TARGETS = $(LIB_FILE) $(PLUGINS) 174 180 175 181 TARGETS = $(CMD_TARGETS) 176 182 177 - 178 183 all: all_cmd 179 184 180 185 all_cmd: $(CMD_TARGETS) 181 186 182 - libtraceevent.so: $(PEVENT_LIB_OBJS) 187 + $(TE_IN): force 188 + $(Q)$(MAKE) $(build)=libtraceevent 189 + 190 + $(OUTPUT)libtraceevent.so: $(TE_IN) 183 191 $(QUIET_LINK)$(CC) --shared $^ -o $@ 184 192 185 - libtraceevent.a: $(PEVENT_LIB_OBJS) 193 + $(OUTPUT)libtraceevent.a: $(TE_IN) 186 194 $(QUIET_LINK)$(RM) $@; $(AR) rcs $@ $^ 187 195 188 196 plugins: $(PLUGINS) 189 197 190 - $(PEVENT_LIB_OBJS): %.o: $(src)/%.c TRACEEVENT-CFLAGS 191 - $(QUIET_CC_FPIC)$(CC) -c $(CFLAGS) $(EXT) -fPIC $< -o $@ 198 + __plugin_obj = $(notdir $@) 199 + plugin_obj = $(__plugin_obj:-in.o=) 192 200 193 - $(PLUGIN_OBJS): %.o : $(src)/%.c 194 - $(QUIET_CC_FPIC)$(CC) -c $(CFLAGS) -fPIC -o $@ $< 201 + $(PLUGINS_IN): force 202 + $(Q)$(MAKE) $(build)=$(plugin_obj) 195 203 196 - $(PLUGINS): %.so: %.o 197 - $(QUIET_LINK)$(CC) $(CFLAGS) -shared -nostartfiles -o $@ $< 204 + $(OUTPUT)%.so: $(OUTPUT)%-in.o 205 + $(QUIET_LINK)$(CC) $(CFLAGS) -shared -nostartfiles -o $@ $^ 198 206 199 207 define make_version.h 200 208 (echo '/* This file is automatically generated. Do not modify. */'; \ ··· 214 254 mv -f $@.tmp $@; \ 215 255 fi); 216 256 endef 217 - 218 - ## make deps 219 - 220 - all_objs := $(sort $(ALL_OBJS)) 221 - all_deps := $(all_objs:%.o=.%.d) 222 - 223 - # let .d file also depends on the source and header files 224 - define check_deps 225 - @set -e; $(RM) $@; \ 226 - $(CC) -MM $(CFLAGS) $< > $@.$$$$; \ 227 - sed 's,\($*\)\.o[ :]*,\1.o $@ : ,g' < $@.$$$$ > $@; \ 228 - $(RM) $@.$$$$ 229 - endef 230 - 231 - $(all_deps): .%.d: $(src)/%.c 232 - $(Q)$(call check_deps) 233 - 234 - $(all_objs) : %.o : .%.d 235 - 236 - dep_includes := $(wildcard $(all_deps)) 237 - 238 - ifneq ($(dep_includes),) 239 - include $(dep_includes) 240 - endif 241 - 242 - ### Detect environment changes 243 - TRACK_CFLAGS = $(subst ','\'',$(CFLAGS)):$(ARCH):$(CROSS_COMPILE) 244 - 245 - TRACEEVENT-CFLAGS: force 246 - @FLAGS='$(TRACK_CFLAGS)'; \ 247 - if test x"$$FLAGS" != x"`cat TRACEEVENT-CFLAGS 2>/dev/null`" ; then \ 248 - echo 1>&2 " FLAGS: * new build flags or cross compiler"; \ 249 - echo "$$FLAGS" >TRACEEVENT-CFLAGS; \ 250 - fi 251 257 252 258 tags: force 253 259 $(RM) tags ··· 253 327 $(RM) *.o *~ $(TARGETS) *.a *.so $(VERSION_FILES) .*.d \ 254 328 $(RM) TRACEEVENT-CFLAGS tags TAGS 255 329 256 - endif # skip-makefile 257 - 258 330 PHONY += force plugins 259 331 force: 260 - 261 - plugins: 262 - @echo > /dev/null 263 332 264 333 # Declare the contents of the .PHONY variable as phony. We keep that 265 334 # information in a variable so we can use it in if_changed and friends.
+3 -8
tools/perf/Makefile.perf
··· 358 358 $(LIB_FILE): $(LIBPERF_IN) 359 359 $(QUIET_AR)$(RM) $@ && $(AR) rcs $@ $(LIBPERF_IN) $(LIB_OBJS) 360 360 361 - # libtraceevent.a 362 - TE_SOURCES = $(wildcard $(TRACE_EVENT_DIR)*.[ch]) 363 - 364 - LIBTRACEEVENT_FLAGS = $(QUIET_SUBDIR1) O=$(OUTPUT) 365 - LIBTRACEEVENT_FLAGS += CFLAGS="-g -Wall $(EXTRA_CFLAGS)" 366 361 LIBTRACEEVENT_FLAGS += plugin_dir=$(plugindir_SQ) 367 362 368 - $(LIBTRACEEVENT): $(TE_SOURCES) 369 - $(QUIET_SUBDIR0)$(TRACE_EVENT_DIR) $(LIBTRACEEVENT_FLAGS) libtraceevent.a plugins 363 + $(LIBTRACEEVENT): FORCE 364 + @$(MAKE) -C $(TRACE_EVENT_DIR) $(LIBTRACEEVENT_FLAGS) O=$(OUTPUT) $(OUTPUT)libtraceevent.a plugins 370 365 371 366 $(LIBTRACEEVENT)-clean: 372 367 $(call QUIET_CLEAN, libtraceevent) 373 368 @$(MAKE) -C $(TRACE_EVENT_DIR) O=$(OUTPUT) clean >/dev/null 374 369 375 370 install-traceevent-plugins: $(LIBTRACEEVENT) 376 - $(QUIET_SUBDIR0)$(TRACE_EVENT_DIR) $(LIBTRACEEVENT_FLAGS) install_plugins 371 + @$(MAKE) -C $(TRACE_EVENT_DIR) $(LIBTRACEEVENT_FLAGS) O=$(OUTPUT) install_plugins 377 372 378 373 $(LIBAPI): FORCE 379 374 @$(MAKE) -C $(LIB_DIR) O=$(OUTPUT) $(OUTPUT)libapi.a