diff options
author | Josef Bacik <josef@redhat.com> | 2012-05-02 14:00:54 -0400 |
---|---|---|
committer | Josef Bacik <josef@redhat.com> | 2012-05-30 10:23:33 -0400 |
commit | 5fd02043553b02867b29de1ac9fff2ec16b84def (patch) | |
tree | f378b1042b4fbd366185b8b12b082bce4fd4fac4 /fs/btrfs/disk-io.c | |
parent | 4e89915220e2f1341c757b610d0f0c3821f3a65f (diff) |
Btrfs: finish ordered extents in their own thread
We noticed that the ordered extent completion doesn't really rely on having
a page and that it could be done independantly of ending the writeback on a
page. This patch makes us not do the threaded endio stuff for normal
buffered writes and direct writes so we can end page writeback as soon as
possible (in irq context) and only start threads to do the ordered work when
it is actually done. Compression needs to be reworked some to take
advantage of this as well, but atm it has to do a find_get_page in its endio
handler so it must be done in its own thread. This makes direct writes
quite a bit faster. Thanks,
Signed-off-by: Josef Bacik <josef@redhat.com>
Diffstat (limited to 'fs/btrfs/disk-io.c')
-rw-r--r-- | fs/btrfs/disk-io.c | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index a7ffc88a7dbe..19f5b450f405 100644 --- a/fs/btrfs/disk-io.c +++ b/fs/btrfs/disk-io.c | |||
@@ -3671,17 +3671,6 @@ int btrfs_cleanup_transaction(struct btrfs_root *root) | |||
3671 | return 0; | 3671 | return 0; |
3672 | } | 3672 | } |
3673 | 3673 | ||
3674 | static int btree_writepage_io_failed_hook(struct bio *bio, struct page *page, | ||
3675 | u64 start, u64 end, | ||
3676 | struct extent_state *state) | ||
3677 | { | ||
3678 | struct super_block *sb = page->mapping->host->i_sb; | ||
3679 | struct btrfs_fs_info *fs_info = btrfs_sb(sb); | ||
3680 | btrfs_error(fs_info, -EIO, | ||
3681 | "Error occured while writing out btree at %llu", start); | ||
3682 | return -EIO; | ||
3683 | } | ||
3684 | |||
3685 | static struct extent_io_ops btree_extent_io_ops = { | 3674 | static struct extent_io_ops btree_extent_io_ops = { |
3686 | .write_cache_pages_lock_hook = btree_lock_page_hook, | 3675 | .write_cache_pages_lock_hook = btree_lock_page_hook, |
3687 | .readpage_end_io_hook = btree_readpage_end_io_hook, | 3676 | .readpage_end_io_hook = btree_readpage_end_io_hook, |
@@ -3689,5 +3678,4 @@ static struct extent_io_ops btree_extent_io_ops = { | |||
3689 | .submit_bio_hook = btree_submit_bio_hook, | 3678 | .submit_bio_hook = btree_submit_bio_hook, |
3690 | /* note we're sharing with inode.c for the merge bio hook */ | 3679 | /* note we're sharing with inode.c for the merge bio hook */ |
3691 | .merge_bio_hook = btrfs_merge_bio_hook, | 3680 | .merge_bio_hook = btrfs_merge_bio_hook, |
3692 | .writepage_io_failed_hook = btree_writepage_io_failed_hook, | ||
3693 | }; | 3681 | }; |