PATCH: Fix / on lvm

PATCH: Fix / on lvm

[Hans de Goede]

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

Hi,

Currently / on lvm is not working because the udev rules test for KERNEL="dm-*" which of course
will not be true for PV's, so PV's are not recognized and thus lvm does not work.

The attached patch fixes this.

Regards,

Hans

[-- Attachment #2: dracut-patch --]
[-- Type: text/plain, Size: 727 bytes --]

diff --git a/modules.d/90lvm/64-lvm-vol_id.rules b/modules.d/90lvm/64-lvm-vol_id.rules
index d4b5269..0ebf009 100644
--- a/modules.d/90lvm/64-lvm-vol_id.rules
+++ b/modules.d/90lvm/64-lvm-vol_id.rules
@@ -6,7 +6,6 @@
 
 SUBSYSTEM!="block", GOTO="lvm_end"
 ACTION!="add|change", GOTO="lvm_end"
-KERNEL!="dm-*", GOTO="lvm_end"
 
 IMPORT{program}="vol_id --export $tempnode"
 
diff --git a/modules.d/90lvm/64-lvm.rules b/modules.d/90lvm/64-lvm.rules
index 3eddd07..7218862 100644
--- a/modules.d/90lvm/64-lvm.rules
+++ b/modules.d/90lvm/64-lvm.rules
@@ -6,7 +6,6 @@
 
 SUBSYSTEM!="block", GOTO="lvm_end"
 ACTION!="add|change", GOTO="lvm_end"
-KERNEL!="dm-*", GOTO="lvm_end"
 
 IMPORT{program}="/sbin/blkid -o udev -p $tempnode"
 

Re: PATCH: Fix / on lvm

[Warren Togami]

On 07/10/2009 12:49 PM, Hans de Goede wrote:
> Hi,
>
> Currently / on lvm is not working because the udev rules test for
> KERNEL="dm-*" which of course
> will not be true for PV's, so PV's are not recognized and thus lvm does
> not work.
>
> The attached patch fixes this.
>
> Regards,
>
> Hans

Could you please describe in detail the type of setup needed to 
reproduce this failure?  I have / on lvm here and dracut is working just 
fine.

Warren Togami
wtogami-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: PATCH: Fix / on lvm

[Hans de Goede]

Hi,

On 07/13/2009 05:09 AM, Warren Togami wrote:
> On 07/10/2009 12:49 PM, Hans de Goede wrote:
>> Hi,
>>
>> Currently / on lvm is not working because the udev rules test for
>> KERNEL="dm-*" which of course
>> will not be true for PV's, so PV's are not recognized and thus lvm does
>> not work.
>>
>> The attached patch fixes this.
>>
>> Regards,
>>
>> Hans
>
> Could you please describe in detail the type of setup needed to
> reproduce this failure? I have / on lvm here and dracut is working just
> fine.
>

Basically the default anaconda partitioning setup so /boot /dev/sda1
(sata disk) and / on an lv whose vg contains one (large) pv which is
/dev/sda2

But this isn't all that interesting, read the rules, they are there to
do: "lvm vgscan; lvm vgchange -ay" when a new PV gets added to the system,
yet they contain the line:

KERNEL!="dm-*", GOTO="lvm_end"

Before doing this, meaning that a PV on a regular scsi disk (instead of a
PV on top of device mapper) will not cause the
"lvm vgscan; lvm vgchange -ay" to get executed and thus my / to not be
found.

Maybe you are using a dmraid setup ?  That would explain why these rules
are working for you.

Regards,

Hans


--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: PATCH: Fix / on lvm

[Harald Hoyer]

On 07/10/2009 06:49 PM, Hans de Goede wrote:
> Hi,
>
> Currently / on lvm is not working because the udev rules test for
> KERNEL="dm-*" which of course
> will not be true for PV's, so PV's are not recognized and thus lvm does
> not work.
>
> The attached patch fixes this.
>
> Regards,
>
> Hans

fixed
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html