mm/vmalloc.c: use seq_open_private() instead of seq_open()
authorRob Jones <rob.jones@codethink.co.uk>
Thu, 9 Oct 2014 22:28:01 +0000 (15:28 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 10 Oct 2014 02:25:56 +0000 (22:25 -0400)
Using seq_open_private() removes boilerplate code from vmalloc_open().

The resultant code is shorter and easier to follow.

However, please note that seq_open_private() call kzalloc() rather than
kmalloc() which may affect timing due to the memory initialisation
overhead.

Signed-off-by: Rob Jones <rob.jones@codethink.co.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
mm/vmalloc.c

index 2b0aa5486092dca2745c2ec201cda44db033550c..90520af7f18604ccd825d2595e6103dccbbf3f32 100644 (file)
@@ -2646,21 +2646,11 @@ static const struct seq_operations vmalloc_op = {
 
 static int vmalloc_open(struct inode *inode, struct file *file)
 {
-       unsigned int *ptr = NULL;
-       int ret;
-
-       if (IS_ENABLED(CONFIG_NUMA)) {
-               ptr = kmalloc(nr_node_ids * sizeof(unsigned int), GFP_KERNEL);
-               if (ptr == NULL)
-                       return -ENOMEM;
-       }
-       ret = seq_open(file, &vmalloc_op);
-       if (!ret) {
-               struct seq_file *m = file->private_data;
-               m->private = ptr;
-       } else
-               kfree(ptr);
-       return ret;
+       if (IS_ENABLED(CONFIG_NUMA))
+               return seq_open_private(file, &vmalloc_op,
+                                       nr_node_ids * sizeof(unsigned int));
+       else
+               return seq_open(file, &vmalloc_op);
 }
 
 static const struct file_operations proc_vmalloc_operations = {