[PATCH] kbuild: improve sanity check for ARCH variable on x86

[PATCH] kbuild: improve sanity check for ARCH variable on x86

[Cao Jin]

When building for x86, ARCH=i{4,5,6}86 and x86_32 will fail the build.
Make it catch all name variants.
---
 Makefile | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/Makefile b/Makefile
index 7c855e6fe544..cd397027891a 100644
--- a/Makefile
+++ b/Makefile
@@ -409,12 +409,10 @@ ARCH		?= $(SUBARCH)
 UTS_MACHINE 	:= $(ARCH)
 SRCARCH 	:= $(ARCH)
 
-# Additional ARCH settings for x86
-ifeq ($(ARCH),i386)
-        SRCARCH := x86
-endif
-ifeq ($(ARCH),x86_64)
-        SRCARCH := x86
+# Additional sanity check for cmdline ARCH settings for x86. This catch
+# more name variants than "i386" and "x86_64".
+ifneq ($(findstring 86,$(ARCH)),)
+    SRCARCH := x86
 endif
 
 # Additional ARCH settings for sparc
-- 
2.55.0

Re: [PATCH] kbuild: improve sanity check for ARCH variable on x86

[Nathan Chancellor]

On Mon, Oct 19, 2026 at 05:17:38PM -0700, Cao Jin wrote:
> When building for x86, ARCH=i{4,5,6}86 and x86_32 will fail the build.

Thanks for the patch. Where is it documented that these other options
are valid? It seems like this is just expected behavior because these
are not valid values for setting ARCH for x86. At the very least, I
think something like this needs to be acked by the x86 folks.

> Make it catch all name variants.

You are missing a sign off.

> ---
>  Makefile | 10 ++++------
>  1 file changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/Makefile b/Makefile
> index 7c855e6fe544..cd397027891a 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -409,12 +409,10 @@ ARCH		?= $(SUBARCH)
>  UTS_MACHINE 	:= $(ARCH)
>  SRCARCH 	:= $(ARCH)
>  
> -# Additional ARCH settings for x86
> -ifeq ($(ARCH),i386)
> -        SRCARCH := x86
> -endif
> -ifeq ($(ARCH),x86_64)
> -        SRCARCH := x86
> +# Additional sanity check for cmdline ARCH settings for x86. This catch
> +# more name variants than "i386" and "x86_64".
> +ifneq ($(findstring 86,$(ARCH)),)
> +    SRCARCH := x86
>  endif
>  
>  # Additional ARCH settings for sparc
> -- 
> 2.55.0
> 

-- 
Cheers,
Nathan

Re: [PATCH] kbuild: improve sanity check for ARCH variable on x86

[Borislav Petkov]

On Mon, Oct 19, 2026 at 05:36:25PM -0700, Nathan Chancellor wrote:
> On Mon, Oct 19, 2026 at 05:17:38PM -0700, Cao Jin wrote:
> > When building for x86, ARCH=i{4,5,6}86 and x86_32 will fail the build.
> 
> Thanks for the patch. Where is it documented that these other options
> are valid? It seems like this is just expected behavior because these
> are not valid values for setting ARCH for x86. At the very least, I
> think something like this needs to be acked by the x86 folks.

I hardly doubt that it'll ever be without a proper explanation why.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH] kbuild: improve sanity check for ARCH variable on x86

[Cao Jin]

Thank you both for prompt response.

I don't think there is document for it, and my rule of thumb tell me 
that all of i{3,4,5,6}86 & x86_32 represent 32-bit x86(unless I am 
wrong). A young developer may try any of those values at cmdline when 
building 32-bit x86 if he doesn't know the nuance among these value.

On 10/19/26 5:59 PM, Borislav Petkov wrote:
> On Mon, Oct 19, 2026 at 05:36:25PM -0700, Nathan Chancellor wrote:
>> On Mon, Oct 19, 2026 at 05:17:38PM -0700, Cao Jin wrote:
>>> When building for x86, ARCH=i{4,5,6}86 and x86_32 will fail the build.
>>
>> Thanks for the patch. Where is it documented that these other options
>> are valid? It seems like this is just expected behavior because these
>> are not valid values for setting ARCH for x86. At the very least, I
>> think something like this needs to be acked by the x86 folks.
> 
> I hardly doubt that it'll ever be without a proper explanation why.
> 

