block/mq: do not invoke preempt_disable()
preempt_disable() and get_cpu() don't play well together with the sleeping locks it tries to allocate later. It seems to be enough to replace it with get_cpu_light() and migrate_disable(). Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
This commit is contained in:
parent
457d5cccfb
commit
4abe6b99f0
|
@ -611,7 +611,7 @@ static void __blk_mq_complete_request(struct request *rq)
|
|||
return;
|
||||
}
|
||||
|
||||
cpu = get_cpu();
|
||||
cpu = get_cpu_light();
|
||||
if (!test_bit(QUEUE_FLAG_SAME_FORCE, &q->queue_flags))
|
||||
shared = cpus_share_cache(cpu, ctx->cpu);
|
||||
|
||||
|
@ -623,7 +623,7 @@ static void __blk_mq_complete_request(struct request *rq)
|
|||
} else {
|
||||
q->mq_ops->complete(rq);
|
||||
}
|
||||
put_cpu();
|
||||
put_cpu_light();
|
||||
}
|
||||
|
||||
static void hctx_unlock(struct blk_mq_hw_ctx *hctx, int srcu_idx)
|
||||
|
@ -1487,14 +1487,14 @@ static void __blk_mq_delay_run_hw_queue(struct blk_mq_hw_ctx *hctx, bool async,
|
|||
return;
|
||||
|
||||
if (!async && !(hctx->flags & BLK_MQ_F_BLOCKING)) {
|
||||
int cpu = get_cpu();
|
||||
int cpu = get_cpu_light();
|
||||
if (cpumask_test_cpu(cpu, hctx->cpumask)) {
|
||||
__blk_mq_run_hw_queue(hctx);
|
||||
put_cpu();
|
||||
put_cpu_light();
|
||||
return;
|
||||
}
|
||||
|
||||
put_cpu();
|
||||
put_cpu_light();
|
||||
}
|
||||
|
||||
kblockd_mod_delayed_work_on(blk_mq_hctx_next_cpu(hctx), &hctx->run_work,
|
||||
|
|
Loading…
Reference in New Issue