> +             if (WARN_ON_ONCE(!sess))
> +                     continue;
> +
> +             spin_lock(&sess->sess_cmd_lock);
>               spin_lock(&cmd->t_state_lock);
>               if (!(cmd->transport_state & CMD_T_ACTIVE)) {
>                       spin_unlock(&cmd->t_state_lock);
> +                     spin_unlock(&sess->sess_cmd_lock);
>                       continue;
>               }
>               if (cmd->t_state == TRANSPORT_ISTATE_PROCESSING) {
>                       spin_unlock(&cmd->t_state_lock);
> +                     spin_unlock(&sess->sess_cmd_lock);
>                       continue;
>               }
> +             cmd->transport_state |= CMD_T_ABORTED;
>               spin_unlock(&cmd->t_state_lock);
>  
> +             rc = kref_get_unless_zero(&cmd->cmd_kref);
> +             spin_unlock(&sess->sess_cmd_lock);

Same here - what is the point of sess_cmd_lock?
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to