Commit 385f4d7f authored by Guoqing Jiang's avatar Guoqing Jiang Committed by Shaohua Li

md-cluster: fix wrong condition check in raid1_write_request

The check used here is to avoid conflict between write and
resync, however we used the wrong logic, it should be the
inverse of the checking inside "if".

Fixes: 589a1c49 ("Suspend writes in RAID1 if within range")
Signed-off-by: default avatarGuoqing Jiang <gqjiang@suse.com>
Signed-off-by: default avatarShaohua Li <shli@fb.com>
parent 938b533d
...@@ -1325,9 +1325,9 @@ static void raid1_write_request(struct mddev *mddev, struct bio *bio, ...@@ -1325,9 +1325,9 @@ static void raid1_write_request(struct mddev *mddev, struct bio *bio,
sigset_t full, old; sigset_t full, old;
prepare_to_wait(&conf->wait_barrier, prepare_to_wait(&conf->wait_barrier,
&w, TASK_INTERRUPTIBLE); &w, TASK_INTERRUPTIBLE);
if (bio_end_sector(bio) <= mddev->suspend_lo || if ((bio_end_sector(bio) <= mddev->suspend_lo ||
bio->bi_iter.bi_sector >= mddev->suspend_hi || bio->bi_iter.bi_sector >= mddev->suspend_hi) &&
(mddev_is_clustered(mddev) && (!mddev_is_clustered(mddev) ||
!md_cluster_ops->area_resyncing(mddev, WRITE, !md_cluster_ops->area_resyncing(mddev, WRITE,
bio->bi_iter.bi_sector, bio->bi_iter.bi_sector,
bio_end_sector(bio)))) bio_end_sector(bio))))
......
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