Commit 32fa0efa authored by Andrii Nakryiko's avatar Andrii Nakryiko
Browse files

Merge branch 'fixes for bpftool's Makefile' 



Quentin Monnet says:

====================

This set contains one fix for bpftool's Makefile, to make sure that the
headers internal to libbpf are installed properly even if we add more
headers to the relevant Makefile variable in the future (although we'd like
to avoid that if possible).

The other patches aim at cleaning up the output from the Makefile, in
particular when running the command "make" another time after bpftool is
built.
====================

Signed-off-by: default avatarAndrii Nakryiko <andrii@kernel.org>
parents d51b6b22 062e1fc0

tools/bpf/bpftool/Makefile

+14 −14
Original line number Diff line number Diff line
@@ -14,7 +14,7 @@ else 
  Q = @

endif



BPF_DIR = $(srctree)/tools/lib/bpf/

BPF_DIR = $(srctree)/tools/lib/bpf



ifneq ($(OUTPUT),)

  _OUTPUT := $(OUTPUT)
@@ -25,6 +25,7 @@ BOOTSTRAP_OUTPUT := $(_OUTPUT)/bootstrap/ 
LIBBPF_OUTPUT := $(_OUTPUT)/libbpf/

LIBBPF_DESTDIR := $(LIBBPF_OUTPUT)

LIBBPF_INCLUDE := $(LIBBPF_DESTDIR)/include

LIBBPF_HDRS_DIR := $(LIBBPF_INCLUDE)/bpf



LIBBPF = $(LIBBPF_OUTPUT)libbpf.a

LIBBPF_BOOTSTRAP_OUTPUT = $(BOOTSTRAP_OUTPUT)libbpf/
@@ -32,7 +33,7 @@ LIBBPF_BOOTSTRAP = $(LIBBPF_BOOTSTRAP_OUTPUT)libbpf.a 


# We need to copy nlattr.h which is not otherwise exported by libbpf, but still

# required by bpftool.

LIBBPF_INTERNAL_HDRS := nlattr.h

LIBBPF_INTERNAL_HDRS := $(addprefix $(LIBBPF_HDRS_DIR)/,nlattr.h)



ifeq ($(BPFTOOL_VERSION),)

BPFTOOL_VERSION := $(shell make -rR --no-print-directory -sC ../../.. kernelversion)
@@ -41,16 +42,15 @@ endif 
$(LIBBPF_OUTPUT) $(BOOTSTRAP_OUTPUT) $(LIBBPF_BOOTSTRAP_OUTPUT):

	$(QUIET_MKDIR)mkdir -p $@



$(LIBBPF): FORCE | $(LIBBPF_OUTPUT)

$(LIBBPF): $(wildcard $(BPF_DIR)/*.[ch] $(BPF_DIR)/Makefile) | $(LIBBPF_OUTPUT)

	$(Q)$(MAKE) -C $(BPF_DIR) OUTPUT=$(LIBBPF_OUTPUT) \

		DESTDIR=$(LIBBPF_DESTDIR) prefix= $(LIBBPF) install_headers



$(LIBBPF_INCLUDE)/bpf/$(LIBBPF_INTERNAL_HDRS): \

		$(addprefix $(BPF_DIR),$(LIBBPF_INTERNAL_HDRS)) $(LIBBPF)

	$(call QUIET_INSTALL, bpf/$(notdir $@))

	$(Q)install -m 644 -t $(LIBBPF_INCLUDE)/bpf/ $(BPF_DIR)$(notdir $@)

$(LIBBPF_INTERNAL_HDRS): $(LIBBPF_HDRS_DIR)/%.h: $(BPF_DIR)/%.h $(LIBBPF)

	$(call QUIET_INSTALL, $@)

	$(Q)install -m 644 -t $(LIBBPF_HDRS_DIR) $<



$(LIBBPF_BOOTSTRAP): FORCE | $(LIBBPF_BOOTSTRAP_OUTPUT)

$(LIBBPF_BOOTSTRAP): $(wildcard $(BPF_DIR)/*.[ch] $(BPF_DIR)/Makefile) | $(LIBBPF_BOOTSTRAP_OUTPUT)

	$(Q)$(MAKE) -C $(BPF_DIR) OUTPUT=$(LIBBPF_BOOTSTRAP_OUTPUT) \

		ARCH= CC=$(HOSTCC) LD=$(HOSTLD) $@
@@ -150,7 +150,7 @@ BOOTSTRAP_OBJS = $(addprefix $(BOOTSTRAP_OUTPUT),main.o common.o json_writer.o g 
$(BOOTSTRAP_OBJS): $(LIBBPF_BOOTSTRAP)



OBJS = $(patsubst %.c,$(OUTPUT)%.o,$(SRCS)) $(OUTPUT)disasm.o

$(OBJS): $(LIBBPF) $(LIBBPF_INCLUDE)/bpf/$(LIBBPF_INTERNAL_HDRS)

$(OBJS): $(LIBBPF) $(LIBBPF_INTERNAL_HDRS)



VMLINUX_BTF_PATHS ?= $(if $(O),$(O)/vmlinux)				\

		     $(if $(KBUILD_OUTPUT),$(KBUILD_OUTPUT)/vmlinux)	\
@@ -198,7 +198,10 @@ $(BOOTSTRAP_OUTPUT)disasm.o: $(srctree)/kernel/bpf/disasm.c 
$(OUTPUT)disasm.o: $(srctree)/kernel/bpf/disasm.c

	$(QUIET_CC)$(CC) $(CFLAGS) -c -MMD -o $@ $<



$(OUTPUT)feature.o: | zdep

$(OUTPUT)feature.o:

ifneq ($(feature-zlib), 1)

	$(error "No zlib found")

endif



$(BPFTOOL_BOOTSTRAP): $(BOOTSTRAP_OBJS) $(LIBBPF_BOOTSTRAP)

	$(QUIET_LINK)$(HOSTCC) $(CFLAGS) $(LDFLAGS) -o $@ $(BOOTSTRAP_OBJS) \
@@ -254,10 +257,7 @@ doc-uninstall: 


FORCE:



zdep:

	@if [ "$(feature-zlib)" != "1" ]; then echo "No zlib found"; exit 1 ; fi



.SECONDARY:

.PHONY: all FORCE clean install-bin install uninstall zdep

.PHONY: all FORCE clean install-bin install uninstall

.PHONY: doc doc-clean doc-install doc-uninstall

.DEFAULT_GOAL := all