Hi Yamada-san Thank you for the patch, On 19/02/2019 09:33, Masahiro Yamada wrote: > scripts/gdb/linux/constants.py is never used in the kernel build > process. There is no good reason to create it so early. > I agree, I originally put it here to ensure it was only built after bounds.h (following the asm-offsets.h example I expect). It has no requirement to be built in this stage itself. Depending on the prepare stage instead sounds a lot more reasonable. Thanks Reviewed-by: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx> > Get it out of the 'prepare' stage. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> > --- > > 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 > > ### > -- Regards -- Kieran
