forked from Minki/linux
Btrfs: use struct completion in scrub_submit_raid56_bio_wait
This changes to use struct completion directly and removes 'struct scrub_bio_ret' along with the code using it. This struct is used to get the return value from bio, but the caller can access bio to get the return value directly and is holding a reference on it so it won't go away underneath us and can be removed safely. Signed-off-by: Liu Bo <bo.li.liu@oracle.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
parent
c9f540fa6f
commit
b4ff5ad72e
@ -1666,17 +1666,9 @@ leave_nomem:
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
struct scrub_bio_ret {
|
|
||||||
struct completion event;
|
|
||||||
blk_status_t status;
|
|
||||||
};
|
|
||||||
|
|
||||||
static void scrub_bio_wait_endio(struct bio *bio)
|
static void scrub_bio_wait_endio(struct bio *bio)
|
||||||
{
|
{
|
||||||
struct scrub_bio_ret *ret = bio->bi_private;
|
complete(bio->bi_private);
|
||||||
|
|
||||||
ret->status = bio->bi_status;
|
|
||||||
complete(&ret->event);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline int scrub_is_page_on_raid56(struct scrub_page *page)
|
static inline int scrub_is_page_on_raid56(struct scrub_page *page)
|
||||||
@ -1689,11 +1681,9 @@ static int scrub_submit_raid56_bio_wait(struct btrfs_fs_info *fs_info,
|
|||||||
struct bio *bio,
|
struct bio *bio,
|
||||||
struct scrub_page *page)
|
struct scrub_page *page)
|
||||||
{
|
{
|
||||||
struct scrub_bio_ret done;
|
DECLARE_COMPLETION_ONSTACK(done);
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
init_completion(&done.event);
|
|
||||||
done.status = 0;
|
|
||||||
bio->bi_iter.bi_sector = page->logical >> 9;
|
bio->bi_iter.bi_sector = page->logical >> 9;
|
||||||
bio->bi_private = &done;
|
bio->bi_private = &done;
|
||||||
bio->bi_end_io = scrub_bio_wait_endio;
|
bio->bi_end_io = scrub_bio_wait_endio;
|
||||||
@ -1704,11 +1694,8 @@ static int scrub_submit_raid56_bio_wait(struct btrfs_fs_info *fs_info,
|
|||||||
if (ret)
|
if (ret)
|
||||||
return ret;
|
return ret;
|
||||||
|
|
||||||
wait_for_completion_io(&done.event);
|
wait_for_completion_io(&done);
|
||||||
if (done.status)
|
return blk_status_to_errno(bio->bi_status);
|
||||||
return -EIO;
|
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
Loading…
Reference in New Issue
Block a user