Staging: bcm: Reverse semaphore locking in IOCTL_BCM_BUFFER_DOWNLOAD_STOP.
authorKevin McKinney <klmckinney1@gmail.com>
Mon, 28 Nov 2011 01:51:47 +0000 (20:51 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 29 Nov 2011 01:09:52 +0000 (10:09 +0900)
This patch reorders the semaphore locking.
It makes better sense to first evaluate
fw_download_sema semaphore then
NVMRdmWrmLocl semaphore. The
fw_download_sema is suppose to be
acquired in the START ioctl.  If this is
not true, then it does not make sense
to continue.

Signed-off-by: Kevin McKinney <klmckinney1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/bcm/Bcmchar.c

index 3b7c15039ee82a9639c1e8476971c0f4bed6585a..72a699c127040371f134b45db3eba0ed48c38906 100644 (file)
@@ -860,6 +860,11 @@ cntrlEnd:
        }
 
        case IOCTL_BCM_BUFFER_DOWNLOAD_STOP: {
+               if (!down_trylock(&Adapter->fw_download_sema)) {
+                       up(&Adapter->fw_download_sema);
+                       return -EINVAL;
+               }
+
                if (down_trylock(&Adapter->NVMRdmWrmLock)) {
                        BCM_DEBUG_PRINT(Adapter, DBG_TYPE_PRINTK, 0, 0,
                                        "FW download blocked as EEPROM Read/Write is in progress\n");
@@ -867,11 +872,6 @@ cntrlEnd:
                        return -EACCES;
                }
 
-               if (!down_trylock(&Adapter->fw_download_sema)) {
-                       up(&Adapter->fw_download_sema);
-                       return -EINVAL;
-               }
-
                Adapter->bBinDownloaded = TRUE;
                Adapter->bCfgDownloaded = TRUE;
                atomic_set(&Adapter->CurrNumFreeTxDesc, 0);