linux-kernel - Re: [PATCH] mm: move the scan_unevictable

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: <1239285872.24817.20.camel@lts-notebook>
Date:	Thu, 09 Apr 2009 10:04:32 -0400
From:	Lee Schermerhorn <Lee.Schermerhorn@...com>
To:	Peter Zijlstra <peterz@...radead.org>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	Rik van Riel <riel@...hat.com>,
	linux-mm <linux-mm@...ck.org>,
	linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] mm: move the scan_unevictable_pages sysctl to the vm
	table

On Thu, 2009-04-09 at 11:42 +0200, Peter Zijlstra wrote:
> Subject: mm: move the scan_unevictable_pages sysctl to the vm table
> From: Peter Zijlstra <a.p.zijlstra@...llo.nl>
> Date: Thu Apr 09 11:38:45 CEST 2009
> 
> vm knobs should go in the vm table. Probably too late for randomize_va_space
> though.

I was surprised to see "scan_unevictable_pages" in the kernel table.
This must be the result of a merge glitch.  I originally put this at the
end of the vm table.  Just went back and looked at some older patches to
be sure.  E.g.,

	http://marc.info/?l=linux-mm&m=121321022603288&w=4

It had been moved to the kernel table by the time the unevictable lru
was merged upstream:

	http://marc.info/?l=linux-mm-commits&m=122453486931267&w=4

Anyway...

> 
> Signed-off-by: Peter Zijlstra <a.p.zijlstra@...llo.nl>

Acked-by: Lee Schermerhorn <lee.schermerhorn@...com>

> ---
>  kernel/sysctl.c |   20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> Index: linux-2.6/kernel/sysctl.c
> ===================================================================
> --- linux-2.6.orig/kernel/sysctl.c
> +++ linux-2.6/kernel/sysctl.c
> @@ -914,16 +914,6 @@ static struct ctl_table kern_table[] = {
>  		.proc_handler	= &proc_dointvec,
>  	},
>  #endif
> -#ifdef CONFIG_UNEVICTABLE_LRU
> -	{
> -		.ctl_name	= CTL_UNNUMBERED,
> -		.procname	= "scan_unevictable_pages",
> -		.data		= &scan_unevictable_pages,
> -		.maxlen		= sizeof(scan_unevictable_pages),
> -		.mode		= 0644,
> -		.proc_handler	= &scan_unevictable_handler,
> -	},
> -#endif
>  #ifdef CONFIG_SLOW_WORK
>  	{
>  		.ctl_name	= CTL_UNNUMBERED,
> @@ -1324,6 +1314,16 @@ static struct ctl_table vm_table[] = {
>  		.extra2		= &one,
>  	},
>  #endif
> +#ifdef CONFIG_UNEVICTABLE_LRU
> +	{
> +		.ctl_name	= CTL_UNNUMBERED,
> +		.procname	= "scan_unevictable_pages",
> +		.data		= &scan_unevictable_pages,
> +		.maxlen		= sizeof(scan_unevictable_pages),
> +		.mode		= 0644,
> +		.proc_handler	= &scan_unevictable_handler,
> +	},
> +#endif
>  /*
>   * NOTE: do not add new entries to this table unless you have read
>   * Documentation/sysctl/ctl_unnumbered.txt
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/