diff options
author | Ilya Loginov <isloginov@gmail.com> | 2009-11-26 03:16:19 -0500 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2009-11-26 03:16:19 -0500 |
commit | 2d4dc890b5c8fabd818a8586607e6843c4375e62 (patch) | |
tree | 9976ed7b0eed0056f8289aeb6a2b0abf8c940454 /block | |
parent | 3586e917f2c7df769d173c4ec99554cb40a911e5 (diff) |
block: add helpers to run flush_dcache_page() against a bio and a request's pages
Mtdblock driver doesn't call flush_dcache_page for pages in request. So,
this causes problems on architectures where the icache doesn't fill from
the dcache or with dcache aliases. The patch fixes this.
The ARCH_IMPLEMENTS_FLUSH_DCACHE_PAGE symbol was introduced to avoid
pointless empty cache-thrashing loops on architectures for which
flush_dcache_page() is a no-op. Every architecture was provided with this
flush pages on architectires where ARCH_IMPLEMENTS_FLUSH_DCACHE_PAGE is
equal 1 or do nothing otherwise.
See "fix mtd_blkdevs problem with caches on some architectures" discussion
on LKML for more information.
Signed-off-by: Ilya Loginov <isloginov@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Peter Horton <phorton@bitbox.co.uk>
Cc: "Ed L. Cashin" <ecashin@coraid.com>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'block')
-rw-r--r-- | block/blk-core.c | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/block/blk-core.c b/block/blk-core.c index 71da5111120c..718897e6d37f 100644 --- a/block/blk-core.c +++ b/block/blk-core.c | |||
@@ -2358,6 +2358,25 @@ void blk_rq_bio_prep(struct request_queue *q, struct request *rq, | |||
2358 | rq->rq_disk = bio->bi_bdev->bd_disk; | 2358 | rq->rq_disk = bio->bi_bdev->bd_disk; |
2359 | } | 2359 | } |
2360 | 2360 | ||
2361 | #if ARCH_IMPLEMENTS_FLUSH_DCACHE_PAGE | ||
2362 | /** | ||
2363 | * rq_flush_dcache_pages - Helper function to flush all pages in a request | ||
2364 | * @rq: the request to be flushed | ||
2365 | * | ||
2366 | * Description: | ||
2367 | * Flush all pages in @rq. | ||
2368 | */ | ||
2369 | void rq_flush_dcache_pages(struct request *rq) | ||
2370 | { | ||
2371 | struct req_iterator iter; | ||
2372 | struct bio_vec *bvec; | ||
2373 | |||
2374 | rq_for_each_segment(bvec, rq, iter) | ||
2375 | flush_dcache_page(bvec->bv_page); | ||
2376 | } | ||
2377 | EXPORT_SYMBOL_GPL(rq_flush_dcache_pages); | ||
2378 | #endif | ||
2379 | |||
2361 | /** | 2380 | /** |
2362 | * blk_lld_busy - Check if underlying low-level drivers of a device are busy | 2381 | * blk_lld_busy - Check if underlying low-level drivers of a device are busy |
2363 | * @q : the queue of the device being checked | 2382 | * @q : the queue of the device being checked |