diff options
| author | NeilBrown <neilb@suse.de> | 2014-09-09 05:54:11 +0200 |
|---|---|---|
| committer | NeilBrown <neilb@suse.de> | 2014-10-14 04:08:28 +0200 |
| commit | c95e6385e8098ef549008ab29d671bf528a50043 (patch) | |
| tree | 1318890ffe32072155fdb164f02d0ec123764649 /drivers/md | |
| parent | md/raid5: fix init_stripe() inconsistencies (diff) | |
| download | linux-c95e6385e8098ef549008ab29d671bf528a50043.tar.xz linux-c95e6385e8098ef549008ab29d671bf528a50043.zip | |
md/raid1: process_checks doesn't use its return value.
process_checks() always returns '0', so change it to 'void'.
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers/md')
| -rw-r--r-- | drivers/md/raid1.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c index 019aa1840210..7c333b5a50fc 100644 --- a/drivers/md/raid1.c +++ b/drivers/md/raid1.c @@ -1947,7 +1947,7 @@ static int fix_sync_read_error(struct r1bio *r1_bio) return 1; } -static int process_checks(struct r1bio *r1_bio) +static void process_checks(struct r1bio *r1_bio) { /* We have read all readable devices. If we haven't * got the block, then there is no hope left. @@ -2039,7 +2039,6 @@ static int process_checks(struct r1bio *r1_bio) bio_copy_data(sbio, pbio); } - return 0; } static void sync_request_write(struct mddev *mddev, struct r1bio *r1_bio) @@ -2057,8 +2056,8 @@ static void sync_request_write(struct mddev *mddev, struct r1bio *r1_bio) return; if (test_bit(MD_RECOVERY_REQUESTED, &mddev->recovery)) - if (process_checks(r1_bio) < 0) - return; + process_checks(r1_bio); + /* * schedule writes */ |