Re: [PATCH] fix `lvm' and `raid' loading for grub-install/core.img and update-grub/grub.cfg problems

[Robert Millan <rmh@aybabtu.com>]

On Tue, Jan 15, 2008 at 01:17:31PM +0100, Marco Gerards wrote:
> Robert Millan <rmh@aybabtu.com> writes:
> 
> > On Tue, Jan 15, 2008 at 12:23:19PM +0100, Marco Gerards wrote:
> >> Robert Millan <rmh@aybabtu.com> writes:
> >> 
> >> > New patch, a bit simpler (after giving it a though, I don't think the gained
> >> > speed is worth the extra complication)
> >> 
> >> What does this patch fix?  Too many problems in one thread...
> >
> > It fixes loading of lvm/raid modules in two situations:
> >
> >   - In core.img, by grub-install, when they're needed to access /boot.
> >     This is still not enough to get it working due to another bug
> >     (detection of partmap).
> >
> >   - In grub.cfg, by update-grub, when they're needed to access /usr
> >     for the font file.
> 
> Oh, great.  Why can't this font be put in /boot/grub?  Using /usr can
> mean we need: raid, lvm, fshelp, a fs and a partitioning scheme module
> in addition to what we need to access /boot.

/boot/grub might be a mount point for a small storage device, and so big
stuff like fonts might not fit (I got reports of this happening in Debian
BTS).

> >> BTW, does it also work for the case both LVM and RAID are used?
> >
> > No, for that we need to handle recursion in grub-probe, which is the same
> > we'll need to get partmap detection working on lvm/raid.
> 
> You are working on this?

No, although I thought a bit about it and how to solve it.

> Or is this bug documented?

Maybe we have something buried deep in Debian BTS.  It might take more time
to find it than to fix the bug, though ;-)

The problem is like:  When LVM is based on physical partitions that belong
to a pc or gpt table, "grub-probe -t partmap /some/lvmed/mntpoint" won't work.

-- 
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 /.)