New RESHAPE_NO_BACKUP flag to track when backup action is needed.

Some arrays (raid10) never need a backup file, so during assembly
we can avoid the whole Grow_continue check in that case.
Achieve this using a flag set by the metadata handler.

Also get "mdadm -I" to fail if a backup process would be
needed.  It currently does fail as the kernel rejects things,
but it is nicer to have this explicit.

Signed-off-by: NeilBrown <neilb@suse.de>

diff --git a/Assemble.c b/Assemble.c
index 8b9f541e4a96059e098dfb3f38a0be67caa58c4a..7a6780605ece3bda1cffd5168ab09acd5fb35a5e 100644 (file)
--- a/Assemble.c
+++ b/Assemble.c
@@ -1192,7 +1192,8 @@ int Assemble(struct supertype *st, char *mddev,
         * The code of doing this lives in Grow.c
         */
 #ifndef MDASSEMBLE
-       if (content->reshape_active) {
+       if (content->reshape_active &&
+           !(content->reshape_active & RESHAPE_NO_BACKUP)) {
                int err = 0;
                int *fdlist = xmalloc(sizeof(int)* bestcnt);
                if (c->verbose > 0)
@@ -1368,6 +1369,7 @@ int Assemble(struct supertype *st, char *mddev,
                        int rv;
 #ifndef MDASSEMBLE
                        if (content->reshape_active &&
+                           !(content->reshape_active & RESHAPE_NO_BACKUP) &&
                            content->delta_disks <= 0) {
                                rv = sysfs_set_str(content, NULL,
                                                   "array_state", "readonly");

diff --git a/Incremental.c b/Incremental.c
index 8e101d7035ecf329edf17369da0b95b501bb7a53..1615c4df043f29c9ec66cad5ea85cab3162b5f0d 100644 (file)
--- a/Incremental.c
+++ b/Incremental.c
@@ -494,6 +494,14 @@ int Incremental(char *devname, struct context *c,
        if (c->runstop > 0 || active_disks >= info.array.working_disks) {
                struct mdinfo *dsk;
                /* Let's try to start it */
+
+               if (info.reshape_active && !(info.reshape_active & RESHAPE_NO_BACKUP)) {
+                       fprintf(stderr, Name
+                               ": %s: This array is being reshaped and cannot be started\n"
+                               "      by --incremental.  Please use --assemble\n",
+                               chosen_name);
+                       goto out;
+               }
                if (match && match->bitmap_file) {
                        int bmfd = open(match->bitmap_file, O_RDWR);
                        if (bmfd < 0) {
@@ -1285,6 +1293,9 @@ int IncrementalScan(int verbose)
                                               me->path, strerror(errno));
                        }
                }
+               /* FIXME check for reshape_active and consider not
+                * starting array.
+                */
                sra = sysfs_read(mdfd, 0, 0);
                if (sra) {
                        if (sysfs_set_str(sra, NULL,

diff --git a/mdadm.h b/mdadm.h
index b808739aea6770687e2281a4ed03448db14c04db..457cc5831e2f7f2c845400a6ef14c18a8da12235 100644 (file)
--- a/mdadm.h
+++ b/mdadm.h
@@ -198,6 +198,7 @@ struct mdinfo {
 #define NO_RESHAPE             0
 #define VOLUME_RESHAPE         1
 #define CONTAINER_RESHAPE      2
+#define RESHAPE_NO_BACKUP      16 /* Mask 'or'ed in */
        int                     reshape_active;
        unsigned long long      reshape_progress;
        int                     recovery_blocked; /* for external metadata it

diff --git a/super1.c b/super1.c
index 444e6bb216b5a88520d7d2e44caa28ab49556c26..54bd5b3268ccec91f8a4e63e341f80e77b355617 100644 (file)
--- a/super1.c
+++ b/super1.c
@@ -766,6 +766,8 @@ static void getinfo_super1(struct supertype *st, struct mdinfo *info, char *map)
 
        if (sb->feature_map & __le32_to_cpu(MD_FEATURE_RESHAPE_ACTIVE)) {
                info->reshape_active = 1;
+               if (info->array.level == 10)
+                       info->reshape_active |= RESHAPE_NO_BACKUP;
                info->reshape_progress = __le64_to_cpu(sb->reshape_position);
                info->new_level = __le32_to_cpu(sb->new_level);
                info->delta_disks = __le32_to_cpu(sb->delta_disks);