summaryrefslogtreecommitdiffstats
path: root/drivers/md/dm-raid.c
diff options
context:
space:
mode:
authorHeinz Mauelshagen <heinzm@redhat.com>2018-12-18 17:35:41 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2019-12-05 09:20:37 +0100
commitfe8e594cb7246815d051d78b74dc5270b7d598f3 (patch)
tree9fef8a02caa484b57ddece56a679731944e0ee9f /drivers/md/dm-raid.c
parent7baf8fd1ffff8e57f903956c10d8a18dc098cdbe (diff)
downloadlinux-stable-fe8e594cb7246815d051d78b74dc5270b7d598f3.tar.gz
linux-stable-fe8e594cb7246815d051d78b74dc5270b7d598f3.tar.bz2
linux-stable-fe8e594cb7246815d051d78b74dc5270b7d598f3.zip
dm raid: fix false -EBUSY when handling check/repair message
[ Upstream commit 74694bcbdf7e28a5ad548cdda9ac56d30be00d13 ] Sending a check/repair message infrequently leads to -EBUSY instead of properly identifying an active resync. This occurs because raid_message() is testing recovery bits in a racy way. Fix by calling decipher_sync_action() from raid_message() to properly identify the idle state of the RAID device. Signed-off-by: Heinz Mauelshagen <heinzm@redhat.com> Signed-off-by: Mike Snitzer <snitzer@redhat.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers/md/dm-raid.c')
-rw-r--r--drivers/md/dm-raid.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/md/dm-raid.c b/drivers/md/dm-raid.c
index 6c9b54288261..23de59a692c5 100644
--- a/drivers/md/dm-raid.c
+++ b/drivers/md/dm-raid.c
@@ -3690,8 +3690,7 @@ static int raid_message(struct dm_target *ti, unsigned int argc, char **argv,
set_bit(MD_RECOVERY_INTR, &mddev->recovery);
md_reap_sync_thread(mddev);
}
- } else if (test_bit(MD_RECOVERY_RUNNING, &mddev->recovery) ||
- test_bit(MD_RECOVERY_NEEDED, &mddev->recovery))
+ } else if (decipher_sync_action(mddev, mddev->recovery) != st_idle)
return -EBUSY;
else if (!strcasecmp(argv[0], "resync"))
; /* MD_RECOVERY_NEEDED set below */