diff options
author | Jeff Mahoney <jeffm@suse.com> | 2012-03-01 08:57:19 -0500 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2012-03-21 20:45:35 -0400 |
commit | d0082371cf086e0ba2bbd0367b2c9920532df24f (patch) | |
tree | 4ae3fb6565f36fd606ab6e4b83d489e66b3b86d7 /fs/btrfs/inode.c | |
parent | 143bede527b054a271053f41bfaca2b57baa9408 (diff) |
btrfs: drop gfp_t from lock_extent
lock_extent and unlock_extent are always called with GFP_NOFS, drop the
argument and use GFP_NOFS consistently.
Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Diffstat (limited to 'fs/btrfs/inode.c')
-rw-r--r-- | fs/btrfs/inode.c | 33 |
1 files changed, 14 insertions, 19 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index dc800eefdbc7..d16bf3f5da0e 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c | |||
@@ -597,7 +597,7 @@ retry: | |||
597 | 597 | ||
598 | lock_extent(io_tree, async_extent->start, | 598 | lock_extent(io_tree, async_extent->start, |
599 | async_extent->start + | 599 | async_extent->start + |
600 | async_extent->ram_size - 1, GFP_NOFS); | 600 | async_extent->ram_size - 1); |
601 | 601 | ||
602 | /* allocate blocks */ | 602 | /* allocate blocks */ |
603 | ret = cow_file_range(inode, async_cow->locked_page, | 603 | ret = cow_file_range(inode, async_cow->locked_page, |
@@ -625,8 +625,7 @@ retry: | |||
625 | } | 625 | } |
626 | 626 | ||
627 | lock_extent(io_tree, async_extent->start, | 627 | lock_extent(io_tree, async_extent->start, |
628 | async_extent->start + async_extent->ram_size - 1, | 628 | async_extent->start + async_extent->ram_size - 1); |
629 | GFP_NOFS); | ||
630 | 629 | ||
631 | trans = btrfs_join_transaction(root); | 630 | trans = btrfs_join_transaction(root); |
632 | BUG_ON(IS_ERR(trans)); | 631 | BUG_ON(IS_ERR(trans)); |
@@ -649,7 +648,7 @@ retry: | |||
649 | async_extent->pages = NULL; | 648 | async_extent->pages = NULL; |
650 | unlock_extent(io_tree, async_extent->start, | 649 | unlock_extent(io_tree, async_extent->start, |
651 | async_extent->start + | 650 | async_extent->start + |
652 | async_extent->ram_size - 1, GFP_NOFS); | 651 | async_extent->ram_size - 1); |
653 | goto retry; | 652 | goto retry; |
654 | } | 653 | } |
655 | 654 | ||
@@ -1574,7 +1573,7 @@ again: | |||
1574 | page_end = page_offset(page) + PAGE_CACHE_SIZE - 1; | 1573 | page_end = page_offset(page) + PAGE_CACHE_SIZE - 1; |
1575 | 1574 | ||
1576 | lock_extent_bits(&BTRFS_I(inode)->io_tree, page_start, page_end, 0, | 1575 | lock_extent_bits(&BTRFS_I(inode)->io_tree, page_start, page_end, 0, |
1577 | &cached_state, GFP_NOFS); | 1576 | &cached_state); |
1578 | 1577 | ||
1579 | /* already ordered? We're done */ | 1578 | /* already ordered? We're done */ |
1580 | if (PagePrivate2(page)) | 1579 | if (PagePrivate2(page)) |
@@ -1765,7 +1764,7 @@ static int btrfs_finish_ordered_io(struct inode *inode, u64 start, u64 end) | |||
1765 | 1764 | ||
1766 | lock_extent_bits(io_tree, ordered_extent->file_offset, | 1765 | lock_extent_bits(io_tree, ordered_extent->file_offset, |
1767 | ordered_extent->file_offset + ordered_extent->len - 1, | 1766 | ordered_extent->file_offset + ordered_extent->len - 1, |
1768 | 0, &cached_state, GFP_NOFS); | 1767 | 0, &cached_state); |
1769 | 1768 | ||
1770 | if (nolock) | 1769 | if (nolock) |
1771 | trans = btrfs_join_transaction_nolock(root); | 1770 | trans = btrfs_join_transaction_nolock(root); |
@@ -3285,8 +3284,7 @@ again: | |||
3285 | } | 3284 | } |
3286 | wait_on_page_writeback(page); | 3285 | wait_on_page_writeback(page); |
3287 | 3286 | ||
3288 | lock_extent_bits(io_tree, page_start, page_end, 0, &cached_state, | 3287 | lock_extent_bits(io_tree, page_start, page_end, 0, &cached_state); |
3289 | GFP_NOFS); | ||
3290 | set_page_extent_mapped(page); | 3288 | set_page_extent_mapped(page); |
3291 | 3289 | ||
3292 | ordered = btrfs_lookup_ordered_extent(inode, page_start); | 3290 | ordered = btrfs_lookup_ordered_extent(inode, page_start); |
@@ -3362,7 +3360,7 @@ int btrfs_cont_expand(struct inode *inode, loff_t oldsize, loff_t size) | |||
3362 | btrfs_wait_ordered_range(inode, hole_start, | 3360 | btrfs_wait_ordered_range(inode, hole_start, |
3363 | block_end - hole_start); | 3361 | block_end - hole_start); |
3364 | lock_extent_bits(io_tree, hole_start, block_end - 1, 0, | 3362 | lock_extent_bits(io_tree, hole_start, block_end - 1, 0, |
3365 | &cached_state, GFP_NOFS); | 3363 | &cached_state); |
3366 | ordered = btrfs_lookup_ordered_extent(inode, hole_start); | 3364 | ordered = btrfs_lookup_ordered_extent(inode, hole_start); |
3367 | if (!ordered) | 3365 | if (!ordered) |
3368 | break; | 3366 | break; |
@@ -5604,7 +5602,7 @@ static int btrfs_get_blocks_direct(struct inode *inode, sector_t iblock, | |||
5604 | free_extent_map(em); | 5602 | free_extent_map(em); |
5605 | /* DIO will do one hole at a time, so just unlock a sector */ | 5603 | /* DIO will do one hole at a time, so just unlock a sector */ |
5606 | unlock_extent(&BTRFS_I(inode)->io_tree, start, | 5604 | unlock_extent(&BTRFS_I(inode)->io_tree, start, |
5607 | start + root->sectorsize - 1, GFP_NOFS); | 5605 | start + root->sectorsize - 1); |
5608 | return 0; | 5606 | return 0; |
5609 | } | 5607 | } |
5610 | 5608 | ||
@@ -5745,7 +5743,7 @@ static void btrfs_endio_direct_read(struct bio *bio, int err) | |||
5745 | } while (bvec <= bvec_end); | 5743 | } while (bvec <= bvec_end); |
5746 | 5744 | ||
5747 | unlock_extent(&BTRFS_I(inode)->io_tree, dip->logical_offset, | 5745 | unlock_extent(&BTRFS_I(inode)->io_tree, dip->logical_offset, |
5748 | dip->logical_offset + dip->bytes - 1, GFP_NOFS); | 5746 | dip->logical_offset + dip->bytes - 1); |
5749 | bio->bi_private = dip->private; | 5747 | bio->bi_private = dip->private; |
5750 | 5748 | ||
5751 | kfree(dip->csums); | 5749 | kfree(dip->csums); |
@@ -5796,7 +5794,7 @@ again: | |||
5796 | 5794 | ||
5797 | lock_extent_bits(&BTRFS_I(inode)->io_tree, ordered->file_offset, | 5795 | lock_extent_bits(&BTRFS_I(inode)->io_tree, ordered->file_offset, |
5798 | ordered->file_offset + ordered->len - 1, 0, | 5796 | ordered->file_offset + ordered->len - 1, 0, |
5799 | &cached_state, GFP_NOFS); | 5797 | &cached_state); |
5800 | 5798 | ||
5801 | if (test_bit(BTRFS_ORDERED_PREALLOC, &ordered->flags)) { | 5799 | if (test_bit(BTRFS_ORDERED_PREALLOC, &ordered->flags)) { |
5802 | ret = btrfs_mark_extent_written(trans, inode, | 5800 | ret = btrfs_mark_extent_written(trans, inode, |
@@ -6211,7 +6209,7 @@ static ssize_t btrfs_direct_IO(int rw, struct kiocb *iocb, | |||
6211 | 6209 | ||
6212 | while (1) { | 6210 | while (1) { |
6213 | lock_extent_bits(&BTRFS_I(inode)->io_tree, lockstart, lockend, | 6211 | lock_extent_bits(&BTRFS_I(inode)->io_tree, lockstart, lockend, |
6214 | 0, &cached_state, GFP_NOFS); | 6212 | 0, &cached_state); |
6215 | /* | 6213 | /* |
6216 | * We're concerned with the entire range that we're going to be | 6214 | * We're concerned with the entire range that we're going to be |
6217 | * doing DIO to, so we need to make sure theres no ordered | 6215 | * doing DIO to, so we need to make sure theres no ordered |
@@ -6365,8 +6363,7 @@ static void btrfs_invalidatepage(struct page *page, unsigned long offset) | |||
6365 | btrfs_releasepage(page, GFP_NOFS); | 6363 | btrfs_releasepage(page, GFP_NOFS); |
6366 | return; | 6364 | return; |
6367 | } | 6365 | } |
6368 | lock_extent_bits(tree, page_start, page_end, 0, &cached_state, | 6366 | lock_extent_bits(tree, page_start, page_end, 0, &cached_state); |
6369 | GFP_NOFS); | ||
6370 | ordered = btrfs_lookup_ordered_extent(page->mapping->host, | 6367 | ordered = btrfs_lookup_ordered_extent(page->mapping->host, |
6371 | page_offset(page)); | 6368 | page_offset(page)); |
6372 | if (ordered) { | 6369 | if (ordered) { |
@@ -6388,8 +6385,7 @@ static void btrfs_invalidatepage(struct page *page, unsigned long offset) | |||
6388 | } | 6385 | } |
6389 | btrfs_put_ordered_extent(ordered); | 6386 | btrfs_put_ordered_extent(ordered); |
6390 | cached_state = NULL; | 6387 | cached_state = NULL; |
6391 | lock_extent_bits(tree, page_start, page_end, 0, &cached_state, | 6388 | lock_extent_bits(tree, page_start, page_end, 0, &cached_state); |
6392 | GFP_NOFS); | ||
6393 | } | 6389 | } |
6394 | clear_extent_bit(tree, page_start, page_end, | 6390 | clear_extent_bit(tree, page_start, page_end, |
6395 | EXTENT_LOCKED | EXTENT_DIRTY | EXTENT_DELALLOC | | 6391 | EXTENT_LOCKED | EXTENT_DIRTY | EXTENT_DELALLOC | |
@@ -6464,8 +6460,7 @@ again: | |||
6464 | } | 6460 | } |
6465 | wait_on_page_writeback(page); | 6461 | wait_on_page_writeback(page); |
6466 | 6462 | ||
6467 | lock_extent_bits(io_tree, page_start, page_end, 0, &cached_state, | 6463 | lock_extent_bits(io_tree, page_start, page_end, 0, &cached_state); |
6468 | GFP_NOFS); | ||
6469 | set_page_extent_mapped(page); | 6464 | set_page_extent_mapped(page); |
6470 | 6465 | ||
6471 | /* | 6466 | /* |