btrfs: merge two flush_write_bio helpers
flush_epd_write_bio is same as flush_write_bio, no point having two such functions. Merge them to flush_write_bio. The 'noinline' attribute is removed as it does not have any meaning. Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
parent
a74b35ec87
commit
e2932ee08e
@ -138,7 +138,8 @@ static void add_extent_changeset(struct extent_state *state, unsigned bits,
|
|||||||
BUG_ON(ret < 0);
|
BUG_ON(ret < 0);
|
||||||
}
|
}
|
||||||
|
|
||||||
static noinline void flush_write_bio(void *data);
|
static void flush_write_bio(void *data);
|
||||||
|
|
||||||
static inline struct btrfs_fs_info *
|
static inline struct btrfs_fs_info *
|
||||||
tree_fs_info(struct extent_io_tree *tree)
|
tree_fs_info(struct extent_io_tree *tree)
|
||||||
{
|
{
|
||||||
@ -4039,8 +4040,10 @@ retry:
|
|||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void flush_epd_write_bio(struct extent_page_data *epd)
|
static void flush_write_bio(void *data)
|
||||||
{
|
{
|
||||||
|
struct extent_page_data *epd = data;
|
||||||
|
|
||||||
if (epd->bio) {
|
if (epd->bio) {
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
@ -4050,12 +4053,6 @@ static void flush_epd_write_bio(struct extent_page_data *epd)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
static noinline void flush_write_bio(void *data)
|
|
||||||
{
|
|
||||||
struct extent_page_data *epd = data;
|
|
||||||
flush_epd_write_bio(epd);
|
|
||||||
}
|
|
||||||
|
|
||||||
int extent_write_full_page(struct page *page, struct writeback_control *wbc)
|
int extent_write_full_page(struct page *page, struct writeback_control *wbc)
|
||||||
{
|
{
|
||||||
int ret;
|
int ret;
|
||||||
@ -4068,7 +4065,7 @@ int extent_write_full_page(struct page *page, struct writeback_control *wbc)
|
|||||||
|
|
||||||
ret = __extent_writepage(page, wbc, &epd);
|
ret = __extent_writepage(page, wbc, &epd);
|
||||||
|
|
||||||
flush_epd_write_bio(&epd);
|
flush_write_bio(&epd);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -4110,7 +4107,7 @@ int extent_write_locked_range(struct inode *inode, u64 start, u64 end,
|
|||||||
start += PAGE_SIZE;
|
start += PAGE_SIZE;
|
||||||
}
|
}
|
||||||
|
|
||||||
flush_epd_write_bio(&epd);
|
flush_write_bio(&epd);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -4128,7 +4125,7 @@ int extent_writepages(struct extent_io_tree *tree,
|
|||||||
|
|
||||||
ret = extent_write_cache_pages(mapping, wbc, __extent_writepage, &epd,
|
ret = extent_write_cache_pages(mapping, wbc, __extent_writepage, &epd,
|
||||||
flush_write_bio);
|
flush_write_bio);
|
||||||
flush_epd_write_bio(&epd);
|
flush_write_bio(&epd);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user