mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
	linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org
Cc: "Sebastian Andrzej Siewior" <sebastian@breakpoint.cc>,
	"Maciej W. Rozycki" <macro@orcam.me.uk>,
	"Bill Wendling" <morbo@google.com>,
	"Borislav Petkov" <bp@alien8.de>,
	"Dave Hansen" <dave.hansen@linux.intel.com>,
	"Ingo Molnar" <mingo@redhat.com>,
	"John Paul Adrian Glaubitz" <glaubitz@physik.fu-berlin.de>,
	"Jonathan Corbet" <corbet@lwn.net>,
	"Justin Stitt" <justinstitt@google.com>,
	"Nathan Chancellor" <nathan@kernel.org>,
	"Neal Gompa" <neal@gompa.dev>,
	"Nick Desaulniers" <ndesaulniers@google.com>,
	"Richard Purdie" <richard.purdie@linuxfoundation.org>,
	"Sam James" <sam@gentoo.org>, "Shuah Khan" <shuah@kernel.org>,
	"Thomas Gleixner" <tglx@kernel.org>,
	"Thomas Weißschuh" <linux@weissschuh.net>,
	"Tomas Glozar" <tglozar@kernel.org>, "Willy Tarreau" <w@1wt.eu>,
	x86@kernel.org
Subject: Re: [PATCH v4 1/2] selftests/nolibc: remove the x32 testcase
Date: Thu, 08 Oct 2026 14:15:09 +0200	[thread overview]
Message-ID: <981CB9F6-143F-4EB0-9B95-8476B5A55633@zytor.com> (raw)
In-Reply-To: <20261008-x32_removal-v4-1-d389a193c506@breakpoint.cc>

