diff mbox series

kbuild: take vmlinux.[ao] out of single-targets

Message ID 20240115001606.15477-1-richard.weiyang@gmail.com
State New
Headers
Series kbuild: take vmlinux.[ao] out of single-targets |

Commit Message

Wei Yang Jan. 15, 2024, 12:16 a.m. UTC 
  For current kernel, when we make vmlinux.a or vmlinux.o, following
message would display.

$make vmlinux.o
/dir/to/kernel/Makefile:1887: warning: overriding recipe for target 'vmlinux.o'
/dir/to/kernel/Makefile:1138: warning: ignoring old recipe for target 'vmlinux.o'
  CALL    scripts/checksyscalls.sh
  DESCEND objtool
  INSTALL libsubcmd_headers
make[2]: Nothing to be done for 'vmlinux.o'.

The reason is vmlinux.[ao] is treated as single target, while the rule
is written in root Makefile.

This patch fixes this by take them out of single-targets.

Signed-off-by: Wei Yang <richard.weiyang@gmail.com>
CC: Masahiro Yamada <masahiroy@kernel.org>
CC: Miguel Ojeda <ojeda@kernel.org>
CC: Nathan Chancellor <nathan@kernel.org>
---
 Makefile | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Masahiro Yamada Jan. 16, 2024, 11 a.m. UTC | #1 
On Mon, Jan 15, 2024 at 9:17 AM Wei Yang <richard.weiyang@gmail.com> wrote:
>
> For current kernel, when we make vmlinux.a or vmlinux.o, following
> message would display.
>
> $make vmlinux.o
> /dir/to/kernel/Makefile:1887: warning: overriding recipe for target 'vmlinux.o'
> /dir/to/kernel/Makefile:1138: warning: ignoring old recipe for target 'vmlinux.o'
>   CALL    scripts/checksyscalls.sh
>   DESCEND objtool
>   INSTALL libsubcmd_headers
> make[2]: Nothing to be done for 'vmlinux.o'.
>
> The reason is vmlinux.[ao] is treated as single target, while the rule
> is written in root Makefile.
>
> This patch fixes this by take them out of single-targets.
>
> Signed-off-by: Wei Yang <richard.weiyang@gmail.com>
> CC: Masahiro Yamada <masahiroy@kernel.org>
> CC: Miguel Ojeda <ojeda@kernel.org>
> CC: Nathan Chancellor <nathan@kernel.org>
> ---


Not all targets can be built by the single-target.

Just do not do "make vmlinux.o".









>  Makefile | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index f1b2fd977275..66fb08f6d971 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -280,6 +280,7 @@ no-dot-config-targets := $(clean-targets) \
>  no-sync-config-targets := $(no-dot-config-targets) %install modules_sign kernelrelease \
>                           image_name
>  single-targets := %.a %.i %.ko %.lds %.ll %.lst %.mod %.o %.rsi %.s %.symtypes %/
> +no-single-targets := vmlinux.o vmlinux.a
>
>  config-build   :=
>  mixed-build    :=
> @@ -315,11 +316,14 @@ ifeq ($(KBUILD_EXTMOD),)
>  endif
>
>  # We cannot build single targets and the others at the same time
> -ifneq ($(filter $(single-targets), $(MAKECMDGOALS)),)
> +ifneq ($(filter-out $(no-single-targets), $(filter $(single-targets), $(MAKECMDGOALS))),)
>         single-build := 1
>         ifneq ($(filter-out $(single-targets), $(MAKECMDGOALS)),)
>                 mixed-build := 1
>         endif
> +       ifneq ($(filter $(no-single-targets), $(MAKECMDGOALS)),)
> +               mixed-build := 1
> +       endif
>  endif
>
>  # For "make -j clean all", "make -j mrproper defconfig all", etc.
> --
> 2.34.1
>
diff mbox series

Patch

diff --git a/Makefile b/Makefile
index f1b2fd977275..66fb08f6d971 100644
--- a/Makefile
+++ b/Makefile
@@ -280,6 +280,7 @@  no-dot-config-targets := $(clean-targets) \
 no-sync-config-targets := $(no-dot-config-targets) %install modules_sign kernelrelease \
 			  image_name
 single-targets := %.a %.i %.ko %.lds %.ll %.lst %.mod %.o %.rsi %.s %.symtypes %/
+no-single-targets := vmlinux.o vmlinux.a
 
 config-build	:=
 mixed-build	:=
@@ -315,11 +316,14 @@  ifeq ($(KBUILD_EXTMOD),)
 endif
 
 # We cannot build single targets and the others at the same time
-ifneq ($(filter $(single-targets), $(MAKECMDGOALS)),)
+ifneq ($(filter-out $(no-single-targets), $(filter $(single-targets), $(MAKECMDGOALS))),)
 	single-build := 1
 	ifneq ($(filter-out $(single-targets), $(MAKECMDGOALS)),)
 		mixed-build := 1
 	endif
+	ifneq ($(filter $(no-single-targets), $(MAKECMDGOALS)),)
+		mixed-build := 1
+	endif
 endif
 
 # For "make -j clean all", "make -j mrproper defconfig all", etc.