linux-kernel - Re: [PATCH] ARM: Use an absolute path to unified.h in KBUILD

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: <CAK7LNAREwF0Yn6R80+1WnrJpHe4vXcjMdqUqqEU=cL1++Yk9SA@mail.gmail.com>
Date: Fri, 20 Jun 2025 11:31:58 +0900
From: Masahiro Yamada <masahiroy@...nel.org>
To: Nathan Chancellor <nathan@...nel.org>
Cc: Russell King <linux@...linux.org.uk>, linux-arm-kernel@...ts.infradead.org, 
	linux-kernel@...r.kernel.org, linux-kbuild@...r.kernel.org, 
	stable@...r.kernel.org, KernelCI bot <bot@...nelci.org>
Subject: Re: [PATCH] ARM: Use an absolute path to unified.h in KBUILD_AFLAGS

On Thu, Jun 19, 2025 at 4:29 AM Nathan Chancellor <nathan@...nel.org> wrote:
>
> After commit d5c8d6e0fa61 ("kbuild: Update assembler calls to use proper
> flags and language target"), which updated as-instr to use the
> 'assembler-with-cpp' language option, the Kbuild version of as-instr
> always fails internally for arch/arm with
>
>   <command-line>: fatal error: asm/unified.h: No such file or directory
>   compilation terminated.
>
> because '-include' flags are now taken into account by the compiler
> driver and as-instr does not have '$(LINUXINCLUDE)', so unified.h is not
> found.
>
> This went unnoticed at the time of the Kbuild change because the last
> use of as-instr in Kbuild that arch/arm could reach was removed in 5.7
> by commit 541ad0150ca4 ("arm: Remove 32bit KVM host support") but a
> stable backport of the Kbuild change to before that point exposed this
> potential issue if one were to be reintroduced.
>
> Follow the general pattern of '-include' paths throughout the tree and
> make unified.h absolute using '$(srctree)' to ensure KBUILD_AFLAGS can
> be used independently.
>
> Cc: stable@...r.kernel.org
> Fixes: d5c8d6e0fa61 ("kbuild: Update assembler calls to use proper flags and language target")
> Reported-by: KernelCI bot <bot@...nelci.org>
> Closes: https://lore.kernel.org/CACo-S-1qbCX4WAVFA63dWfHtrRHZBTyyr2js8Lx=Az03XHTTHg@mail.gmail.com/
> Signed-off-by: Nathan Chancellor <nathan@...nel.org>


Reviewed-by: Masahiro Yamada <masahiroy@...nel.org>





-- 
Best Regards
Masahiro Yamada