sysctl table check failed ... Writable sysctl directory

From: Luck, Tony <tony.luck_at_intel.com>
Date: 2007-11-07 06:59:55
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,
 	},
  	{}
-
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:02:34 2007

This archive was generated by hypermail 2.1.8 : 2007-11-07 07:03:08 EST