[SCSI] pm80xx: no need for tag allocation when issuing the command of unregistering...
authorXinHong Zhu <zxh3737@163.com>
Thu, 13 Feb 2014 03:59:38 +0000 (11:59 +0800)
committerJames Bottomley <JBottomley@Parallels.com>
Sat, 15 Mar 2014 17:19:22 +0000 (10:19 -0700)
In function pm8001_dev_gone_notify we consume a tag from bitmap resource and
later don't free that tag resource, so the tag resource will never be
freed. In the function we don't need to use any tag to issue command for
unregistering the device .

Signed-off-by: XinHong Zhu <zxh3737@163.com>
Acked-by: Lindar Liu <lindar_liu@usish.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/pm8001/pm8001_sas.c

index f50ac44..ba3088c 100644 (file)
@@ -865,13 +865,11 @@ ex_err:
 static void pm8001_dev_gone_notify(struct domain_device *dev)
 {
        unsigned long flags = 0;
-       u32 tag;
        struct pm8001_hba_info *pm8001_ha;
        struct pm8001_device *pm8001_dev = dev->lldd_dev;
 
        pm8001_ha = pm8001_find_ha_by_dev(dev);
        spin_lock_irqsave(&pm8001_ha->lock, flags);
-       pm8001_tag_alloc(pm8001_ha, &tag);
        if (pm8001_dev) {
                u32 device_id = pm8001_dev->device_id;