diff options
| author | Don Brace <don.brace@pmcs.com> | 2015-07-18 11:12:22 -0500 |
|---|---|---|
| committer | James Bottomley <JBottomley@Odin.com> | 2015-08-26 16:19:50 -0700 |
| commit | 77678d3a35455d7b7c4da4e56b20de17ee63cec1 (patch) | |
| tree | 5116101a89a57903f5d18a3f82e20df95c71200a /drivers/scsi/hpsa.c | |
| parent | 5aeeb78aeb4c8607cbda54a7b0dc7315171e214f (diff) | |
| download | lwn-77678d3a35455d7b7c4da4e56b20de17ee63cec1.tar.gz lwn-77678d3a35455d7b7c4da4e56b20de17ee63cec1.zip | |
hpsa: Correct double unlock of mutex
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Scott Teel <scott.teel@pmcs.com>
Reviewed-by: Kevin Barnett <kevin.barnett@pmcs.com>
Reviewed-by: Tomas Henzl <thenzl@redhat.com>
Signed-off-by: Don Brace <don.brace@pmcs.com>
Signed-off-by: James Bottomley <JBottomley@Odin.com>
Diffstat (limited to 'drivers/scsi/hpsa.c')
| -rw-r--r-- | drivers/scsi/hpsa.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index cab4e98b2b0e..cb11421f9cb8 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c @@ -2749,11 +2749,10 @@ static int hpsa_do_reset(struct ctlr_info *h, struct hpsa_scsi_dev_t *dev, lockup_detected(h)); if (unlikely(lockup_detected(h))) { - dev_warn(&h->pdev->dev, - "Controller lockup detected during reset wait\n"); - mutex_unlock(&h->reset_mutex); - rc = -ENODEV; - } + dev_warn(&h->pdev->dev, + "Controller lockup detected during reset wait\n"); + rc = -ENODEV; + } if (unlikely(rc)) atomic_set(&dev->reset_cmds_out, 0); |