diff options
Diffstat (limited to 'drivers/scsi/aacraid')
-rw-r--r-- | drivers/scsi/aacraid/linit.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/drivers/scsi/aacraid/linit.c b/drivers/scsi/aacraid/linit.c index cad6f9abaeb9..6e233362dd7b 100644 --- a/drivers/scsi/aacraid/linit.c +++ b/drivers/scsi/aacraid/linit.c | |||
@@ -38,7 +38,7 @@ | |||
38 | #include <linux/moduleparam.h> | 38 | #include <linux/moduleparam.h> |
39 | #include <linux/pci.h> | 39 | #include <linux/pci.h> |
40 | #include <linux/slab.h> | 40 | #include <linux/slab.h> |
41 | #include <linux/smp_lock.h> | 41 | #include <linux/mutex.h> |
42 | #include <linux/spinlock.h> | 42 | #include <linux/spinlock.h> |
43 | #include <linux/syscalls.h> | 43 | #include <linux/syscalls.h> |
44 | #include <linux/delay.h> | 44 | #include <linux/delay.h> |
@@ -76,6 +76,7 @@ MODULE_DESCRIPTION("Dell PERC2, 2/Si, 3/Si, 3/Di, " | |||
76 | MODULE_LICENSE("GPL"); | 76 | MODULE_LICENSE("GPL"); |
77 | MODULE_VERSION(AAC_DRIVER_FULL_VERSION); | 77 | MODULE_VERSION(AAC_DRIVER_FULL_VERSION); |
78 | 78 | ||
79 | static DEFINE_MUTEX(aac_mutex); | ||
79 | static LIST_HEAD(aac_devices); | 80 | static LIST_HEAD(aac_devices); |
80 | static int aac_cfg_major = -1; | 81 | static int aac_cfg_major = -1; |
81 | char aac_driver_version[] = AAC_DRIVER_FULL_VERSION; | 82 | char aac_driver_version[] = AAC_DRIVER_FULL_VERSION; |
@@ -678,7 +679,7 @@ static int aac_cfg_open(struct inode *inode, struct file *file) | |||
678 | unsigned minor_number = iminor(inode); | 679 | unsigned minor_number = iminor(inode); |
679 | int err = -ENODEV; | 680 | int err = -ENODEV; |
680 | 681 | ||
681 | lock_kernel(); /* BKL pushdown: nothing else protects this list */ | 682 | mutex_lock(&aac_mutex); /* BKL pushdown: nothing else protects this list */ |
682 | list_for_each_entry(aac, &aac_devices, entry) { | 683 | list_for_each_entry(aac, &aac_devices, entry) { |
683 | if (aac->id == minor_number) { | 684 | if (aac->id == minor_number) { |
684 | file->private_data = aac; | 685 | file->private_data = aac; |
@@ -686,7 +687,7 @@ static int aac_cfg_open(struct inode *inode, struct file *file) | |||
686 | break; | 687 | break; |
687 | } | 688 | } |
688 | } | 689 | } |
689 | unlock_kernel(); | 690 | mutex_unlock(&aac_mutex); |
690 | 691 | ||
691 | return err; | 692 | return err; |
692 | } | 693 | } |
@@ -711,9 +712,9 @@ static long aac_cfg_ioctl(struct file *file, | |||
711 | int ret; | 712 | int ret; |
712 | if (!capable(CAP_SYS_RAWIO)) | 713 | if (!capable(CAP_SYS_RAWIO)) |
713 | return -EPERM; | 714 | return -EPERM; |
714 | lock_kernel(); | 715 | mutex_lock(&aac_mutex); |
715 | ret = aac_do_ioctl(file->private_data, cmd, (void __user *)arg); | 716 | ret = aac_do_ioctl(file->private_data, cmd, (void __user *)arg); |
716 | unlock_kernel(); | 717 | mutex_unlock(&aac_mutex); |
717 | 718 | ||
718 | return ret; | 719 | return ret; |
719 | } | 720 | } |
@@ -722,7 +723,7 @@ static long aac_cfg_ioctl(struct file *file, | |||
722 | static long aac_compat_do_ioctl(struct aac_dev *dev, unsigned cmd, unsigned long arg) | 723 | static long aac_compat_do_ioctl(struct aac_dev *dev, unsigned cmd, unsigned long arg) |
723 | { | 724 | { |
724 | long ret; | 725 | long ret; |
725 | lock_kernel(); | 726 | mutex_lock(&aac_mutex); |
726 | switch (cmd) { | 727 | switch (cmd) { |
727 | case FSACTL_MINIPORT_REV_CHECK: | 728 | case FSACTL_MINIPORT_REV_CHECK: |
728 | case FSACTL_SENDFIB: | 729 | case FSACTL_SENDFIB: |
@@ -756,7 +757,7 @@ static long aac_compat_do_ioctl(struct aac_dev *dev, unsigned cmd, unsigned long | |||
756 | ret = -ENOIOCTLCMD; | 757 | ret = -ENOIOCTLCMD; |
757 | break; | 758 | break; |
758 | } | 759 | } |
759 | unlock_kernel(); | 760 | mutex_unlock(&aac_mutex); |
760 | return ret; | 761 | return ret; |
761 | } | 762 | } |
762 | 763 | ||