diff options
| author | Nick Piggin <npiggin@suse.de> | 2009-04-15 10:32:07 +0200 |
|---|---|---|
| committer | Jens Axboe <jens.axboe@oracle.com> | 2009-04-15 12:10:13 +0200 |
| commit | c2572f2b4ffc27ba79211aceee3bef53a59bb5cd (patch) | |
| tree | 8c6680ad7778659c447c2a60b8565490b0d6f427 | |
| parent | dfbc4752eab33e66f113f9daa2effbe241cd661d (diff) | |
| download | op-kernel-dev-c2572f2b4ffc27ba79211aceee3bef53a59bb5cd.zip op-kernel-dev-c2572f2b4ffc27ba79211aceee3bef53a59bb5cd.tar.gz | |
brd: fix cacheflushing
brd is missing a flush_dcache_page. On 2nd thoughts, perhaps it is the
pagecache's responsibility to flush user virtual aliases (the driver of
course should flush kernel virtual mappings)... but anyway, there
already exists cache flushing for one direction of transfer, so we
should add the other.
Signed-off-by: Nick Piggin <npiggin@suse.de>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
| -rw-r--r-- | drivers/block/brd.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/block/brd.c b/drivers/block/brd.c index ee8ed68..5f7e64b 100644 --- a/drivers/block/brd.c +++ b/drivers/block/brd.c @@ -275,8 +275,10 @@ static int brd_do_bvec(struct brd_device *brd, struct page *page, if (rw == READ) { copy_from_brd(mem + off, brd, sector, len); flush_dcache_page(page); - } else + } else { + flush_dcache_page(page); copy_to_brd(brd, mem + off, sector, len); + } kunmap_atomic(mem, KM_USER0); out: |
