Commit eb8d0c6d authored by Christoph Hellwig's avatar Christoph Hellwig Committed by David Sterba

btrfs: remove the force_bio_submit to submit_extent_page

If force_bio_submit, submit_extent_page simply calls submit_one_bio as
the first thing.  This can just be moved to the only caller that sets
force_bio_submit to true.
Reviewed-by: default avatarJohannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarDavid Sterba <dsterba@suse.com>
Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
parent 67998cf4
...@@ -1027,8 +1027,7 @@ static int submit_extent_page(blk_opf_t opf, ...@@ -1027,8 +1027,7 @@ static int submit_extent_page(blk_opf_t opf,
struct btrfs_bio_ctrl *bio_ctrl, struct btrfs_bio_ctrl *bio_ctrl,
u64 disk_bytenr, struct page *page, u64 disk_bytenr, struct page *page,
size_t size, unsigned long pg_offset, size_t size, unsigned long pg_offset,
enum btrfs_compression_type compress_type, enum btrfs_compression_type compress_type)
bool force_bio_submit)
{ {
struct btrfs_inode *inode = BTRFS_I(page->mapping->host); struct btrfs_inode *inode = BTRFS_I(page->mapping->host);
unsigned int cur = pg_offset; unsigned int cur = pg_offset;
...@@ -1040,9 +1039,6 @@ static int submit_extent_page(blk_opf_t opf, ...@@ -1040,9 +1039,6 @@ static int submit_extent_page(blk_opf_t opf,
ASSERT(bio_ctrl->end_io_func); ASSERT(bio_ctrl->end_io_func);
if (force_bio_submit)
submit_one_bio(bio_ctrl);
while (cur < pg_offset + size) { while (cur < pg_offset + size) {
u32 offset = cur - pg_offset; u32 offset = cur - pg_offset;
int added; int added;
...@@ -1331,10 +1327,11 @@ static int btrfs_do_readpage(struct page *page, struct extent_map **em_cached, ...@@ -1331,10 +1327,11 @@ static int btrfs_do_readpage(struct page *page, struct extent_map **em_cached,
continue; continue;
} }
if (force_bio_submit)
submit_one_bio(bio_ctrl);
ret = submit_extent_page(REQ_OP_READ | read_flags, NULL, ret = submit_extent_page(REQ_OP_READ | read_flags, NULL,
bio_ctrl, disk_bytenr, page, iosize, bio_ctrl, disk_bytenr, page, iosize,
pg_offset, this_bio_flag, pg_offset, this_bio_flag);
force_bio_submit);
if (ret) { if (ret) {
/* /*
* We have to unlock the remaining range, or the page * We have to unlock the remaining range, or the page
...@@ -1645,8 +1642,7 @@ static noinline_for_stack int __extent_writepage_io(struct btrfs_inode *inode, ...@@ -1645,8 +1642,7 @@ static noinline_for_stack int __extent_writepage_io(struct btrfs_inode *inode,
ret = submit_extent_page(op | write_flags, wbc, ret = submit_extent_page(op | write_flags, wbc,
bio_ctrl, disk_bytenr, bio_ctrl, disk_bytenr,
page, iosize, page, iosize,
cur - page_offset(page), cur - page_offset(page), 0);
0, false);
if (ret) { if (ret) {
has_error = true; has_error = true;
if (!saved_ret) if (!saved_ret)
...@@ -2139,7 +2135,7 @@ static int write_one_subpage_eb(struct extent_buffer *eb, ...@@ -2139,7 +2135,7 @@ static int write_one_subpage_eb(struct extent_buffer *eb,
ret = submit_extent_page(REQ_OP_WRITE | write_flags, wbc, ret = submit_extent_page(REQ_OP_WRITE | write_flags, wbc,
bio_ctrl, eb->start, page, eb->len, bio_ctrl, eb->start, page, eb->len,
eb->start - page_offset(page), 0, false); eb->start - page_offset(page), 0);
if (ret) { if (ret) {
btrfs_subpage_clear_writeback(fs_info, page, eb->start, eb->len); btrfs_subpage_clear_writeback(fs_info, page, eb->start, eb->len);
set_btree_ioerr(page, eb); set_btree_ioerr(page, eb);
...@@ -2180,7 +2176,7 @@ static noinline_for_stack int write_one_eb(struct extent_buffer *eb, ...@@ -2180,7 +2176,7 @@ static noinline_for_stack int write_one_eb(struct extent_buffer *eb,
set_page_writeback(p); set_page_writeback(p);
ret = submit_extent_page(REQ_OP_WRITE | write_flags, wbc, ret = submit_extent_page(REQ_OP_WRITE | write_flags, wbc,
bio_ctrl, disk_bytenr, p, bio_ctrl, disk_bytenr, p,
PAGE_SIZE, 0, 0, false); PAGE_SIZE, 0, 0);
if (ret) { if (ret) {
set_btree_ioerr(p, eb); set_btree_ioerr(p, eb);
if (PageWriteback(p)) if (PageWriteback(p))
...@@ -4449,7 +4445,7 @@ static int read_extent_buffer_subpage(struct extent_buffer *eb, int wait, ...@@ -4449,7 +4445,7 @@ static int read_extent_buffer_subpage(struct extent_buffer *eb, int wait,
btrfs_subpage_start_reader(fs_info, page, eb->start, eb->len); btrfs_subpage_start_reader(fs_info, page, eb->start, eb->len);
ret = submit_extent_page(REQ_OP_READ, NULL, &bio_ctrl, ret = submit_extent_page(REQ_OP_READ, NULL, &bio_ctrl,
eb->start, page, eb->len, eb->start, page, eb->len,
eb->start - page_offset(page), 0, false); eb->start - page_offset(page), 0);
if (ret) { if (ret) {
/* /*
* In the endio function, if we hit something wrong we will * In the endio function, if we hit something wrong we will
...@@ -4559,7 +4555,7 @@ int read_extent_buffer_pages(struct extent_buffer *eb, int wait, int mirror_num, ...@@ -4559,7 +4555,7 @@ int read_extent_buffer_pages(struct extent_buffer *eb, int wait, int mirror_num,
ClearPageError(page); ClearPageError(page);
err = submit_extent_page(REQ_OP_READ, NULL, err = submit_extent_page(REQ_OP_READ, NULL,
&bio_ctrl, page_offset(page), page, &bio_ctrl, page_offset(page), page,
PAGE_SIZE, 0, 0, false); PAGE_SIZE, 0, 0);
if (err) { if (err) {
/* /*
* We failed to submit the bio so it's the * We failed to submit the bio so it's the
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment