proc: warn on non-existing proc entries

[safe/jmp/linux-2.6] / fs / proc / generic.c

diff --git a/fs/proc/generic.c b/fs/proc/generic.c
index 9580abe..08f4d71 100644 (file)
--- a/fs/proc/generic.c
+++ b/fs/proc/generic.c
@@ -291,19 +291,17 @@ static const struct inode_operations proc_file_inode_operations = {
  * returns the struct proc_dir_entry for "/proc/tty/driver", and
  * returns "serial" in residual.
  */
-static int xlate_proc_name(const char *name,
-                          struct proc_dir_entry **ret, const char **residual)
+static int __xlate_proc_name(const char *name, struct proc_dir_entry **ret,
+                            const char **residual)
 {
        const char              *cp = name, *next;
        struct proc_dir_entry   *de;
        int                     len;
-       int                     rtn = 0;
 
        de = *ret;
        if (!de)
                de = &proc_root;
 
-       spin_lock(&proc_subdir_lock);
        while (1) {
                next = strchr(cp, '/');
                if (!next)
@@ -315,16 +313,25 @@ static int xlate_proc_name(const char *name,
                                break;
                }
                if (!de) {
-                       rtn = -ENOENT;
-                       goto out;
+                       WARN(1, "name '%s'\n", name);
+                       return -ENOENT;
                }
                cp += len + 1;
        }
        *residual = cp;
        *ret = de;
-out:
+       return 0;
+}
+
+static int xlate_proc_name(const char *name, struct proc_dir_entry **ret,
+                          const char **residual)
+{
+       int rv;
+
+       spin_lock(&proc_subdir_lock);
+       rv = __xlate_proc_name(name, ret, residual);
        spin_unlock(&proc_subdir_lock);
-       return rtn;
+       return rv;
 }
 
 static DEFINE_IDA(proc_inum_ida);
@@ -797,11 +804,13 @@ void remove_proc_entry(const char *name, struct proc_dir_entry *parent)
        const char *fn = name;
        int len;
 
-       if (xlate_proc_name(name, &parent, &fn) != 0)
+       spin_lock(&proc_subdir_lock);
+       if (__xlate_proc_name(name, &parent, &fn) != 0) {
+               spin_unlock(&proc_subdir_lock);
                return;
+       }
        len = strlen(fn);
 
-       spin_lock(&proc_subdir_lock);
        for (p = &parent->subdir; *p; p=&(*p)->next ) {
                if (proc_match(len, fn, *p)) {
                        de = *p;
@@ -811,8 +820,10 @@ void remove_proc_entry(const char *name, struct proc_dir_entry *parent)
                }
        }
        spin_unlock(&proc_subdir_lock);
-       if (!de)
+       if (!de) {
+               WARN(1, "name '%s'\n", name);
                return;
+       }
 
        spin_lock(&de->pde_unload_lock);
        /*