malhuda/msm-4.14
1
0
Fork 0
mirror of https://github.com/rd-stuffs/msm-4.14.git synced 2025-02-20 11:45:48 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

blk-mq: Directly schedule q->timeout_work when aborting a request

Browse Source 
Request abortion is performed by overriding deadline to now and
scheduling timeout handling immediately.  For the latter part, the
code was using mod_timer(timeout, 0) which can't guarantee that the
timer runs afterwards.  Let's schedule the underlying work item
directly instead.

This fixes the hangs during probing reported by Sitsofe but it isn't
yet clear to me how the failure can happen reliably if it's just the
above described race condition.

Signed-off-by: Tejun Heo <tj@kernel.org>
Reported-by: Sitsofe Wheeler <sitsofe@gmail.com>
Reported-by: Meelis Roos <mroos@linux.ee>
Fixes: 358f70da49d7 ("blk-mq: make blk_abort_request() trigger timeout path")
Cc: stable@vger.kernel.org # v4.16
Link: http://lkml.kernel.org/r/CALjAwxh-PVYFnYFCJpGOja+m5SzZ8Sa4J7ohxdK=r8NyOF-EMA@mail.gmail.com
Link: http://lkml.kernel.org/r/alpine.LRH.2.21.1802261049140.4893@math.ut.ee
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: azrim <mirzaspc@gmail.com>
This commit is contained in:
Tejun Heo 2018-04-02 15:04:58 -07:00 committed by azrim
parent 9dbda712da
commit aff7775cc5
No known key found for this signature in database
GPG Key ID: 497F8FB059B45D1C
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
block/blk-timeout.c
View File

@ -163,7 +163,7 @@ void blk_abort_request(struct request *req)
* No need for fancy synchronizations. * No need for fancy synchronizations.
*/ */
blk_rq_set_deadline(req, jiffies); blk_rq_set_deadline(req, jiffies);
mod_timer(&req->q->timeout, 0); kblockd_schedule_work(&req->q->timeout_work);
} else { } else {
if (blk_mark_rq_complete(req)) if (blk_mark_rq_complete(req))
return; return;