These tests (which were SUSE-specific anyway) are not needed here. The kpartx invocation further down is guarded separately. SYMLINK actions must be run in any case for consistency, and there's no need to access the udev db here because kpartx_id also works on otherwise broken devices. Signed-off-by: Martin Wilck <mwilck@xxxxxxxx> --- kpartx/kpartx.rules | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/kpartx/kpartx.rules b/kpartx/kpartx.rules index 64d550de..ea34e449 100644 --- a/kpartx/kpartx.rules +++ b/kpartx/kpartx.rules @@ -5,12 +5,12 @@ # KERNEL!="dm-*", GOTO="kpartx_end" -ACTION=="remove", GOTO="kpartx_end" +ACTION!="add|change", GOTO="kpartx_end" +ENV{DM_UUID}!="?*", GOTO="kpartx_end" -ENV{DM_TABLE_STATE}!="LIVE", GOTO="kpartx_end" -ENV{DM_DEPS}=="0", GOTO="kpartx_end" - -ENV{DM_UUID}=="?*", IMPORT{program}=="kpartx_id %M %m $env{DM_UUID}" +# kpartx_id is very robust, it works for suspended maps and maps +# with 0 dependencies +IMPORT{program}=="kpartx_id %M %m $env{DM_UUID}" OPTIONS="link_priority=50" -- 2.14.0 -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel