Re: [patch] [SCSI] mptfusion: unlock on error in mpt_config()

[Dan Carpenter <error27@xxxxxxxxx>]

Ping?

regards,
dan carpenter

On 8/30/11, Nandigama, Nagalakshmi <Nagalakshmi.Nandigama@xxxxxxx> wrote:
> The patch seem to be fine. Please consider this patch as Acked-by:
> "Nandigama, Nagalakshmi" <Nagalakshmi.Nandigama@xxxxxxx>
>
>
> Regards,
> Nagalakshmi
>
>
> -----Original Message-----
> From: Dan Carpenter [mailto:error27@xxxxxxxxx]
> Sent: Saturday, August 27, 2011 3:30 PM
> To: Moore, Eric; Desai, Kashyap
> Cc: Support; DL-MPT Fusion Linux; open list:LSILOGIC MPT FUSI...;
> kernel-janitors@xxxxxxxxxxxxxxx
> Subject: [patch] [SCSI] mptfusion: unlock on error in mpt_config()
>
> We introduced a new return here and forgot to unlock the mutex.
>
> Signed-off-by: Dan Carpenter <error27@xxxxxxxxx>
>
> diff --git a/drivers/message/fusion/mptbase.c
> b/drivers/message/fusion/mptbase.c
> index e9c6a60..e72320e 100644
> --- a/drivers/message/fusion/mptbase.c
> +++ b/drivers/message/fusion/mptbase.c
> @@ -6482,6 +6482,7 @@ mpt_config(MPT_ADAPTER *ioc, CONFIGPARMS *pCfg)
>  				printk(MYIOC_s_INFO_FMT "%s: host reset in"
>  					" progress mpt_config timed out.!!\n",
>  					__func__, ioc->name);
> +				mutex_unlock(&ioc->mptbase_cmds.mutex);
>  				return -EFAULT;
>  			}
>  			spin_unlock_irqrestore(&ioc->taskmgmt_lock, flags);
>
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html