diff options
| author | Suganath prabu Subramani | 2016-02-11 15:02:55 +0530 |
|---|---|---|
| committer | Martin K. Petersen | 2016-02-23 21:27:02 -0500 |
| commit | da3cec2515f0094796679876ba17ba359331dbf6 (patch) | |
| tree | 49ef4b95534acb383bcc131ad9fecc3100d2fd4a | |
| parent | 1c2ba475eb0e574f96cf51c21157ffdf0af0dd2a (diff) | |
mpt3sas: Remove cpumask_clear for zalloc_cpumask_var and don't free free_cpu_mask_var before reply_q
Removed cpumask_clear as it is not required for zalloc_cpumask_var and
free free_cpumask_var before freeing reply_q.
Signed-off-by: Suganath prabu Subramani <suganath-prabu.subramani@broadcom.com>
Signed-off-by: Chaitra P B <chaitra.basappa@broadcom.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
| -rw-r--r-- | drivers/scsi/mpt3sas/mpt3sas_base.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.c b/drivers/scsi/mpt3sas/mpt3sas_base.c index c0a9d97b343f..afdb13acd3a2 100644 --- a/drivers/scsi/mpt3sas/mpt3sas_base.c +++ b/drivers/scsi/mpt3sas/mpt3sas_base.c @@ -1855,7 +1855,6 @@ _base_request_irq(struct MPT3SAS_ADAPTER *ioc, u8 index, u32 vector) kfree(reply_q); return -ENOMEM; } - cpumask_clear(reply_q->affinity_hint); } atomic_set(&reply_q->busy, 0); @@ -1870,8 +1869,8 @@ _base_request_irq(struct MPT3SAS_ADAPTER *ioc, u8 index, u32 vector) if (r) { pr_err(MPT3SAS_FMT "unable to allocate interrupt %d!\n", reply_q->name, vector); - kfree(reply_q); free_cpumask_var(reply_q->affinity_hint); + kfree(reply_q); return -EBUSY; } |