]> git.baikalelectronics.ru Git - kernel.git/commit
block: defer flush request no matter whether we have elevator
authorYufen Yu <yuyufen@huawei.com>
Thu, 16 Jul 2020 06:52:01 +0000 (02:52 -0400)
committerJens Axboe <axboe@kernel.dk>
Fri, 17 Jul 2020 13:14:28 +0000 (07:14 -0600)
commit3633f30e69870a50852c3b6c556933a5572caab5
tree6737291920782b8fbaf65f97419c9d492a395326
parent3518421748287ae040bcba5121cabcade3cd0722
block: defer flush request no matter whether we have elevator

Commit cbebeb8d8974 ("block: don't defer flushes on blk-mq + scheduling")
tried to fix deadlock for cycled wait between flush requests and data
request into flush_data_in_flight. The former holded all driver tags
and wait for data request completion, but the latter can not complete
for waiting free driver tags.

After commit d57bcaccf9d9 ("blk-mq: don't allocate driver tag upfront
for flush rq"), flush requests will not get driver tag before queuing
into flush queue.

* With elevator, flush request just get sched_tags before inserting
  flush queue. It will not get driver tag until issue them to driver.
  data request on list fq->flush_data_in_flight will complete in
  the end.

* Without elevator, each flush request will get a driver tag when
  allocate request. Then data request on fq->flush_data_in_flight
  don't worry about lacking driver tag.

In both of these cases, cycled wait cannot be true. So we may allow
to defer flush request.

Signed-off-by: Yufen Yu <yuyufen@huawei.com>
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/blk-flush.c