diff options
author | Shaohua Li <shli@fb.com> | 2015-10-08 21:54:10 -0700 |
---|---|---|
committer | NeilBrown <neilb@suse.com> | 2015-11-01 13:48:29 +1100 |
commit | 7dde2ad3c5b4afb4b2544b864fa34dd1f4897ab6 (patch) | |
tree | 301d84d8773e13a6b815355446b485ed3dcfc595 | |
parent | a97b7896447a89749d9258fbb9d8c3faf48a7a4e (diff) | |
download | op-kernel-dev-7dde2ad3c5b4afb4b2544b864fa34dd1f4897ab6.zip op-kernel-dev-7dde2ad3c5b4afb4b2544b864fa34dd1f4897ab6.tar.gz |
raid5-cache: start raid5 readonly if journal is missing
If raid array is expected to have journal (eg, journal is set in MD
superblock feature map) and the array is started without journal disk,
start the array readonly.
Signed-off-by: Shaohua Li <shli@fb.com>
Signed-off-by: NeilBrown <neilb@suse.com>
-rw-r--r-- | drivers/md/raid5-cache.c | 3 | ||||
-rw-r--r-- | drivers/md/raid5.c | 7 |
2 files changed, 9 insertions, 1 deletions
diff --git a/drivers/md/raid5-cache.c b/drivers/md/raid5-cache.c index 62e5fe4..b887e04 100644 --- a/drivers/md/raid5-cache.c +++ b/drivers/md/raid5-cache.c @@ -806,8 +806,9 @@ void r5l_quiesce(struct r5l_log *log, int state) bool r5l_log_disk_error(struct r5conf *conf) { + /* don't allow write if journal disk is missing */ if (!conf->log) - return false; + return test_bit(MD_HAS_JOURNAL, &conf->mddev->flags); return test_bit(Faulty, &conf->log->rdev->flags); } diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c index 68c36ce..e1d18be 100644 --- a/drivers/md/raid5.c +++ b/drivers/md/raid5.c @@ -6810,6 +6810,13 @@ static int run(struct mddev *mddev) if (IS_ERR(conf)) return PTR_ERR(conf); + if (test_bit(MD_HAS_JOURNAL, &mddev->flags) && !journal_dev) { + printk(KERN_ERR "md/raid:%s: journal disk is missing, force array readonly\n", + mdname(mddev)); + mddev->ro = 1; + set_disk_ro(mddev->gendisk, 1); + } + conf->min_offset_diff = min_offset_diff; mddev->thread = conf->thread; conf->thread = NULL; |