diff options
| author | Jens Axboe <axboe@kernel.dk> | 2018-11-16 16:37:34 +0100 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2018-11-19 16:34:49 +0100 |
| commit | 849a370016a5489c49253338507ee6cc4a08df4b (patch) | |
| tree | 3e8794bb4312b0fa73e3092849b8b6e163437a2d /mm/page_io.c | |
| parent | nvme: default to 0 poll queues (diff) | |
| download | linux-849a370016a5489c49253338507ee6cc4a08df4b.tar.xz linux-849a370016a5489c49253338507ee6cc4a08df4b.zip | |
block: avoid ordered task state change for polled IO
For the core poll helper, the task state setting don't need to imply any
atomics, as it's the current task itself that is being modified and
we're not going to sleep.
For IRQ driven, the wakeup path have the necessary barriers to not need
us using the heavy handed version of the task state setting.
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'mm/page_io.c')
| -rw-r--r-- | mm/page_io.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mm/page_io.c b/mm/page_io.c index 57572ff46016..a7271fa481f6 100644 --- a/mm/page_io.c +++ b/mm/page_io.c @@ -405,7 +405,8 @@ int swap_readpage(struct page *page, bool synchronous) bio_get(bio); qc = submit_bio(bio); while (synchronous) { - set_current_state(TASK_UNINTERRUPTIBLE); + __set_current_state(TASK_UNINTERRUPTIBLE); + if (!READ_ONCE(bio->bi_private)) break; |