The patch titled
md: Change md/bitmap file handling to use bmap to file blocks-fix
has been added to the -mm tree. Its filename is
md-change-md-bitmap-file-handling-to-use-bmap-to-file-blocks-fix.patch
See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
out what to do about this
From: NeilBrown <neilb@xxxxxxx>
Fix problems with new bmap based access to bitmap files.
1/ When not using a file based bitmap, attach a NULL list of buffers
to each page so the common free_buffer routine can cope.
2/ Use submit_bh to read as well as write, rather than vfs_read.
This makes read and write more symetric.
3/ sync the file before reading, to ensure that the page cache has no
dirty pages that might get written out later.
Signed-off-by: Neil Brown <neilb@xxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
---
drivers/md/bitmap.c | 47 ++++++++++++++++++++----------------------
1 files changed, 23 insertions(+), 24 deletions(-)
diff -puN drivers/md/bitmap.c~md-change-md-bitmap-file-handling-to-use-bmap-to-file-blocks-fix drivers/md/bitmap.c
--- devel/drivers/md/bitmap.c~md-change-md-bitmap-file-handling-to-use-bmap-to-file-blocks-fix 2006-05-15 21:35:34.000000000 -0700
+++ devel-akpm/drivers/md/bitmap.c 2006-05-15 21:35:58.000000000 -0700
@@ -247,6 +247,8 @@ static struct page *read_sb_page(mddev_t
if (sync_page_io(rdev->bdev, target, PAGE_SIZE, page, READ)) {
page->index = index;
+ attach_page_buffers(page, NULL); /* so that free_buffer will
+ * quietly no-op */
return page;
}
}
@@ -349,11 +351,8 @@ static struct page *read_page(struct fil
{
struct page *page = NULL;
struct inode *inode = file->f_dentry->d_inode;
- loff_t pos = index << PAGE_SHIFT;
- int ret;
struct buffer_head *bh;
sector_t block;
- mm_segment_t oldfs;
PRINTK("read bitmap file (%dB @ %Lu)\n", (int)PAGE_SIZE,
(unsigned long long)index << PAGE_SHIFT);
@@ -364,18 +363,6 @@ static struct page *read_page(struct fil
if (IS_ERR(page))
goto out;
- oldfs = get_fs();
- set_fs(KERNEL_DS);
- ret = vfs_read(file, (char __user*) page_address(page), count, &pos);
- set_fs(oldfs);
-
- if (ret >= 0 && ret != count)
- ret = -EIO;
- if (ret < 0) {
- put_page(page);
- page = ERR_PTR(ret);
- goto out;
- }
bh = alloc_page_buffers(page, 1<<inode->i_blkbits, 0);
if (!bh) {
put_page(page);
@@ -403,12 +390,22 @@ static struct page *read_page(struct fil
bh->b_end_io = end_bitmap_write;
bh->b_private = bitmap;
+ atomic_inc(&bitmap->pending_writes);
+ set_buffer_locked(bh);
+ set_buffer_mapped(bh);
+ submit_bh(READ, bh);
}
block++;
bh = bh->b_this_page;
}
-
page->index = index;
+
+ wait_event(bitmap->write_wait,
+ atomic_read(&bitmap->pending_writes)==0);
+ if (bitmap->flags & BITMAP_WRITE_ERROR) {
+ free_buffers(page);
+ page = ERR_PTR(-EIO);
+ }
out:
if (IS_ERR(page))
printk(KERN_ALERT "md: bitmap read error: (%dB @ %Lu): %ld\n",
@@ -1415,15 +1412,20 @@ int bitmap_create(mddev_t *mddev)
return -ENOMEM;
spin_lock_init(&bitmap->lock);
+ atomic_set(&bitmap->pending_writes, 0);
+ init_waitqueue_head(&bitmap->write_wait);
+
bitmap->mddev = mddev;
bitmap->file = file;
bitmap->offset = mddev->bitmap_offset;
- if (file) get_file(file);
-
- /* Ensure we read fresh data */
- invalidate_inode_pages(file->f_dentry->d_inode->i_mapping);
-
+ if (file) {
+ get_file(file);
+ do_sync_file_range(file, 0, LLONG_MAX,
+ SYNC_FILE_RANGE_WAIT_BEFORE |
+ SYNC_FILE_RANGE_WRITE |
+ SYNC_FILE_RANGE_WAIT_AFTER);
+ }
/* read superblock from bitmap file (this sets bitmap->chunksize) */
err = bitmap_read_sb(bitmap);
if (err)
@@ -1446,9 +1448,6 @@ int bitmap_create(mddev_t *mddev)
bitmap->syncchunk = ~0UL;
- atomic_set(&bitmap->pending_writes, 0);
- init_waitqueue_head(&bitmap->write_wait);
-
#ifdef INJECT_FATAL_FAULT_1
bitmap->bp = NULL;
#else
_
Patches currently in -mm which might be from neilb@xxxxxxx are
nfs-fix-error-handling-on-access_ok-in-compat_sys_nfsservctl.patch
nfs-server-subtree_check-returns-dubious-value.patch
fix-dcache-race-during-umount.patch
fix-dcache-race-during-umount-fix.patch
prune_one_dentry-tweaks.patch
remove-softlockup-from-invalidate_mapping_pages.patch
kconfig-select-things-at-the-closest-tristate-instead-of-bool.patch
make-address_space_operations-invalidatepage-return-void-reiser4.patch
md-reformat-code-in-raid1_end_write_request-to-avoid-goto.patch
md-remove-arbitrary-limit-on-chunk-size.patch
md-remove-useless-ioctl-warning.patch
md-increase-the-delay-before-marking-metadata-clean-and-make-it-configurable.patch
md-merge-raid5-and-raid6-code.patch
md-remove-nuisance-message-at-shutdown.patch
md-allow-checkpoint-of-recovery-with-version-1-superblock.patch
md-allow-a-linear-array-to-have-drives-added-while-active.patch
md-support-stripe-offset-mode-in-raid10.patch
md-make-md_print_devices-static.patch
md-split-reshape-portion-of-raid5-sync_request-into-a-separate-function.patch
md-bitmap-fix-online-removal-of-file-backed-bitmaps.patch
md-bitmap-remove-bitmap-writeback-daemon.patch
md-bitmap-cleaner-separation-of-page-attribute-handlers-in-md-bitmap.patch
md-bitmap-use-set_bit-etc-for-bitmap-page-attributes.patch
md-bitmap-remove-unnecessary-page-reference-manipulations-from-md-bitmap-code.patch
md-bitmap-remove-dead-code-from-md-bitmap.patch
md-bitmap-tidy-up-i_writecount-handling-in-md-bitmap.patch
md-bitmap-change-md-bitmap-file-handling-to-use-bmap-to-file-blocks.patch
md-change-md-bitmap-file-handling-to-use-bmap-to-file-blocks-fix.patch
md-dm-reduce-stack-usage-with-stacked-block-devices.patch
-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
