Re: [PATCH] Fix prefix problem with LVM on RAID

[Robert Millan <rmh@aybabtu.com>]

[-- Attachment #1: Type: text/plain, Size: 921 bytes --]

On Sat, Feb 23, 2008 at 09:54:35PM +0100, Michael Guntsche wrote:
> Hello,
> 
> While testing a soon to be deployed setup here I noticed that grub- 
> setup is setting the prefix wrong if you run
> LVM on Software RAID
> 
> grub-setup -v /dev/md0
> ...
> grub-setup: info: dos partition is -2, bsd partition is -2, prefix is  
> (md0)(fatoftheland-main)/boot/grub
> ...
> 
> As you can see the there is also (md0) in the prefix which does not  
> work.
> Attached a small patch that fixes this problem for me. I am not sure  
> if it is the best way to fix it so please take a look.
> If the root device is on LVM only add the LVM part if it is on RAID  
> only add the RAID part.

Sorry for the delay in repliing.  I think it's more correct this way.  Does
it work for you?

-- 
Robert Millan

<GPLv2> I know my rights; I want my phone call!
<DRM> What use is a phone call… if you are unable to speak?
(as seen on /.)

[-- Attachment #2: lvm_on_raid.diff --]
[-- Type: text/x-diff, Size: 474 bytes --]

diff -x configure -x config.h.in -x CVS -x '*~' -x '*.mk' -urp ../grub2/util/i386/pc/grub-setup.c ./util/i386/pc/grub-setup.c
--- ../grub2/util/i386/pc/grub-setup.c	2008-02-19 15:00:11.000000000 +0100
+++ ./util/i386/pc/grub-setup.c	2008-04-13 17:03:21.000000000 +0200
@@ -749,6 +749,7 @@ main (int argc, char *argv[])
       prefix = newprefix;
     }
     
+  else
   if (dest_dev[0] == 'm' && dest_dev[1] == 'd'
       && dest_dev[2] >= '0' && dest_dev[2] <= '9')
     {