Re: [PATCH] multipath-tools: Remove bad udev rules

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


On mar., 2011-12-06 at 11:12 -0500, Phillip Susi wrote:
> This sample udev rules file contains some rules relating to dmraid
> that both should not be there and are broken anyhow.  They should
> not be there because firstly, what is a dmraid rule doing in a
> kpartx rule file, and secondly, dmraid already activates partitions
> itself, so there is no need to run kpartx to do that.  The rule is
> broken because it is matching on the DM_UUID starting with "dmraid-",
> but this comparison is case sensitive, and it actually starts with
> "DMRAID-".
> 
> Signed-off-by: Phillip Susi <psusi@xxxxxxxxxx>
> ---
>  kpartx/kpartx.rules |    6 ------
>  1 files changed, 0 insertions(+), 6 deletions(-)
> 
> diff --git a/kpartx/kpartx.rules b/kpartx/kpartx.rules
> index 4c23dec..64863a0 100644
> --- a/kpartx/kpartx.rules
> +++ b/kpartx/kpartx.rules
> @@ -17,10 +17,6 @@ OPTIONS="link_priority=50"
>  ENV{DM_UUID}=="mpath-*", \
>  	SYMLINK+="disk/by-id/$env{DM_TYPE}-$env{DM_NAME}"
>  
> -# Create persistent links for dmraid tables
> -ENV{DM_UUID}=="dmraid-*", \
> -        SYMLINK+="disk/by-id/$env{DM_TYPE}-$env{DM_NAME}"
> -
>  # Create persistent links for partitions
>  ENV{DM_PART}=="?*", \
>          SYMLINK+="disk/by-id/$env{DM_TYPE}-$env{DM_NAME}-part$env{DM_PART}"
> @@ -28,8 +24,6 @@ ENV{DM_PART}=="?*", \
>  # Create dm tables for partitions
>  ENV{DM_STATE}=="ACTIVE", ENV{DM_UUID}=="mpath-*", \
>          RUN+="/sbin/kpartx -u -p -part /dev/$name"
> -ENV{DM_STATE}=="ACTIVE", ENV{DM_UUID}=="dmraid-*", \
> -        RUN+="/sbin/kpartx -u -p -part /dev/$name"
>  
>  LABEL="kpartx_end"
>  
Merged.


--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel


[DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]     [Fedora Docs]

Add to Google Powered by Linux