diff options
author | Josef Bacik <jbacik@fusionio.com> | 2013-02-11 11:33:00 -0500 |
---|---|---|
committer | Josef Bacik <jbacik@fusionio.com> | 2013-02-20 12:59:52 -0500 |
commit | c8f2f24bd56a9f0bc7372e6d6aded6fc122b9120 (patch) | |
tree | 61c3e67ca47eca0740d9bdb6c5af593d02b64990 /fs/btrfs | |
parent | 210549ebe9047ae5a8cc47487203d3ee16a7749b (diff) |
Btrfs: remove unused extent io tree ops V2
Nobody uses these io tree ops anymore so just remove them and clean up the code
a bit. Thanks,
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'fs/btrfs')
-rw-r--r-- | fs/btrfs/extent_io.c | 35 | ||||
-rw-r--r-- | fs/btrfs/extent_io.h | 3 |
2 files changed, 11 insertions, 27 deletions
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c index b9019c847c58..5c00d6aeae75 100644 --- a/fs/btrfs/extent_io.c +++ b/fs/btrfs/extent_io.c | |||
@@ -2646,6 +2646,8 @@ static int __extent_read_full_page(struct extent_io_tree *tree, | |||
2646 | } | 2646 | } |
2647 | } | 2647 | } |
2648 | while (cur <= end) { | 2648 | while (cur <= end) { |
2649 | unsigned long pnr = (last_byte >> PAGE_CACHE_SHIFT) + 1; | ||
2650 | |||
2649 | if (cur >= last_byte) { | 2651 | if (cur >= last_byte) { |
2650 | char *userpage; | 2652 | char *userpage; |
2651 | struct extent_state *cached = NULL; | 2653 | struct extent_state *cached = NULL; |
@@ -2733,26 +2735,17 @@ static int __extent_read_full_page(struct extent_io_tree *tree, | |||
2733 | continue; | 2735 | continue; |
2734 | } | 2736 | } |
2735 | 2737 | ||
2736 | ret = 0; | 2738 | pnr -= page->index; |
2737 | if (tree->ops && tree->ops->readpage_io_hook) { | 2739 | ret = submit_extent_page(READ, tree, page, |
2738 | ret = tree->ops->readpage_io_hook(page, cur, | ||
2739 | cur + iosize - 1); | ||
2740 | } | ||
2741 | if (!ret) { | ||
2742 | unsigned long pnr = (last_byte >> PAGE_CACHE_SHIFT) + 1; | ||
2743 | pnr -= page->index; | ||
2744 | ret = submit_extent_page(READ, tree, page, | ||
2745 | sector, disk_io_size, pg_offset, | 2740 | sector, disk_io_size, pg_offset, |
2746 | bdev, bio, pnr, | 2741 | bdev, bio, pnr, |
2747 | end_bio_extent_readpage, mirror_num, | 2742 | end_bio_extent_readpage, mirror_num, |
2748 | *bio_flags, | 2743 | *bio_flags, |
2749 | this_bio_flag); | 2744 | this_bio_flag); |
2750 | if (!ret) { | 2745 | if (!ret) { |
2751 | nr++; | 2746 | nr++; |
2752 | *bio_flags = this_bio_flag; | 2747 | *bio_flags = this_bio_flag; |
2753 | } | 2748 | } else { |
2754 | } | ||
2755 | if (ret) { | ||
2756 | SetPageError(page); | 2749 | SetPageError(page); |
2757 | unlock_extent(tree, cur, cur + iosize - 1); | 2750 | unlock_extent(tree, cur, cur + iosize - 1); |
2758 | } | 2751 | } |
@@ -3441,15 +3434,9 @@ retry: | |||
3441 | * swizzled back from swapper_space to tmpfs file | 3434 | * swizzled back from swapper_space to tmpfs file |
3442 | * mapping | 3435 | * mapping |
3443 | */ | 3436 | */ |
3444 | if (tree->ops && | 3437 | if (!trylock_page(page)) { |
3445 | tree->ops->write_cache_pages_lock_hook) { | 3438 | flush_fn(data); |
3446 | tree->ops->write_cache_pages_lock_hook(page, | 3439 | lock_page(page); |
3447 | data, flush_fn); | ||
3448 | } else { | ||
3449 | if (!trylock_page(page)) { | ||
3450 | flush_fn(data); | ||
3451 | lock_page(page); | ||
3452 | } | ||
3453 | } | 3440 | } |
3454 | 3441 | ||
3455 | if (unlikely(page->mapping != mapping)) { | 3442 | if (unlikely(page->mapping != mapping)) { |
diff --git a/fs/btrfs/extent_io.h b/fs/btrfs/extent_io.h index 2eacfabd3263..ff182322d112 100644 --- a/fs/btrfs/extent_io.h +++ b/fs/btrfs/extent_io.h | |||
@@ -75,7 +75,6 @@ struct extent_io_ops { | |||
75 | int (*merge_bio_hook)(struct page *page, unsigned long offset, | 75 | int (*merge_bio_hook)(struct page *page, unsigned long offset, |
76 | size_t size, struct bio *bio, | 76 | size_t size, struct bio *bio, |
77 | unsigned long bio_flags); | 77 | unsigned long bio_flags); |
78 | int (*readpage_io_hook)(struct page *page, u64 start, u64 end); | ||
79 | int (*readpage_io_failed_hook)(struct page *page, int failed_mirror); | 78 | int (*readpage_io_failed_hook)(struct page *page, int failed_mirror); |
80 | int (*readpage_end_io_hook)(struct page *page, u64 start, u64 end, | 79 | int (*readpage_end_io_hook)(struct page *page, u64 start, u64 end, |
81 | struct extent_state *state, int mirror); | 80 | struct extent_state *state, int mirror); |
@@ -90,8 +89,6 @@ struct extent_io_ops { | |||
90 | struct extent_state *other); | 89 | struct extent_state *other); |
91 | void (*split_extent_hook)(struct inode *inode, | 90 | void (*split_extent_hook)(struct inode *inode, |
92 | struct extent_state *orig, u64 split); | 91 | struct extent_state *orig, u64 split); |
93 | int (*write_cache_pages_lock_hook)(struct page *page, void *data, | ||
94 | void (*flush_fn)(void *)); | ||
95 | }; | 92 | }; |
96 | 93 | ||
97 | struct extent_io_tree { | 94 | struct extent_io_tree { |