On Mon, Oct 12, 2015 at 10:10:49AM -0700, Bart Van Assche wrote:
> This patch avoids that the following kernel crash can occur with
> synchronous TMF handling:
So this is a preparation patch? Maybe this needs to be state more
clearly.
> void transport_free_session(struct se_session *se_sess)
> {
> + spin_lock_irq(&se_sess->sess_cmd_lock);
> + while (wait_event_interruptible_lock_irq(se_sess->cmd_list_wq,
> + list_empty(&se_sess->sess_cmd_list),
> + se_sess->sess_cmd_lock) < 0)
> + ;
> + spin_unlock_irq(&se_sess->sess_cmd_lock);
> +
Shouldn't this be a single uninterruptible wait?
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
