diff options
author | NeilBrown <neilb@suse.de> | 2007-09-27 06:47:43 -0400 |
---|---|---|
committer | Jens Axboe <axboe@carl.home.kernel.dk> | 2007-10-10 03:25:57 -0400 |
commit | 6712ecf8f648118c3363c142196418f89a510b90 (patch) | |
tree | 347d39a7d5a7ed96d3b1afecd28de2a0f98b98c9 /drivers/md/dm-raid1.c | |
parent | 5bb23a688b2de23d7765a1dd439d89c038378978 (diff) |
Drop 'size' argument from bio_endio and bi_end_io
As bi_end_io is only called once when the reqeust is complete,
the 'size' argument is now redundant. Remove it.
Now there is no need for bio_endio to subtract the size completed
from bi_size. So don't do that either.
While we are at it, change bi_end_io to return void.
Signed-off-by: Neil Brown <neilb@suse.de>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'drivers/md/dm-raid1.c')
-rw-r--r-- | drivers/md/dm-raid1.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/md/dm-raid1.c b/drivers/md/dm-raid1.c index 144071e70a93..d09ff15490a5 100644 --- a/drivers/md/dm-raid1.c +++ b/drivers/md/dm-raid1.c | |||
@@ -820,7 +820,7 @@ static void write_callback(unsigned long error, void *context) | |||
820 | break; | 820 | break; |
821 | } | 821 | } |
822 | } | 822 | } |
823 | bio_endio(bio, bio->bi_size, 0); | 823 | bio_endio(bio, 0); |
824 | } | 824 | } |
825 | 825 | ||
826 | static void do_write(struct mirror_set *ms, struct bio *bio) | 826 | static void do_write(struct mirror_set *ms, struct bio *bio) |
@@ -900,7 +900,7 @@ static void do_writes(struct mirror_set *ms, struct bio_list *writes) | |||
900 | */ | 900 | */ |
901 | if (unlikely(ms->log_failure)) | 901 | if (unlikely(ms->log_failure)) |
902 | while ((bio = bio_list_pop(&sync))) | 902 | while ((bio = bio_list_pop(&sync))) |
903 | bio_endio(bio, bio->bi_size, -EIO); | 903 | bio_endio(bio, -EIO); |
904 | else while ((bio = bio_list_pop(&sync))) | 904 | else while ((bio = bio_list_pop(&sync))) |
905 | do_write(ms, bio); | 905 | do_write(ms, bio); |
906 | 906 | ||