[dm-devel] [PATCH] multipath-tools: Remove bad udev rules
Christophe Varoqui
christophe.varoqui at gmail.com
Tue Dec 6 22:27:51 UTC 2011
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 at cfl.rr.com>
> ---
> 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.
More information about the dm-devel
mailing list