[PATCH 03/47] block: defer timeouts to a workqueue
Jeff Moyer
jmoyer at redhat.com
Mon Nov 23 12:59:49 PST 2015
Christoph Hellwig <hch at lst.de> writes:
> On Mon, Nov 23, 2015 at 03:31:32PM -0500, Jeff Moyer wrote:
>> I don't see where the blk-mq timeout work is ever scheduled. You
>> removed the call to setup_timer for the mq case, so what causes the mq
>> timeout work to run? I must be missing something.
>
> No we use the timer which we initialize in blk_alloc_queue_node instead
> of overwriting it later in the blk-mq case. The timer gets added in
> blk_add_timer either way.
Ah, I see. I missed the call to blk_alloc_queue_node from
blk_mq_init_queue. Thanks.
The patch looks good to me. Everything will still be running in the
same context (irqs off).
Reviewed-by: Jeff Moyer <jmoyer at redhat.com>
More information about the Linux-nvme
mailing list