Re: [perfmon] sysctl table check failed ... Writable sysctl directory

From: Stephane Eranian <eranian_at_hpl.hp.com>
Date: 2007-11-07 07:18:11
Tony,

Don't need 755, please apply your patch.
Thanks.

On Tue, Nov 06, 2007 at 11:59:55AM -0800, Luck, Tony wrote:
> Stephane,
> 
> For a couple of weeks now the kernel has been complaining about
> a couple of "Writable sysctl directories".  When I pulled from
> Linus latest git tree today, the complaint got louder with an
> added stack trace ... pointing at pfm_init().
> 
> This patch fixes the complaint.  Did you need mode 0755 for
> some reason, or should I just apply this?
> 
> -Tony
> 
> ---
> 
> diff --git a/arch/ia64/kernel/perfmon.c b/arch/ia64/kernel/perfmon.c
> index 59169bf..73e7c2e 100644
> --- a/arch/ia64/kernel/perfmon.c
> +++ b/arch/ia64/kernel/perfmon.c
> @@ -558,7 +558,7 @@ static ctl_table pfm_sysctl_dir[] = {
>  	{
>  		.ctl_name	= CTL_UNNUMBERED,
>  		.procname	= "perfmon",
> -		.mode		= 0755,
> +		.mode		= 0555,
>  		.child		= pfm_ctl_table,
>  	},
>   	{}
> @@ -567,7 +567,7 @@ static ctl_table pfm_sysctl_root[] = {
>  	{
>  		.ctl_name	= CTL_KERN,
>  		.procname	= "kernel",
> -		.mode		= 0755,
> +		.mode		= 0555,
>  		.child		= pfm_sysctl_dir,
>  	},
>   	{}
> 
> _______________________________________________
> perfmon mailing list
> perfmon@linux.hpl.hp.com
> http://www.hpl.hp.com/hosted/linux/mail-archives/perfmon/

-- 

-Stephane
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Wed Nov 07 07:18:27 2007

This archive was generated by hypermail 2.1.8 : 2007-11-07 07:18:42 EST