@@ -412,18 +412,18 @@ _debug:
.PHONY: _clean
_clean:
- $(MAKE) $(clean) tools
- $(MAKE) $(clean) include
- $(MAKE) $(clean) common
- $(MAKE) $(clean) drivers
- $(MAKE) $(clean) lib
- $(MAKE) $(clean) xsm
- $(MAKE) $(clean) crypto
- $(MAKE) $(clean) arch/arm
- $(MAKE) $(clean) arch/riscv
- $(MAKE) $(clean) arch/x86
- $(MAKE) $(clean) test
- $(MAKE) $(clean) tools/kconfig
+ $(Q)$(MAKE) $(clean)=tools
+ $(Q)$(MAKE) $(clean)=include
+ $(Q)$(MAKE) $(clean)=common
+ $(Q)$(MAKE) $(clean)=drivers
+ $(Q)$(MAKE) $(clean)=lib
+ $(Q)$(MAKE) $(clean)=xsm
+ $(Q)$(MAKE) $(clean)=crypto
+ $(Q)$(MAKE) $(clean)=arch/arm
+ $(Q)$(MAKE) $(clean)=arch/riscv
+ $(Q)$(MAKE) $(clean)=arch/x86
+ $(Q)$(MAKE) $(clean)=test
+ $(Q)$(MAKE) $(clean)=tools/kconfig
find . \( -name "*.o" -o -name ".*.d" -o -name ".*.d2" \
-o -name ".*.o.tmp" -o -name "*~" -o -name "core" \
-o -name '*.lex.c' -o -name '*.tab.[ch]' \
@@ -116,7 +116,4 @@ $(obj)/xen.lds: $(src)/xen.lds.S FORCE
$(obj)/dtb.o: $(patsubst "%",%,$(CONFIG_DTB_FILE))
-.PHONY: clean
-clean::
- rm -f $(obj)/xen.lds
- rm -f $(BASEDIR)/.xen-syms.[0-9]*
+clean-files := $(objtree)/.xen-syms.[0-9]*
@@ -77,6 +77,9 @@ obj-$(CONFIG_COMPAT) += x86_64/platform_hypercall.o
obj-y += sysctl.o
endif
+# Allows "clean" to descend into boot/
+subdir- += boot
+
extra-y += asm-macros.i
extra-y += xen.lds
@@ -190,8 +193,8 @@ note_file :=
endif
note_file_option ?= $(note_file)
+extra-$(XEN_BUILD_PE) += efi.lds
ifeq ($(XEN_BUILD_PE),y)
-extra-y += efi.lds
$(TARGET).efi: $(BASEDIR)/prelink.o $(note_file) $(obj)/efi.lds $(obj)/efi/relocs-dummy.o $(obj)/efi/mkreloc
ifeq ($(CONFIG_DEBUG_INFO),y)
$(if $(filter --strip-debug,$(EFI_LDFLAGS)),echo,:) "Will strip debug info from $(@F)"
@@ -256,10 +259,11 @@ $(obj)/boot/mkelf32: $(src)/boot/mkelf32.c
$(obj)/efi/mkreloc: $(src)/efi/mkreloc.c
$(HOSTCC) $(HOSTCFLAGS) -g -o $@ $<
-.PHONY: clean
-clean::
- rm -f *.lds boot/mkelf32
- rm -f asm-macros.i $(BASEDIR)/arch/x86/include/asm/asm-macros.*
- rm -f $(BASEDIR)/.xen-syms.[0-9]* $(BASEDIR)/.xen.elf32
- rm -f $(BASEDIR)/.xen.efi.[0-9]* efi/*.efi efi/mkreloc
- rm -f boot/cmdline.S boot/reloc.S boot/*.lnk boot/*.bin
+clean-files := \
+ boot/mkelf32 \
+ include/asm/asm-macros.* \
+ $(objtree)/.xen-syms.[0-9]* \
+ $(objtree)/.xen.elf32 \
+ $(objtree)/.xen.efi.[0-9]* \
+ efi/*.efi \
+ efi/mkreloc
@@ -21,3 +21,5 @@ $(obj)/cmdline.S: $(src)/cmdline.c $(CMDLINE_DEPS) $(src)/build32.lds
$(obj)/reloc.S: $(src)/reloc.c $(RELOC_DEPS) $(src)/build32.lds
$(MAKE) -f $(abs_srctree)/$(src)/build32.mk -C $(obj) $(@F) RELOC_DEPS="$(RELOC_DEPS)"
+
+clean-files := cmdline.S reloc.S *.lnk *.bin
@@ -84,5 +84,4 @@ $(obj)/config_data.S: $(BASEDIR)/tools/binfile FORCE
$(call if_changed,binfile,$(obj)/config.gz xen_config_data)
targets += config_data.S
-clean::
- rm -f config_data.S config.gz 2>/dev/null
+clean-files := config.gz
@@ -131,6 +131,4 @@ lib-x86-all:
all: lib-x86-all
endif
-clean::
- rm -rf compat config generated headers*.chk
- rm -f $(BASEDIR)/include/xen/lib/x86/cpuid-autogen.h
+clean-files := compat config generated headers*.chk xen/lib/x86/cpuid-autogen.h
@@ -99,8 +99,8 @@ build := -f $(srctree)/Rules.mk obj
# Shorthand for $(MAKE) clean
# Usage:
-# $(MAKE) $(clean) dir
-clean := -f $(BASEDIR)/scripts/Makefile.clean clean -C
+# $(Q)$(MAKE) $(clean)=dir
+clean := -f $(srctree)/scripts/Makefile.clean obj
# echo command.
# Short version is used, if $(quiet) equals `quiet_', otherwise full one.
@@ -3,7 +3,6 @@
# Cleaning up
# ==========================================================================
-obj := .
src := $(obj)
clean::
@@ -17,11 +16,20 @@ include $(src)/Makefile
subdir-all := $(subdir-y) $(subdir-n) $(subdir-) \
$(patsubst %/,%, $(filter %/, $(obj-y) $(obj-n) $(obj-)))
+subdir-all := $(addprefix $(obj)/,$(subdir-all))
+
__clean-files := \
$(clean-files) $(hostprogs-y) $(hostprogs-) \
+ $(extra-y) $(extra-) $(targets) \
$(hostprogs-always-y) $(hostprogs-always-)
-__clean-files := $(wildcard $(__clean-files))
+# clean-files is given relative to the current directory, unless it
+# starts with $(objtree)/ (which means "./", so do not add "./" unless
+# you want to delete a file from the toplevel object directory).
+
+__clean-files := $(wildcard \
+ $(addprefix $(obj)/, $(filter-out /% $(objtree)/%, $(__clean-files))) \
+ $(filter /% $(objtree)/%, $(__clean-files)))
.PHONY: clean
clean:: $(subdir-all)
@@ -35,6 +43,6 @@ endif
PHONY += $(subdir-all)
$(subdir-all):
- $(MAKE) $(clean) $@
+ $(Q)$(MAKE) $(clean)=$@
.PHONY: $(PHONY)
@@ -160,6 +160,4 @@ install: $(addprefix $(obj)/,$(LIVEPATCHES))
uninstall:
cd $(DESTDIR)$(LIVEPATCH_DEBUG_DIR) && rm -f $(LIVEPATCHES)
-.PHONY: clean
-clean::
- rm -f *.livepatch config.h expect_config.h
+clean-files := config.h expect_config.h
@@ -49,6 +49,4 @@ $(obj)/policy.bin: FORCE
FLASK_BUILD_DIR=$(FLASK_BUILD_DIR) POLICY_FILENAME=$(POLICY_SRC)
cmp -s $(POLICY_SRC) $@ || cp $(POLICY_SRC) $@
-.PHONY: clean
-clean::
- rm -f $(ALL_H_FILES) policy.* $(POLICY_SRC) flask-policy.S
+clean-files := policy.* $(POLICY_SRC)
This will allow "clean" to work from an out-of-tree build when it will be available. Some of the file been removed in current "clean" target aren't added to $(clean-files) because they are already listed in $(extra-) or $(extra-y). Also start to clean files listed in $(targets). This allows to clean "common/config_data.S" and "xsm/flask/flask-policy.S" without having to list them a second time. Also clean files in "arch/x86/boot" from that directory by allowing "clean" to descend into the subdir by adding "boot" into $(subdir-). Signed-off-by: Anthony PERARD <anthony.perard@citrix.com> --- Notes: v8: - fix comment for the shorthand - move this patch earlier in the series - replace some BASEDIR by objtree - start to clean $(targets) files. xen/Makefile | 24 ++++++++++++------------ xen/arch/arm/Makefile | 5 +---- xen/arch/x86/Makefile | 20 ++++++++++++-------- xen/arch/x86/boot/Makefile | 2 ++ xen/common/Makefile | 3 +-- xen/include/Makefile | 4 +--- xen/scripts/Kbuild.include | 4 ++-- xen/scripts/Makefile.clean | 14 +++++++++++--- xen/test/livepatch/Makefile | 4 +--- xen/xsm/flask/Makefile | 4 +--- 10 files changed, 44 insertions(+), 40 deletions(-)