On Wed, Oct 20, 2010 at 11:01 AM, Joe Perches [off-list ref] wrote:

On Wed, 2010-10-20 at 10:54 +0800, Changli Gao wrote:

quoted

Signed-off-by: Changli Gao <redacted>

[]

quoted

diff --git a/net/appletalk/sysctl_net_atalk.c b/net/appletalk/sysctl_net_atalk.c
index 04e9c0d..b92f269 100644
--- a/net/appletalk/sysctl_net_atalk.c
+++ b/net/appletalk/sysctl_net_atalk.c

@@ -42,16 +42,16 @@ static struct ctl_table atalk_table[] = {

      { },
 };

-static struct ctl_path atalk_path[] = {
-     { .procname = "net", },
-     { .procname = "appletalk", },
-     { }
-};
-
 static struct ctl_table_header *atalk_table_header;

 void atalk_register_sysctl(void)
 {
+     const struct ctl_path atalk_path[] = {

Shouldn't all of these be static const struct ?

They needn't. And some variables are specified __net_initdata currently.

-- 
Regards,
Changli Gao(xiaosuo@gmail.com)