On Tue, Feb 19, 2019 at 6:34 PM Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> wrote: > > scripts/gdb/linux/constants.py is never used in the kernel build > process. There is no good reason to create it so early. > > Get it out of the 'prepare' stage. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> > --- Added Kieran's Reviewed-by, and applied to linux-kbuild. > > Kbuild | 10 ---------- > Makefile | 11 +++++++++++ > 2 files changed, 11 insertions(+), 10 deletions(-) > > diff --git a/Kbuild b/Kbuild > index 65db5be..4cebcc7 100644 > --- a/Kbuild > +++ b/Kbuild > @@ -6,7 +6,6 @@ > # 2) Generate timeconst.h > # 3) Generate asm-offsets.h (may need bounds.h and timeconst.h) > # 4) Check for missing system calls > -# 5) Generate constants.py (may need bounds.h) > > ##### > # 1) Generate bounds.h > @@ -58,14 +57,5 @@ quiet_cmd_syscalls = CALL $< > missing-syscalls: scripts/checksyscalls.sh $(offsets-file) FORCE > $(call cmd,syscalls) > > -##### > -# 5) Generate constants for Python GDB integration > -# > - > -extra-$(CONFIG_GDB_SCRIPTS) += build_constants_py > - > -build_constants_py: $(timeconst-file) $(bounds-file) > - @$(MAKE) $(build)=scripts/gdb/linux $@ > - > # Keep these three files during make clean > no-clean-files := $(bounds-file) $(offsets-file) $(timeconst-file) > diff --git a/Makefile b/Makefile > index 88db36b..26dbcb7 100644 > --- a/Makefile > +++ b/Makefile > @@ -1513,6 +1513,17 @@ PHONY += $(DOC_TARGETS) > $(DOC_TARGETS): scripts_basic FORCE > $(Q)$(MAKE) $(build)=Documentation $@ > > +# Misc > +# --------------------------------------------------------------------------- > + > +PHONY += scripts_gdb > +scripts_gdb: prepare > + $(Q)$(MAKE) $(build)=scripts/gdb/linux build_constants_py > + > +ifdef CONFIG_GDB_SCRIPTS > +all: scripts_gdb > +endif > + > else # KBUILD_EXTMOD > > ### > -- > 2.7.4 > -- Best Regards Masahiro Yamada