Re: [patch] target: lock => unlock typo

["Nicholas A. Bellinger" <nab@xxxxxxxxxxxxxxx>]

On Tue, 2012-05-08 at 13:28 +0300, Dan Carpenter wrote:
> target_stop_cmd() returns with the lock held and IRQs disabled.  The
> intent was to unlock here.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> 
> diff --git a/drivers/target/target_core_transport.c b/drivers/target/target_core_transport.c
> index 0d838ab..8bd9ffe 100644
> --- a/drivers/target/target_core_transport.c
> +++ b/drivers/target/target_core_transport.c
> @@ -3769,7 +3769,7 @@ static int transport_lun_wait_for_tasks(struct se_cmd *cmd, struct se_lun *lun)
>  	    (cmd->transport_state & CMD_T_SENT)) {
>  		if (!target_stop_cmd(cmd, &flags))
>  			ret++;
> -		spin_lock_irqsave(&cmd->t_state_lock, flags);
> +		spin_unlock_irqrestore(&cmd->t_state_lock, flags);
>  	} else {
>  		spin_unlock_irqrestore(&cmd->t_state_lock,
>  				flags);
> --

Applied to lio-core, and folding into hch's original commit that
introduced the bug in for-next.

Thanks Dan!

--nab

--
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