aboutsummaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorJens Axboe <jaxboe@fusionio.com>2011-04-16 07:27:55 -0400
committerJens Axboe <jaxboe@fusionio.com>2011-04-16 07:27:55 -0400
commita237c1c5bc5dc5c76a21be922dca4826f3eca8ca (patch)
treea216c9a6d9e870b84424938e9e0b4722dc8634cd /include
parent5853b4f06f7b9b56f37f457d7923f7b96496074e (diff)
block: let io_schedule() flush the plug inline
Linus correctly observes that the most important dispatch cases are now done from kblockd, this isn't ideal for latency reasons. The original reason for switching dispatches out-of-line was to avoid too deep a stack, so by _only_ letting the "accidental" flush directly in schedule() be guarded by offload to kblockd, we should be able to get the best of both worlds. So add a blk_schedule_flush_plug() that offloads to kblockd, and only use that from the schedule() path. Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'include')
-rw-r--r--include/linux/blkdev.h13
1 files changed, 13 insertions, 0 deletions
diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h
index 1c76506fcf11..ec0357d8c4a5 100644
--- a/include/linux/blkdev.h
+++ b/include/linux/blkdev.h
@@ -872,6 +872,14 @@ static inline void blk_flush_plug(struct task_struct *tsk)
872 struct blk_plug *plug = tsk->plug; 872 struct blk_plug *plug = tsk->plug;
873 873
874 if (plug) 874 if (plug)
875 blk_flush_plug_list(plug, false);
876}
877
878static inline void blk_schedule_flush_plug(struct task_struct *tsk)
879{
880 struct blk_plug *plug = tsk->plug;
881
882 if (plug)
875 blk_flush_plug_list(plug, true); 883 blk_flush_plug_list(plug, true);
876} 884}
877 885
@@ -1317,6 +1325,11 @@ static inline void blk_flush_plug(struct task_struct *task)
1317{ 1325{
1318} 1326}
1319 1327
1328static inline void blk_schedule_flush_plug(struct task_struct *task)
1329{
1330}
1331
1332
1320static inline bool blk_needs_flush_plug(struct task_struct *tsk) 1333static inline bool blk_needs_flush_plug(struct task_struct *tsk)
1321{ 1334{
1322 return false; 1335 return false;