[S390] monreader: remove lock_kernel() from open() function
authorGerald Schaefer <gerald.schaefer@de.ibm.com>
Mon, 7 Dec 2009 11:52:17 +0000 (12:52 +0100)
committerMartin Schwidefsky <sky@mschwide.boeblingen.de.ibm.com>
Mon, 7 Dec 2009 11:51:38 +0000 (12:51 +0100)
The lock_kernel() calls in the open() function are unnecessary because
misc_register() is called last in module_init() and concurrent open()
calls are handled by other means.

Signed-off-by: Gerald Schaefer <gerald.schaefer@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
drivers/s390/char/monreader.c

index 66e21dd23154b4c5c27b0c254a3bfd348cc88d48..60473f86e1f9271e6f12d1fcddcd85991d66df50 100644 (file)
@@ -12,7 +12,6 @@
 #include <linux/module.h>
 #include <linux/moduleparam.h>
 #include <linux/init.h>
-#include <linux/smp_lock.h>
 #include <linux/errno.h>
 #include <linux/types.h>
 #include <linux/kernel.h>
@@ -283,7 +282,6 @@ static int mon_open(struct inode *inode, struct file *filp)
        /*
         * only one user allowed
         */
-       lock_kernel();
        rc = -EBUSY;
        if (test_and_set_bit(MON_IN_USE, &mon_in_use))
                goto out;
@@ -321,7 +319,6 @@ static int mon_open(struct inode *inode, struct file *filp)
        }
        filp->private_data = monpriv;
        dev_set_drvdata(monreader_device, monpriv);
-       unlock_kernel();
        return nonseekable_open(inode, filp);
 
 out_path:
@@ -331,7 +328,6 @@ out_priv:
 out_use:
        clear_bit(MON_IN_USE, &mon_in_use);
 out:
-       unlock_kernel();
        return rc;
 }
 
@@ -607,6 +603,10 @@ static int __init mon_init(void)
        }
        dcss_mkname(mon_dcss_name, &user_data_connect[8]);
 
+       /*
+        * misc_register() has to be the last action in module_init(), because
+        * file operations will be available right after this.
+        */
        rc = misc_register(&mon_dev);
        if (rc < 0 )
                goto out;