linux-kernel - Re: [PATCH] kbuild: Add vmlinux_install to facilitate debugging

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <f1ad7092-af82-459b-a5a1-863f6254decc@gmail.com>
Date: Tue, 8 Oct 2024 00:13:54 +0800
From: Zach Wade <zachwade.k@...il.com>
To: masahiroy@...nel.org
Cc: nathan@...nel.org, nicolas@...sle.eu, linux-kbuild@...r.kernel.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH] kbuild: Add vmlinux_install to facilitate debugging



On 2024/10/8 0:07, Zach Wade wrote:
> When testing multiple versions of the kernel with the same source code,it
> is often necessary to recompile the kernel, which is time-consuming for
> small hosts. I need to cp vmlinux to the corresponding module directory.
> I think adding this will make debugging the kernel a little more
> convenient.
> 
> Signed-off-by: Zach Wade <zachwade.k@...il.com>
> ---
>   Makefile | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
> 
> diff --git a/Makefile b/Makefile
> index c5493c0c0ca1..1caab011599f 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1579,6 +1579,7 @@ help:
>   	@echo  '* vmlinux	  - Build the bare kernel'
>   	@echo  '* modules	  - Build all modules'
>   	@echo  '  modules_install - Install all modules to INSTALL_MOD_PATH (default: /)'
> +	@echo  '  vmlinux_install - Install vmlinux to INSTALL_MOD_PATH (default: /)'
>   	@echo  '  vdso_install    - Install unstripped vdso to INSTALL_MOD_PATH (default: /)'
>   	@echo  '  dir/            - Build all files in dir and below'
>   	@echo  '  dir/file.[ois]  - Build specified target only'
> @@ -1887,6 +1888,19 @@ modpost: $(if $(single-build),, $(if $(KBUILD_BUILTIN), vmlinux.o)) \
>   	 $(if $(KBUILD_MODULES), modules_check)
>   	$(Q)$(MAKE) -f $(srctree)/scripts/Makefile.modpost
>   
> +# ---------------------------------------------------------------------------
> +# vmlinux install
> +
> +PHONY += vmlinux_install
> +
> +vmlinux_install:
> +	@if [ -f vmlinux ]; then \
> +		echo "INSTALL ${MODLIB}/vmlinux"; \
> +		cp -f vmlinux ${MODLIB}/ ; \
> +	else \
> +		echo "vmlinux file does not exist."; \
> +	fi
> +
>   # Single targets
>   # ---------------------------------------------------------------------------
>   # To build individual files in subdirectories, you can do like this:

Hi Masahiro Yamada,

I think this patch will make me feel comfortable, so I submitted this 
change.
However, this submission may not be perfect. I just put forward my 
ideas. If you think it can be added, you are welcome to point out the 
parts that need to be modified. I will fix them as required and resubmit 
the v2 version.

Thanks,
Zach