support for multipath

Bug #312447 reported by Timo Aaltonen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
grub (Debian)
Confirmed
Unknown
grub (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: grub

There is a patch which adds support to use grub from a multipathed device. I've tested it and it works fine. Just adding this hunk of the patch is enough for the installer:

diff --git a/lib/device.c b/lib/device.c
index 04a61f6..7e35b4a 100644
--- a/lib/device.c
+++ b/lib/device.c
@@ -1119,6 +1119,8 @@ write_to_partition (char **map, int drive, int partition,
          (strncmp (dev, "/dev/cciss/", 11) == 0) ||
          (strncmp (dev, "/dev/rd/", 8) == 0))
         strcpy (dev + strlen(dev), "p");
+ else if (strncmp (dev, "/dev/mapper/mpath", 17) == 0)
+ strcpy (dev + strlen(dev), "-part");
     }
   sprintf (dev + strlen(dev), "%d", ((partition >> 16) & 0xFF) + 1);

Related branches

Changed in grub:
status: Unknown → New
Revision history for this message
Timo Aaltonen (tjaalton) wrote :

bumping the importance for selfish reasons :)

Changed in grub:
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Evan (ev) wrote :

Marking as Fix Committed as Timo added this to the Ubuntu grub branch after receiving approval from Colin on #ubuntu-installer.

Changed in grub:
status: Triaged → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

I double-checked that this new grub doesn't break boot on a more "normal" system.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package grub - 0.97-29ubuntu49

---------------
grub (0.97-29ubuntu49) jaunty; urgency=low

  * Add multipath.diff; support booting from a multipathed device
    (LP: #312447)

 -- Timo Aaltonen <email address hidden> Wed, 11 Feb 2009 13:04:58 +0100

Changed in grub:
status: Fix Committed → Fix Released
Changed in grub (Debian):
status: New → Fix Released
Changed in grub (Debian):
status: Fix Released → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.