linux-kernel - RE: [PATCH 2/4] lib/test_vmalloc.c: Allow built-in execution

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] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <TYZPR03MB6192DABDDF6F9E7D333D54E6B3852@TYZPR03MB6192.apcprd03.prod.outlook.com>
Date: Thu, 24 Apr 2025 13:37:27 +0000
From: Adrian Huang12 <ahuang12@...ovo.com>
To: "Uladzislau Rezki (Sony)" <urezki@...il.com>,
        Andrew Morton
	<akpm@...ux-foundation.org>
CC: "linux-mm@...ck.org" <linux-mm@...ck.org>,
        LKML
	<linux-kernel@...r.kernel.org>, Baoquan He <bhe@...hat.com>,
        Christoph
 Hellwig <hch@...radead.org>,
        Oleksiy Avramchenko
	<oleksiy.avramchenko@...y.com>
Subject: RE: [PATCH 2/4] lib/test_vmalloc.c: Allow built-in execution

> -----Original Message-----
> From: owner-linux-mm@...ck.org <owner-linux-mm@...ck.org> On Behalf
> Of Uladzislau Rezki (Sony)
> Sent: Friday, April 18, 2025 12:12 AM
> To: Andrew Morton <akpm@...ux-foundation.org>
> Cc: linux-mm@...ck.org; LKML <linux-kernel@...r.kernel.org>; Baoquan He
> <bhe@...hat.com>; Christoph Hellwig <hch@...radead.org>; Uladzislau Rezki
> <urezki@...il.com>; Oleksiy Avramchenko <oleksiy.avramchenko@...y.com>
> Subject: [External] [PATCH 2/4] lib/test_vmalloc.c: Allow built-in execution
> 
> This patch removes the dependency on module loading ("m") for the vmalloc
> test suite, enabling it to be built directly into the kernel, so both ("=m") and
> ("=y") are supported.
> 
> Motivation:
> - Faster debugging/testing of vmalloc code;
> - It allows to configure the test via kernel-boot parameters.
> 
> Configuration example:
>   test_vmalloc.nr_threads=64
>   test_vmalloc.run_test_mask=7
>   test_vmalloc.sequential_test_order=1
> 
> Signed-off-by: Uladzislau Rezki (Sony) <urezki@...il.com>
> ---
>  lib/Kconfig.debug  | 3 +--
>  lib/test_vmalloc.c | 5 +++--
>  2 files changed, 4 insertions(+), 4 deletions(-)

Reviewed-by: Adrian Huang <ahuang12@...ovo.com>
Tested-by: Adrian Huang <ahuang12@...ovo.com>

> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug index
> f9051ab610d54..166b9d830a85a 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -2574,8 +2574,7 @@ config TEST_BITOPS  config TEST_VMALLOC
>  	tristate "Test module for stress/performance analysis of vmalloc allocator"
>  	default n
> -       depends on MMU
> -	depends on m
> +	depends on MMU
>  	help
>  	  This builds the "test_vmalloc" module that should be used for
>  	  stress and performance analysis. So, any new change for vmalloc diff
> --git a/lib/test_vmalloc.c b/lib/test_vmalloc.c index
> 4ab23e5e772d0..6d65ef725d42c 100644
> --- a/lib/test_vmalloc.c
> +++ b/lib/test_vmalloc.c
> @@ -591,10 +591,11 @@ static void do_concurrent_test(void)
>  	kvfree(tdriver);
>  }
> 
> -static int vmalloc_test_init(void)
> +static int __init vmalloc_test_init(void)
>  {
>  	do_concurrent_test();
> -	return -EAGAIN; /* Fail will directly unload the module */
> +	/* Fail will directly unload the module */
> +	return IS_BUILTIN(CONFIG_TEST_VMALLOC) ? 0:-EAGAIN;
>  }
> 
>  module_init(vmalloc_test_init)
> --
> 2.39.5
>