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
![]() |