linux-kernel - Re: [PATCH v4 -next 08/15] mm: nommu: move sysctl to mm/nommu.c

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: <1959b553-decd-7525-865e-425776841833@huawei.com>
Date: Thu, 9 Jan 2025 10:26:51 +0800
From: yukaixiong <yukaixiong@...wei.com>
To: Lorenzo Stoakes <lorenzo.stoakes@...cle.com>
CC: <akpm@...ux-foundation.org>, <mcgrof@...nel.org>,
	<ysato@...rs.sourceforge.jp>, <dalias@...c.org>,
	<glaubitz@...sik.fu-berlin.de>, <luto@...nel.org>, <tglx@...utronix.de>,
	<mingo@...hat.com>, <bp@...en8.de>, <dave.hansen@...ux.intel.com>,
	<hpa@...or.com>, <viro@...iv.linux.org.uk>, <brauner@...nel.org>,
	<jack@...e.cz>, <kees@...nel.org>, <j.granados@...sung.com>,
	<willy@...radead.org>, <Liam.Howlett@...cle.com>, <vbabka@...e.cz>,
	<trondmy@...nel.org>, <anna@...nel.org>, <chuck.lever@...cle.com>,
	<jlayton@...nel.org>, <neilb@...e.de>, <okorniev@...hat.com>,
	<Dai.Ngo@...cle.com>, <tom@...pey.com>, <davem@...emloft.net>,
	<edumazet@...gle.com>, <kuba@...nel.org>, <pabeni@...hat.com>,
	<paul@...l-moore.com>, <jmorris@...ei.org>, <linux-sh@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>, <linux-fsdevel@...r.kernel.org>,
	<linux-mm@...ck.org>, <linux-nfs@...r.kernel.org>, <netdev@...r.kernel.org>,
	<linux-security-module@...r.kernel.org>, <dhowells@...hat.com>,
	<haifeng.xu@...pee.com>, <baolin.wang@...ux.alibaba.com>,
	<shikemeng@...weicloud.com>, <dchinner@...hat.com>, <bfoster@...hat.com>,
	<souravpanda@...gle.com>, <hannes@...xchg.org>, <rientjes@...gle.com>,
	<pasha.tatashin@...een.com>, <david@...hat.com>, <ryan.roberts@....com>,
	<ying.huang@...el.com>, <yang@...amperecomputing.com>,
	<zev@...ilderbeest.net>, <serge@...lyn.com>, <vegard.nossum@...cle.com>,
	<wangkefeng.wang@...wei.com>
Subject: Re: [PATCH v4 -next 08/15] mm: nommu: move sysctl to mm/nommu.c



On 2025/1/2 22:09, Lorenzo Stoakes wrote:
> On Mon, Dec 23, 2024 at 10:15:27PM +0800, Kaixiong Yu wrote:
>> The sysctl_nr_trim_pages belongs to nommu.c, move it to mm/nommu.c
>> from /kernel/sysctl.c. And remove the useless extern variable declaration
>> from include/linux/mm.h
>>
>> Signed-off-by: Kaixiong Yu <yukaixiong@...wei.com>
> Looks good to me,
>
> Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@...cle.com>
Thanks for your review !

Best ...
>> ---
>> v4:
>>   - const qualify struct ctl_table nommu_table
>> v3:
>>   - change the title
>> v2:
>>   - fix the build error: expected ';' after top level declarator
>>   - fix the build error: call to undeclared function 'register_syscall_init',
>>     use 'register_sysctl_init' to replace it.
>> ---
>> ---
>>   include/linux/mm.h |  2 --
>>   kernel/sysctl.c    | 10 ----------
>>   mm/nommu.c         | 15 ++++++++++++++-
>>   3 files changed, 14 insertions(+), 13 deletions(-)
>>
>> diff --git a/include/linux/mm.h b/include/linux/mm.h
>> index b3b87c1dc1e4..9813b5b9c093 100644
>> --- a/include/linux/mm.h
>> +++ b/include/linux/mm.h
>> @@ -4080,8 +4080,6 @@ unsigned long wp_shared_mapping_range(struct address_space *mapping,
>>   				      pgoff_t first_index, pgoff_t nr);
>>   #endif
>>
>> -extern int sysctl_nr_trim_pages;
>> -
>>   #ifdef CONFIG_PRINTK
>>   void mem_dump_obj(void *object);
>>   #else
>> diff --git a/kernel/sysctl.c b/kernel/sysctl.c
>> index 62a58e417c40..97f9abffff0f 100644
>> --- a/kernel/sysctl.c
>> +++ b/kernel/sysctl.c
>> @@ -2031,16 +2031,6 @@ static struct ctl_table vm_table[] = {
>>   		.extra1		= SYSCTL_ONE,
>>   		.extra2		= SYSCTL_FOUR,
>>   	},
>> -#ifndef CONFIG_MMU
>> -	{
>> -		.procname	= "nr_trim_pages",
>> -		.data		= &sysctl_nr_trim_pages,
>> -		.maxlen		= sizeof(sysctl_nr_trim_pages),
>> -		.mode		= 0644,
>> -		.proc_handler	= proc_dointvec_minmax,
>> -		.extra1		= SYSCTL_ZERO,
>> -	},
>> -#endif
> Of course later on in the series you do what I asked in a previous commit :P Nice.
>
>>   	{
>>   		.procname	= "vfs_cache_pressure",
>>   		.data		= &sysctl_vfs_cache_pressure,
>> diff --git a/mm/nommu.c b/mm/nommu.c
>> index baa79abdaf03..3c32f8b1eb54 100644
>> --- a/mm/nommu.c
>> +++ b/mm/nommu.c
>> @@ -48,7 +48,6 @@ struct page *mem_map;
>>   unsigned long max_mapnr;
>>   EXPORT_SYMBOL(max_mapnr);
>>   unsigned long highest_memmap_pfn;
>> -int sysctl_nr_trim_pages = CONFIG_NOMMU_INITIAL_TRIM_EXCESS;
>>   int heap_stack_gap = 0;
>>
>>   atomic_long_t mmap_pages_allocated;
>> @@ -392,6 +391,19 @@ SYSCALL_DEFINE1(brk, unsigned long, brk)
>>   	return mm->brk = brk;
>>   }
>>
>> +static int sysctl_nr_trim_pages = CONFIG_NOMMU_INITIAL_TRIM_EXCESS;
>> +
>> +static const struct ctl_table nommu_table[] = {
>> +	{
>> +		.procname	= "nr_trim_pages",
>> +		.data		= &sysctl_nr_trim_pages,
>> +		.maxlen		= sizeof(sysctl_nr_trim_pages),
>> +		.mode		= 0644,
>> +		.proc_handler	= proc_dointvec_minmax,
>> +		.extra1		= SYSCTL_ZERO,
>> +	},
>> +};
>> +
>>   /*
>>    * initialise the percpu counter for VM and region record slabs
>>    */
>> @@ -402,6 +414,7 @@ void __init mmap_init(void)
>>   	ret = percpu_counter_init(&vm_committed_as, 0, GFP_KERNEL);
>>   	VM_BUG_ON(ret);
>>   	vm_region_jar = KMEM_CACHE(vm_region, SLAB_PANIC|SLAB_ACCOUNT);
>> +	register_sysctl_init("vm", nommu_table);
>>   }
>>
>>   /*
>> --
>> 2.34.1
>>
> .
>