Re: [PATCH] kbuild: improve sanity check for ARCH variable on x86

[Cao Jin]

On 10/19/26 5:36 PM, Nathan Chancellor wrote:
> On Mon, Oct 19, 2026 at 05:17:38PM -0700, Cao Jin wrote:
>> When building for x86, ARCH=i{4,5,6}86 and x86_32 will fail the build.
> 
> Thanks for the patch. Where is it documented that these other options
> are valid? It seems like this is just expected behavior because these
> are not valid values for setting ARCH for x86. At the very least, I
> think something like this needs to be acked by the x86 folks.
> 

Forget to mention, "i386" & "x86_64" are not valid value for ARCH for 
x86 either, the top Makefile correct them to "x86". So we may want to 
correct all other possible invalid value for x86 ARCH, if my 
understanding of i{3,4,5,6}86 are the same for 32-bit x86 were right.

Re: [PATCH] kbuild: improve sanity check for ARCH variable on x86

[Randy Dunlap]

On 10/19/26 5:17 PM, Cao Jin wrote:
> When building for x86, ARCH=i{4,5,6}86 and x86_32 will fail the build.
> Make it catch all name variants.
> ---
>  Makefile | 10 ++++------
>  1 file changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/Makefile b/Makefile
> index 7c855e6fe544..cd397027891a 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -409,12 +409,10 @@ ARCH		?= $(SUBARCH)
>  UTS_MACHINE 	:= $(ARCH)
>  SRCARCH 	:= $(ARCH)
>  
> -# Additional ARCH settings for x86
> -ifeq ($(ARCH),i386)
> -        SRCARCH := x86
> -endif
> -ifeq ($(ARCH),x86_64)
> -        SRCARCH := x86
> +# Additional sanity check for cmdline ARCH settings for x86. This catch
> +# more name variants than "i386" and "x86_64".
> +ifneq ($(findstring 86,$(ARCH)),)
> +    SRCARCH := x86
>  endif
>  
>  # Additional ARCH settings for sparc

From Documentation/kbuild/kbuild.rst:

ARCH
----
Set ARCH to the architecture to be built.

In most cases the name of the architecture is the same as the
directory name found in the arch/ directory.

But some architectures such as x86 and sparc have aliases.

- x86: i386 for 32 bit, x86_64 for 64 bit
- parisc: parisc64 for 64 bit
- sparc: sparc32 for 32 bit, sparc64 for 64 bit

-- 
~Randy

Re: [PATCH] kbuild: improve sanity check for ARCH variable on x86

[Randy Dunlap]

On 10/19/26 6:33 PM, Cao Jin wrote:
> 
> 
> On 10/19/26 5:36 PM, Nathan Chancellor wrote:
>> On Mon, Oct 19, 2026 at 05:17:38PM -0700, Cao Jin wrote:
>>> When building for x86, ARCH=i{4,5,6}86 and x86_32 will fail the build.
>>
>> Thanks for the patch. Where is it documented that these other options
>> are valid? It seems like this is just expected behavior because these
>> are not valid values for setting ARCH for x86. At the very least, I
>> think something like this needs to be acked by the x86 folks.
>>
> 
> Forget to mention, "i386" & "x86_64" are not valid value for ARCH for x86 either, the top Makefile correct them to "x86". So we may want to correct all other possible invalid value for x86 ARCH, if my understanding of i{3,4,5,6}86 are the same for 32-bit x86 were right.

They are documented aliases for x86.

-- 
~Randy

Re: [PATCH] kbuild: improve sanity check for ARCH variable on x86

[Cao Jin]

On 10/19/26 6:38 PM, Randy Dunlap wrote:
> 

> 
>  From Documentation/kbuild/kbuild.rst:
> 
> ARCH
> ----
> Set ARCH to the architecture to be built.
> 
> In most cases the name of the architecture is the same as the
> directory name found in the arch/ directory.
> 
> But some architectures such as x86 and sparc have aliases.
> 
> - x86: i386 for 32 bit, x86_64 for 64 bit
> - parisc: parisc64 for 64 bit
> - sparc: sparc32 for 32 bit, sparc64 for 64 bit
> 
I see. Thank you.
Sorry for the noise.