summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@lst.de>2007-09-10 20:02:33 -0400
committerDavid Woodhouse <dwmw2@hera.kernel.org>2007-09-10 20:02:33 -0400
commit0e2752a72cb37075b24899f01e9bc6a589de3b6c (patch)
tree63bd02f3ba81018ffa0178f6f0982defd33b1d99
parentb06355f0fe04a62d08238438654ec453e05304ba (diff)
downloadop-kernel-dev-0e2752a72cb37075b24899f01e9bc6a589de3b6c.zip
op-kernel-dev-0e2752a72cb37075b24899f01e9bc6a589de3b6c.tar.gz
Btrfs: [PATCH] extent_map: add writepage_end_io hook
XFS updates the ondisk inode size only after the data I/O has finished, so it needs a hook when the writepage end_bio handler has finished. Might not be worth applying as-is as the per-page callback is very ineffcient. What XFS really wants is a callback when writeout of a whole extent has completed. This delayed i_size updates scheme might be worthwile for btrfs aswell, btw. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Chris Mason <chris.mason@oracle.com>
-rw-r--r--fs/btrfs/extent_map.c2
-rw-r--r--fs/btrfs/extent_map.h1
2 files changed, 3 insertions, 0 deletions
diff --git a/fs/btrfs/extent_map.c b/fs/btrfs/extent_map.c
index 1627667..ab5bde31 100644
--- a/fs/btrfs/extent_map.c
+++ b/fs/btrfs/extent_map.c
@@ -1231,6 +1231,8 @@ static int end_bio_extent_writepage(struct bio *bio,
end_page_writeback(page);
else
check_page_writeback(tree, page);
+ if (tree->ops && tree->ops->writepage_end_io_hook)
+ tree->ops->writepage_end_io_hook(page, start, end);
} while (bvec >= bio->bi_io_vec);
bio_put(bio);
diff --git a/fs/btrfs/extent_map.h b/fs/btrfs/extent_map.h
index da41aca..40b53ee2 100644
--- a/fs/btrfs/extent_map.h
+++ b/fs/btrfs/extent_map.h
@@ -11,6 +11,7 @@ struct extent_map_ops {
int (*writepage_io_hook)(struct page *page, u64 start, u64 end);
int (*readpage_io_hook)(struct page *page, u64 start, u64 end);
int (*readpage_end_io_hook)(struct page *page, u64 start, u64 end);
+ void (*writepage_end_io_hook)(struct page *page, u64 start, u64 end);
};
struct extent_map_tree {
OpenPOWER on IntegriCloud