|
 |
2910d5 |
From 1e93d0d15913c3fa6d0de5af3fb5e4e3b3f068da Mon Sep 17 00:00:00 2001
|
|
|
2910d5 |
From: Blazej Kucman <blazej.kucman@intel.com>
|
|
|
2910d5 |
Date: Fri, 17 Jan 2020 15:24:04 +0100
|
|
|
2910d5 |
Subject: [RHEL8.2 PATCH 61/61] imsm: fill working_disks according to metadata.
|
|
|
2910d5 |
|
|
|
2910d5 |
Imsm tracks as "working_disk" each visible drive.
|
|
|
2910d5 |
Assemble routine expects that the value will return count
|
|
|
2910d5 |
of active member drives recorded in metadata.
|
|
|
2910d5 |
As a side effect "--no-degraded" doesn't work correctly for imsm.
|
|
|
2910d5 |
Align this field to others.
|
|
|
2910d5 |
Added check, if the option --no-degraded is called with --scan.
|
|
|
2910d5 |
|
|
|
2910d5 |
Signed-off-by: Blazej Kucman <blazej.kucman@intel.com>
|
|
|
2910d5 |
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
|
|
|
2910d5 |
---
|
|
|
2910d5 |
mdadm.c | 9 ++++++---
|
|
|
2910d5 |
super-intel.c | 5 ++---
|
|
|
2910d5 |
2 files changed, 8 insertions(+), 6 deletions(-)
|
|
|
2910d5 |
|
|
|
2910d5 |
diff --git a/mdadm.c b/mdadm.c
|
|
|
2910d5 |
index 256a97e..13dc24e 100644
|
|
|
2910d5 |
--- a/mdadm.c
|
|
|
2910d5 |
+++ b/mdadm.c
|
|
|
2910d5 |
@@ -1485,9 +1485,12 @@ int main(int argc, char *argv[])
|
|
|
2910d5 |
rv = Manage_stop(devlist->devname, mdfd, c.verbose, 0);
|
|
|
2910d5 |
break;
|
|
|
2910d5 |
case ASSEMBLE:
|
|
|
2910d5 |
- if (devs_found == 1 && ident.uuid_set == 0 &&
|
|
|
2910d5 |
- ident.super_minor == UnSet && ident.name[0] == 0 &&
|
|
|
2910d5 |
- !c.scan ) {
|
|
|
2910d5 |
+ if (!c.scan && c.runstop == -1) {
|
|
|
2910d5 |
+ pr_err("--no-degraded not meaningful without a --scan assembly.\n");
|
|
|
2910d5 |
+ exit(1);
|
|
|
2910d5 |
+ } else if (devs_found == 1 && ident.uuid_set == 0 &&
|
|
|
2910d5 |
+ ident.super_minor == UnSet && ident.name[0] == 0 &&
|
|
|
2910d5 |
+ !c.scan) {
|
|
|
2910d5 |
/* Only a device has been given, so get details from config file */
|
|
|
2910d5 |
struct mddev_ident *array_ident = conf_get_ident(devlist->devname);
|
|
|
2910d5 |
if (array_ident == NULL) {
|
|
|
2910d5 |
diff --git a/super-intel.c b/super-intel.c
|
|
|
2910d5 |
index 5c1f759..47809bc 100644
|
|
|
2910d5 |
--- a/super-intel.c
|
|
|
2910d5 |
+++ b/super-intel.c
|
|
|
2910d5 |
@@ -7946,7 +7946,8 @@ static struct mdinfo *container_content_imsm(struct supertype *st, char *subarra
|
|
|
2910d5 |
skip = 1;
|
|
|
2910d5 |
if (!skip && (ord & IMSM_ORD_REBUILD))
|
|
|
2910d5 |
recovery_start = 0;
|
|
|
2910d5 |
-
|
|
|
2910d5 |
+ if (!(ord & IMSM_ORD_REBUILD))
|
|
|
2910d5 |
+ this->array.working_disks++;
|
|
|
2910d5 |
/*
|
|
|
2910d5 |
* if we skip some disks the array will be assmebled degraded;
|
|
|
2910d5 |
* reset resync start to avoid a dirty-degraded
|
|
|
2910d5 |
@@ -7988,8 +7989,6 @@ static struct mdinfo *container_content_imsm(struct supertype *st, char *subarra
|
|
|
2910d5 |
else
|
|
|
2910d5 |
this->array.spare_disks++;
|
|
|
2910d5 |
}
|
|
|
2910d5 |
- if (info_d->recovery_start == MaxSector)
|
|
|
2910d5 |
- this->array.working_disks++;
|
|
|
2910d5 |
|
|
|
2910d5 |
info_d->events = __le32_to_cpu(mpb->generation_num);
|
|
|
2910d5 |
info_d->data_offset = pba_of_lba0(map);
|
|
|
2910d5 |
--
|
|
|
2910d5 |
2.7.5
|
|
|
2910d5 |
|