diff options
author | Jan Kara <jack@suse.cz> | 2016-11-04 13:08:15 -0400 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2016-11-04 16:34:47 -0400 |
commit | e64855c6cfaa0a80c1b71c5f647cb792dc436668 (patch) | |
tree | 751d56f17914506a5e54a380f9a4d4519cadfa18 /fs/buffer.c | |
parent | 69a9bea146b185be8ec50e80eaecd8e487e689f8 (diff) |
fs: Add helper to clean bdev aliases under a bh and use it
Add a helper function that clears buffer heads from a block device
aliasing passed bh. Use this helper function from filesystems instead of
the original unmap_underlying_metadata() to save some boiler plate code
and also have a better name for the functionalily since it is not
unmapping anything for a *long* time.
Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'fs/buffer.c')
-rw-r--r-- | fs/buffer.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/fs/buffer.c b/fs/buffer.c index f8beca55240a..912d70169fca 100644 --- a/fs/buffer.c +++ b/fs/buffer.c | |||
@@ -1821,8 +1821,7 @@ int __block_write_full_page(struct inode *inode, struct page *page, | |||
1821 | if (buffer_new(bh)) { | 1821 | if (buffer_new(bh)) { |
1822 | /* blockdev mappings never come here */ | 1822 | /* blockdev mappings never come here */ |
1823 | clear_buffer_new(bh); | 1823 | clear_buffer_new(bh); |
1824 | unmap_underlying_metadata(bh->b_bdev, | 1824 | clean_bdev_bh_alias(bh); |
1825 | bh->b_blocknr); | ||
1826 | } | 1825 | } |
1827 | } | 1826 | } |
1828 | bh = bh->b_this_page; | 1827 | bh = bh->b_this_page; |
@@ -2068,8 +2067,7 @@ int __block_write_begin_int(struct page *page, loff_t pos, unsigned len, | |||
2068 | } | 2067 | } |
2069 | 2068 | ||
2070 | if (buffer_new(bh)) { | 2069 | if (buffer_new(bh)) { |
2071 | unmap_underlying_metadata(bh->b_bdev, | 2070 | clean_bdev_bh_alias(bh); |
2072 | bh->b_blocknr); | ||
2073 | if (PageUptodate(page)) { | 2071 | if (PageUptodate(page)) { |
2074 | clear_buffer_new(bh); | 2072 | clear_buffer_new(bh); |
2075 | set_buffer_uptodate(bh); | 2073 | set_buffer_uptodate(bh); |
@@ -2709,7 +2707,7 @@ int nobh_write_begin(struct address_space *mapping, | |||
2709 | if (!buffer_mapped(bh)) | 2707 | if (!buffer_mapped(bh)) |
2710 | is_mapped_to_disk = 0; | 2708 | is_mapped_to_disk = 0; |
2711 | if (buffer_new(bh)) | 2709 | if (buffer_new(bh)) |
2712 | unmap_underlying_metadata(bh->b_bdev, bh->b_blocknr); | 2710 | clean_bdev_bh_alias(bh); |
2713 | if (PageUptodate(page)) { | 2711 | if (PageUptodate(page)) { |
2714 | set_buffer_uptodate(bh); | 2712 | set_buffer_uptodate(bh); |
2715 | continue; | 2713 | continue; |