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

Bug#721625: pu: package lvm2/2.02.95-8



Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: pu

lvm2 is slightly broken in Wheezy.  There is a race condition between
udev and the snapshot removal.

Index: debian/changelog
===================================================================
--- debian/changelog	(revision 952)
+++ debian/changelog	(revision 953)
@@ -1,3 +1,11 @@
+lvm2 (2.02.95-8) UNRELEASED; urgency=low
+
+  * Fix udev rules.
+    - Properly exclude special devices, this part got lost. (closes: #718582)
+    - Always call udev sync.
+
+ -- Bastian Blank <waldi@debian.org>  Wed, 07 Aug 2013 14:48:55 +0200
+
 lvm2 (2.02.95-7) testing; urgency=low
 
   * Re-enable cluster support. (closes: #697676)
Index: debian/tree/lvm2/lib/udev/rules.d/60-persistent-storage-lvm.rules
===================================================================
--- debian/tree/lvm2/lib/udev/rules.d/60-persistent-storage-lvm.rules	(revision 952)
+++ debian/tree/lvm2/lib/udev/rules.d/60-persistent-storage-lvm.rules	(revision 953)
@@ -1,12 +0,0 @@
-# Udev rules for LVM.
-# See /usr/share/doc/lvm2/README.udev for further information.
-
-ACTION!="add|change", GOTO="persistent_storage_lvm_end"
-ENV{DM_UDEV_RULES}=="", GOTO="persistent_storage_lvm_end"
-ENV{DM_UUID}!="LVM-?*", GOTO="persistent_storage_lvm_end"
-
-ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}=="1", GOTO="persistent_storage_lvm_end"
-
-ENV{DM_VG_NAME}=="?*", ENV{DM_LV_NAME}=="?*", ENV{DM_LV_LAYER}=="", SYMLINK+="$env{DM_VG_NAME}/$env{DM_LV_NAME}"
-
-LABEL="persistent_storage_lvm_end"
Index: debian/tree/lvm2/lib/udev/rules.d/56-lvm.rules
===================================================================
--- debian/tree/lvm2/lib/udev/rules.d/56-lvm.rules	(revision 952)
+++ debian/tree/lvm2/lib/udev/rules.d/56-lvm.rules	(revision 953)
@@ -8,8 +8,16 @@
 # Use DM name and split it up into its VG/LV/layer constituents.
 IMPORT{program}="/sbin/dmsetup splitname --nameprefixes --noheadings --rows $env{DM_NAME}"
 
-ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}!="1", GOTO="lvm_end"
+ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}=="1", GOTO="lvm_end"
 
+# Do not create symlinks for inappropriate subdevices.
+ENV{DM_LV_NAME}=="pvmove?*|?*_vorigin", GOTO="lvm_disable"
+ENV{DM_LV_LAYER}=="?*", GOTO="lvm_disable"
+
+# Create symlinks for top-level devices only.
+ENV{DM_VG_NAME}=="?*", ENV{DM_LV_NAME}=="?*", SYMLINK+="$env{DM_VG_NAME}/$env{DM_LV_NAME}", GOTO="lvm_end"
+
+LABEL="lvm_disable"
 ENV{DM_UDEV_DISABLE_DISK_RULES_FLAG}="1"
 ENV{DM_UDEV_DISABLE_OTHER_RULES_FLAG}="1"
 OPTIONS:="nowatch"
Index: debian/tree/dmsetup/lib/udev/rules.d/55-dm.rules
===================================================================
--- debian/tree/dmsetup/lib/udev/rules.d/55-dm.rules	(revision 952)
+++ debian/tree/dmsetup/lib/udev/rules.d/55-dm.rules	(revision 953)
@@ -3,9 +3,6 @@
 
 KERNEL=="device-mapper", NAME="mapper/control"
 
-SUBSYSTEM!="block", GOTO="dm_end"
-KERNEL!="dm-[0-9]*", GOTO="dm_end"
-
 # Decode udev control flags and set environment variables appropriately.
 # These flags are encoded in DM_COOKIE variable that was introduced in
 # kernel version 2.6.31. Therefore, we can use this feature with
@@ -13,6 +10,9 @@
 ENV{DM_COOKIE}=="?*", IMPORT{program}="/sbin/dmsetup udevflags $env{DM_COOKIE}"
 ENV{DM_COOKIE}=="?*", RUN+="/sbin/dmsetup udevcomplete $env{DM_COOKIE}"
 
+SUBSYSTEM!="block", GOTO="dm_end"
+KERNEL!="dm-[0-9]*", GOTO="dm_end"
+
 # Device created, major and minor number assigned - "add" event generated.
 # Table loaded - no event generated.
 # Device resumed (or renamed) - "change" event generated.
Index: .
===================================================================
--- .	(revision 952)
+++ .	(revision 953)

Property changes on: .
___________________________________________________________________
Added: svn:mergeinfo
   Merged /lvm2/trunk:r948,951

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable'), (500, 'stable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.10-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash


Reply to: