[SUNRPC]: Use proc_create() to setup ->proc_fops first
authorWang Chen <wangchen@cn.fujitsu.com>
Thu, 28 Feb 2008 22:00:59 +0000 (14:00 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 28 Feb 2008 22:00:59 +0000 (14:00 -0800)
Use proc_create() to make sure that ->proc_fops be setup before gluing
PDE to main tree.

Signed-off-by: Wang Chen <wangchen@cn.fujitsu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sunrpc/cache.c
net/sunrpc/stats.c

index 636c8e04e0bebb0fff9b33e34a23bd8e40941dfc..b5f2786251b95368f4bccda69290683e1c3f0b28 100644 (file)
@@ -316,31 +316,29 @@ static int create_cache_proc_entries(struct cache_detail *cd)
        cd->proc_ent->owner = cd->owner;
        cd->channel_ent = cd->content_ent = NULL;
 
-       p = create_proc_entry("flush", S_IFREG|S_IRUSR|S_IWUSR, cd->proc_ent);
+       p = proc_create("flush", S_IFREG|S_IRUSR|S_IWUSR,
+                       cd->proc_ent, &cache_flush_operations);
        cd->flush_ent = p;
        if (p == NULL)
                goto out_nomem;
-       p->proc_fops = &cache_flush_operations;
        p->owner = cd->owner;
        p->data = cd;
 
        if (cd->cache_request || cd->cache_parse) {
-               p = create_proc_entry("channel", S_IFREG|S_IRUSR|S_IWUSR,
-                                     cd->proc_ent);
+               p = proc_create("channel", S_IFREG|S_IRUSR|S_IWUSR,
+                               cd->proc_ent, &cache_file_operations);
                cd->channel_ent = p;
                if (p == NULL)
                        goto out_nomem;
-               p->proc_fops = &cache_file_operations;
                p->owner = cd->owner;
                p->data = cd;
        }
        if (cd->cache_show) {
-               p = create_proc_entry("content", S_IFREG|S_IRUSR|S_IWUSR,
-                                     cd->proc_ent);
+               p = proc_create("content", S_IFREG|S_IRUSR|S_IWUSR,
+                               cd->proc_ent, &content_file_operations);
                cd->content_ent = p;
                if (p == NULL)
                        goto out_nomem;
-               p->proc_fops = &content_file_operations;
                p->owner = cd->owner;
                p->data = cd;
        }
index 5a16875f5ac8b0686ccdf5e9448dac850aab7064..c6061a4346c8ea73c94edd6d99f3e55e28d134c2 100644 (file)
@@ -229,9 +229,8 @@ do_register(const char *name, void *data, const struct file_operations *fops)
        rpc_proc_init();
        dprintk("RPC:       registering /proc/net/rpc/%s\n", name);
 
-       ent = create_proc_entry(name, 0, proc_net_rpc);
+       ent = proc_create(name, 0, proc_net_rpc, fops);
        if (ent) {
-               ent->proc_fops = fops;
                ent->data = data;
        }
        return ent;