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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Sparc Linux]     [DCCP]     [Linux ARM]     [Yosemite News]     [Linux SCSI]     [Linux x86_64]     [Linux for Ham Radio]

  Powered by Linux