Changeset 2989 in MondoRescue for branches/3.0/mondo/src/mondorestore/mondo-prep.c

Timestamp:

Apr 4, 2012, 2:27:02 PM (12 years ago)

Author:

Bruno Cornec

Message:

r4626@localhost: bruno | 2012-03-30 16:24:25 +0200

• Handle the md version as an additional param in raidtab per device to solve various bug reports, including the previous one mentioned today. Doesn't still fully work, as not generating correctly the -e option, whereas the raidrec structure is correct

File:

• branches/3.0/mondo/src/mondorestore/mondo-prep.c (modified) (3 diffs)

branches/3.0/mondo/src/mondorestore/mondo-prep.c

@@ -565 +565 @@
   char *tmp = NULL;
   char *tmp1 = NULL;
-  char *oldmd = NULL;
-  char *name;
-  char *ver;
-
-  malloc_string(name);
-  malloc_string(ver);
-  
+
   // leave straight away if raidlist is initial or has no entries
   if (!raidlist || raidlist->entries == 0) {
@@ -620 +614 @@
   // - faulty devices ignored
   // - persistent superblock always used as this is recommended
-  // As per bug #473, the parameter "-e 0.90" is used only when:
-  //   1) It detects that system boots from Raid-1
-  //   2) grub bootloader < v1 is used.
-  // Otherwise it won't boot which is bad.
-    if (read_cfg_var(g_mondo_cfg_file, "bootloader.name", name) != 0) {
-            log_msg(2, "Unable to find bootloader.name param in %s conf file\n", g_mondo_cfg_file);
-        strcpy(name,"NONE");
-    }
-    if (read_cfg_var(g_mondo_cfg_file, "bootloader.ver", ver) != 0) {
-            log_msg(2, "No bootloader.ver param in %s conf file\n", g_mondo_cfg_file);
-        strcpy(ver,"NONE");
-    }
-    if (strcmp(name,"GRUB") == 0) {
-        if ((strstr(ver, "GRUB 0.9") != NULL) && (strcmp(raidlist->el[i].raid_device,device) == 0)) {
-            mr_asprintf(&oldmd, "-e 0.90");
-            log_it("Forcing old metadata (%s) for md on %s for old GRUB", oldmd, device);
-        } else {
-            mr_asprintf(&oldmd, "");
-        }
-    } else if ((strcmp(name,"LILO") == 0) && (strcmp(raidlist->el[i].raid_device,device) == 0)) {
-        mr_asprintf(&oldmd, "-e 0.90");
-        log_it("Forcing old metadata (%s) for md on %s for LILO", oldmd, device);
-    } else {
-        mr_asprintf(&oldmd, "");
-    }
-    mr_free(name);
-    mr_free(ver);
-        log_msg(4, "RAID oldmd: %s", oldmd);
-
-  mr_asprintf(&program, "mdadm --create --force --run --auto=yes %s --level=%s --raid-devices=%d %s", raidlist->el[i].raid_device, level, raidlist->el[i].data_disks.entries, oldmd);
-  mr_free(oldmd);
+
+  mr_asprintf(&program, "mdadm --create --force --run --auto=yes %s --level=%s --raid-devices=%d", raidlist->el[i].raid_device, level, raidlist->el[i].data_disks.entries);
   mr_free(level);
         log_msg(4, "cmd built: %s", program);
-  // Restoring the UUID stored at backup time of present
-  for (v = 0; v <= raidlist->el[i].additional_vars.entries ; v++ ) {
+  // Restoring the UUID and Version stored at backup time of present
+  for (v = 0; v < raidlist->el[i].additional_vars.entries ; v++ ) {
+      log_msg(4,"Working on additional param #%d (Label: %s)",v,raidlist->el[i].additional_vars.el[v].label);
     if ((raidlist->el[i].additional_vars.el[v].label != NULL) && (strcmp(raidlist->el[i].additional_vars.el[v].label,"UUID") == 0)) {
         // We have a UUID to handle
@@ -663 +629 @@
             break;
         } else {
-            log_it("Unable to manage previous NULL UUID");
+            log_msg(1,"Unable to manage previous NULL UUID");
+        }
+    }
+    if ((raidlist->el[i].additional_vars.el[v].label != NULL) && (strcmp(raidlist->el[i].additional_vars.el[v].label,"Version") == 0)) {
+        // We have a Version to handle
+        if (raidlist->el[i].additional_vars.el[v].value != NULL) {
+            // force its restoration in order to support all complex boot loader + md format cases
+            // Also see bug #473
+            log_it("Managing previous Version %s", raidlist->el[i].additional_vars.el[v].value);
+            mr_strcat(program, " -e %s",raidlist->el[i].additional_vars.el[v].value);
+            break;
+        } else {
+            log_msg(1,"Unable to manage previous NULL Version");
         }
     }