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
> [EMAIL PROTECTED]
> 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 [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to