This looks good:

Reviewed-by: Christoph Hellwig <h...@lst.de>

On Tue, Oct 18, 2016 at 02:51:33PM -0700, Bart Van Assche wrote:
>  static void dm_mq_start_queue(struct request_queue *q)
>  {
> -     unsigned long flags;
> -
> -     spin_lock_irqsave(q->queue_lock, flags);
> -     queue_flag_clear(QUEUE_FLAG_STOPPED, q);
> -     spin_unlock_irqrestore(q->queue_lock, flags);
> -
>       blk_mq_start_stopped_hw_queues(q, true);
>       blk_mq_kick_requeue_list(q);

FYI, I'm tempted to say we should always call blk_mq_kick_requeue_list
from blk_mq_start_stopped_hw_queues, but that's a separate issue.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to