Re: [PATCH] Bug fix for LVM

[Patrik Horník]

Hello Bean,

great, it is working now.

However, "grub-probe -t abstraction" separates different modules with
newline character, so grub-mkconfig generates incorrect grub.cfg.

Patrik Horník


On Tue, Jul 28, 2009 at 17:15, Bean<address@hidden> wrote:
> Hi,
>
> BTW, I found a regression on r2421 that cause raid5rec to fail:
>
> http://svn.savannah.gnu.org/viewvc/trunk/grub2/disk/raid6_recover.c?root=grub&r1=2197&r2=2421
>
> buf and buf2 are two different variable.
>
> This new patch include the revert raid5_recover.c to r1828.
>
> On Tue, Jul 28, 2009 at 9:57 PM, Bean<address@hidden> wrote:
>> Hi,
>>
>> Oh, I've found the problem. The abstraction is ok, but now partmap is
>> wrong. The original code assumes at most one level of abstraction when
>> detecting partmap, and it doesn't work on two level abstraction like
>> LVM on RAID. If you add module minicmd, you can use lsmod and see pc
>> module is missing.
>>
>> I also make some improvement to the patch. Now it also detect the raid
>> level, and add raid5rec or raid6rec accordingly.
>>
>> --
>> Bean
>>
>
>
>
> --
> Bean
>
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/grub-devel
>
>