On October 8, 2026 11:07:42 AM GMT+02:00, Sebastian Andrzej Siewior <bigeasy@linutronix.de> wrote:
>From: Sebastian Andrzej Siewior <sebastian@breakpoint.cc>
>
>From: Thomas Weißschuh <linux@weissschuh.net>
>
>Support for the x32 ABI is about to be removed from the kernel itself.
>This will break the nolibc x32 testcase.
>
>Remove the testcase to avoid breaking the testsuite in general.
>
>This also ends official support from nolibc proper for x32. But as there
>is no clear x32-specific code in the nolibc codebase, there won't be a
>removal patch for that.
>
>Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
>Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
>---
> tools/testing/selftests/nolibc/Makefile.nolibc | 6 ------
> tools/testing/selftests/nolibc/run-tests.sh    | 7 +------
> 2 files changed, 1 insertion(+), 12 deletions(-)
>
>diff --git a/tools/testing/selftests/nolibc/Makefile.nolibc b/tools/testing/selftests/nolibc/Makefile.nolibc
>index f70c8dfca0186..895a6ea7a8e45 100644
>--- a/tools/testing/selftests/nolibc/Makefile.nolibc
>+++ b/tools/testing/selftests/nolibc/Makefile.nolibc
>@@ -47,7 +47,6 @@ XARCH_riscv      = riscv64
> XARCH            = $(or $(XARCH_$(ARCH)),$(ARCH))
> 
> # map from user input variants to their kernel supported architectures
>-ARCH_x32         = x86
> ARCH_armthumb    = arm
> ARCH_ppc         = powerpc
> ARCH_ppc64       = powerpc
>@@ -70,7 +69,6 @@ ARCH            := $(or $(ARCH_$(XARCH)),$(XARCH))
> # kernel image names by architecture
> IMAGE_i386       = arch/x86/boot/bzImage
> IMAGE_x86_64     = arch/x86/boot/bzImage
>-IMAGE_x32        = arch/x86/boot/bzImage
> IMAGE_x86        = arch/x86/boot/bzImage
> IMAGE_arm64      = arch/arm64/boot/Image
> IMAGE_arm        = arch/arm/boot/zImage
>@@ -106,7 +104,6 @@ DEFCONFIG_sh4        = rts7751r2dplus_defconfig
> DEFCONFIG_openrisc   = virt_defconfig
> DEFCONFIG            = $(or $(DEFCONFIG_$(XARCH)),defconfig)
> 
>-EXTRACONFIG_x32       = -e CONFIG_X86_X32_ABI
> EXTRACONFIG_arm       = -e CONFIG_NAMESPACES
> EXTRACONFIG_armthumb  = -e CONFIG_NAMESPACES
> EXTRACONFIG_sparc32   = -e CONFIG_TMPFS
>@@ -119,7 +116,6 @@ EXTRACONFIG           = $(EXTRACONFIG_$(XARCH))
> TEST =
> 
> # QEMU_ARCH: arch names used by qemu
>-QEMU_ARCH_x32        = x86_64
> QEMU_ARCH_x86        = x86_64
> QEMU_ARCH_arm64      = aarch64
> QEMU_ARCH_armthumb   = arm
>@@ -151,7 +147,6 @@ endif
> # QEMU_ARGS : some arch-specific args to pass to qemu
> QEMU_ARGS_i386       = -M pc -append "console=ttyS0,9600 i8042.noaux panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> QEMU_ARGS_x86_64     = -M pc -append "console=ttyS0,9600 i8042.noaux panic=-1 $(TEST:%=NOLIBC_TEST=%)"
>-QEMU_ARGS_x32        = -M pc -append "console=ttyS0,9600 i8042.noaux panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> QEMU_ARGS_x86        = -M pc -append "console=ttyS0,9600 i8042.noaux panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> QEMU_ARGS_arm64      = -M virt -cpu cortex-a53 -append "panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> QEMU_ARGS_arm        = -M virt -append "panic=-1 $(TEST:%=NOLIBC_TEST=%)"
>@@ -189,7 +184,6 @@ Q=@
> endif
> 
> CFLAGS_i386 = $(call cc-option,-m32)
>-CFLAGS_x32 = -mx32
> CFLAGS_arm = -marm
> CFLAGS_armthumb = -mthumb -march=armv6t2
> CFLAGS_parisc32 = -mfast-indirect-calls
>diff --git a/tools/testing/selftests/nolibc/run-tests.sh b/tools/testing/selftests/nolibc/run-tests.sh
>index dc0b1649c6419..20e0d2dc0e9b9 100755
>--- a/tools/testing/selftests/nolibc/run-tests.sh
>+++ b/tools/testing/selftests/nolibc/run-tests.sh
>@@ -18,7 +18,7 @@ test_mode=system
> werror=1
> llvm=
> all_archs=(
>-	i386 x86_64 x32
>+	i386 x86_64
> 	arm64 arm armthumb
> 	mips32le mips32be mipsn32le mipsn32be mips64le mips64be
> 	openrisc
>@@ -119,7 +119,6 @@ crosstool_arch() {
> 	mips*) echo mips;;
> 	s390*) echo s390;;
> 	sparc*) echo sparc64;;
>-	x32*) echo x86_64;;
> 	parisc32) echo hppa;;
> 	*) echo "$1";;
> 	esac
>@@ -198,10 +197,6 @@ test_arch() {
> 		echo "Unsupported configuration"
> 		return
> 	fi
>-	if [ "$arch" = "x32" ] && [ "$test_mode" = "user" ]; then
>-		echo "Unsupported configuration"
>-		return
>-	fi
> 
> 	mkdir -p "$build_dir"
> 	swallow_output "${MAKE[@]}" defconfig
>

Acked-by: H. Peter Anvin (Rosaic) <hpa@zytor.com>

  reply	other threads:[~2026-10-08 12:19 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-08  9:07 [PATCH v4 0/2] Start removing X86_X32_ABI Sebastian Andrzej Siewior
2026-10-08  9:07 ` [PATCH v4 1/2] selftests/nolibc: remove the x32 testcase Sebastian Andrzej Siewior
2026-10-08 12:15   ` H. Peter Anvin [this message]
2026-10-08  9:07 ` [PATCH v4 2/2] x86: Start removing X86_X32_ABI Sebastian Andrzej Siewior
2026-10-08 12:15   ` H. Peter Anvin
2026-10-10  2:13   ` Tancred
2026-10-10  6:51     ` H. Peter Anvin
2026-10-10  7:28       ` John Paul Adrian Glaubitz
2026-10-10 10:55         ` H. Peter Anvin
2026-10-10  6:38   ` H. Peter Anvin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=981CB9F6-143F-4EB0-9B95-8476B5A55633@zytor.com \
    --to=hpa@zytor.com \
    --cc=bigeasy@linutronix.de \
    --cc=bp@alien8.de \
    --cc=corbet@lwn.net \
    --cc=dave.hansen@linux.intel.com \
    --cc=glaubitz@physik.fu-berlin.de \
    --cc=justinstitt@google.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=linux@weissschuh.net \
    --cc=macro@orcam.me.uk \
    --cc=mingo@redhat.com \
    --cc=morbo@google.com \
    --cc=nathan@kernel.org \
    --cc=ndesaulniers@google.com \
    --cc=neal@gompa.dev \
    --cc=richard.purdie@linuxfoundation.org \
    --cc=sam@gentoo.org \
    --cc=sebastian@breakpoint.cc \
    --cc=shuah@kernel.org \
    --cc=tglozar@kernel.org \
    --cc=tglx@kernel.org \
    --cc=w@1wt.eu \
    --cc=x86@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®