linux-kernel - Re: [PATCH] selftests/nolibc: start qemu with 1 GiB of memory

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: <f1e98283-459e-478c-85fd-e4015e13b8d9@t-8ch.de>
Date: Mon, 6 Jan 2025 21:01:55 +0100
From: Thomas Weißschuh <linux@...ssschuh.net>
To: Shuah Khan <skhan@...uxfoundation.org>
Cc: Thomas Weißschuh <thomas.weissschuh@...utronix.de>, 
	Willy Tarreau <w@....eu>, Shuah Khan <shuah@...nel.org>, linux-kselftest@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] selftests/nolibc: start qemu with 1 GiB of memory

On 2025-01-06 10:51:52-0700, Shuah Khan wrote:
> On 10/7/24 02:10, Thomas Weißschuh wrote:
> > Recently the loongarch defconfig stopped working with the default 128 MiB
> > of memory. The VM just spins infinitively.
> > Increasing the available memory to 1 GiB, similar to s390, fixes the
> > issue. To avoid having to do this for each architecture on its own,
> > proactively apply to all architectures.
> > 
> > Signed-off-by: Thomas Weißschuh <thomas.weissschuh@...utronix.de>
> > ---
> >   tools/testing/selftests/nolibc/Makefile | 4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/tools/testing/selftests/nolibc/Makefile b/tools/testing/selftests/nolibc/Makefile
> > index 8de98ea7af8071caa0597aa7b86d91a2d1d50e68..e92e0b88586111072a0e043cb15f3b59cf42c3a6 100644
> > --- a/tools/testing/selftests/nolibc/Makefile
> > +++ b/tools/testing/selftests/nolibc/Makefile
> > @@ -130,9 +130,9 @@ QEMU_ARGS_ppc        = -M g3beige -append "console=ttyS0 panic=-1 $(TEST:%=NOLIB
> >   QEMU_ARGS_ppc64      = -M powernv -append "console=hvc0 panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> >   QEMU_ARGS_ppc64le    = -M powernv -append "console=hvc0 panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> >   QEMU_ARGS_riscv      = -M virt -append "console=ttyS0 panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> > -QEMU_ARGS_s390       = -M s390-ccw-virtio -m 1G -append "console=ttyS0 panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> > +QEMU_ARGS_s390       = -M s390-ccw-virtio -append "console=ttyS0 panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> >   QEMU_ARGS_loongarch  = -M virt -append "console=ttyS0,115200 panic=-1 $(TEST:%=NOLIBC_TEST=%)"
> > -QEMU_ARGS            = $(QEMU_ARGS_$(XARCH)) $(QEMU_ARGS_BIOS) $(QEMU_ARGS_EXTRA)
> > +QEMU_ARGS            = -m 1G $(QEMU_ARGS_$(XARCH)) $(QEMU_ARGS_BIOS) $(QEMU_ARGS_EXTRA)
> >   # OUTPUT is only set when run from the main makefile, otherwise
> >   # it defaults to this nolibc directory.
> > 
> > ---
> > base-commit: 8cf0b93919e13d1e8d4466eb4080a4c4d9d66d7b
> > change-id: 20241007-nolibc-qemu-mem-5ed605520472
> > 
> > Best regards,
> 
> Did we take care of this one? Apologies if I didn't. We can include
> this in 6.14 pr.

This is already in mainline.


Thomas