lvm2-commits October 2013

lvm2-commits@lists.fedorahosted.org

7 participants
86 discussions

master - configure: detect compiler flag
by Zdenek Kabelac 24 Oct '13

24 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=d95751cb0b750ea6… Commit: d95751cb0b750ea698d665bd8ad8086fe9ee6945 Parent: 772fa460d1ef7dede99478e30de756136ea0c7aa Author: Zdenek Kabelac <zkabelac(a)redhat.com> AuthorDate: Thu Oct 24 21:40:25 2013 +0200 Committer: Zdenek Kabelac <zkabelac(a)redhat.com> CommitterDate: Fri Oct 25 00:41:36 2013 +0200 configure: detect compiler flag Use m4 macro AC_TRY_CCFLAG to detect presence of some compiler option. Use it to detect -Wjump-misses-init. --- acinclude.m4 | 39 +++++++++++++++++++++++++++++++++++++++ aclocal.m4 | 1 + configure | 42 ++++++++++++++++++++++++++++++++++++++++++ configure.in | 4 ++++ make.tmpl.in | 6 +++++- 5 files changed, 91 insertions(+), 1 deletions(-) diff --git a/acinclude.m4 b/acinclude.m4 new file mode 100644 index 0000000..333abb5 --- /dev/null +++ b/acinclude.m4 @@ -0,0 +1,39 @@ +dnl AC_GCC_VERSION +dnl check for compiler version +dnl sets COMPILER_VERSION and GCC_VERSION + +AC_DEFUN([AC_CC_VERSION], +[ + AC_MSG_CHECKING([C compiler version]) + COMPILER_VERSION=`$CC -v 2>&1 | grep version` + case "$COMPILER_VERSION" in + *gcc*) + dnl Ok, how to turn $3 into the real $3 + GCC_VERSION=`echo $COMPILER_VERSION | \ + sed -e 's/[[^ ]]*\ [[^ ]]*\ $[[^ ]]*$\ .*/\1/'` ;; + *) GCC_VERSION=unknown ;; + esac + AC_MSG_RESULT($GCC_VERSION) +]) + +dnl AC_TRY_CCFLAG([CCFLAG], [VAR], [ACTION-IF-WORKS], [ACTION-IF-FAILS]) +dnl check if $CC supports a given flag + +AC_DEFUN([AC_TRY_CCFLAG], +[ + AC_REQUIRE([AC_PROG_CC]) + ac_save_CFLAGS=$CFLAGS + CFLAGS=$1 + AC_MSG_RESULT([KOUKEJ $name]) + AC_CACHE_CHECK([whether $CC accepts $1 flag], [ac_cv_flag_$2], + [AC_COMPILE_IFELSE([AC_LANG_PROGRAM()], + [AS_VAR_SET([ac_cv_flag_$2], [yes])], + [AS_VAR_SET([ac_cv_flag_$2], [no])])]) + CFLAGS=$ac_save_CFLAGS + $2=AS_VAR_GET([ac_cv_flag_$2]) + if test "$2" = yes; then + ifelse([$3], [], [:], [$3]) + else + ifelse([$4], [], [:], [$4]) + fi +]) diff --git a/aclocal.m4 b/aclocal.m4 index 263b50e..165d9bc 100644 --- a/aclocal.m4 +++ b/aclocal.m4 @@ -169,3 +169,4 @@ else fi[]dnl ])# PKG_CHECK_MODULES +m4_include([acinclude.m4]) diff --git a/configure b/configure index 002a7d2..48dfdc6 100755 --- a/configure +++ b/configure @@ -708,6 +708,7 @@ CUNIT_CFLAGS GENPNG GENHTML LCOV +HAVE_WJUMP SACKPT_LIBS SACKPT_CFLAGS DLM_LIBS @@ -8677,6 +8678,47 @@ else CSCOPE_CMD= fi + + + ac_save_CFLAGS=$CFLAGS + CFLAGS=-Wjump-misses-init + { $as_echo "$as_me:${as_lineno-$LINENO}: result: KOUKEJ $name" >&5 +$as_echo "KOUKEJ $name" >&6; } + { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether $CC accepts -Wjump-misses-init flag" >&5 +$as_echo_n "checking whether $CC accepts -Wjump-misses-init flag... " >&6; } +if test "${ac_cv_flag_HAVE_WJUMP+set}" = set; then : + $as_echo_n "(cached) " >&6 +else + cat confdefs.h - <<_ACEOF >conftest.$ac_ext +/* end confdefs.h. */ + +int +main () +{ + + ; + return 0; +} +_ACEOF +if ac_fn_c_try_compile "$LINENO"; then : + ac_cv_flag_HAVE_WJUMP=yes +else + ac_cv_flag_HAVE_WJUMP=no +fi +rm -f core conftest.err conftest.$ac_objext conftest.$ac_ext +fi +{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_cv_flag_HAVE_WJUMP" >&5 +$as_echo "$ac_cv_flag_HAVE_WJUMP" >&6; } + CFLAGS=$ac_save_CFLAGS + HAVE_WJUMP=$ac_cv_flag_HAVE_WJUMP + if test "HAVE_WJUMP" = yes; then + : + else + : + fi + + + ################################################################################ { $as_echo "$as_me:${as_lineno-$LINENO}: checking for C optimisation flag" >&5 $as_echo_n "checking for C optimisation flag... " >&6; } diff --git a/configure.in b/configure.in index 07b0afc..3a25fef 100644 --- a/configure.in +++ b/configure.in @@ -835,6 +835,10 @@ else CSCOPE_CMD= fi +dnl -- Check if compiler supports -Wjump-misses-init +AC_TRY_CCFLAG([-Wjump-misses-init], [HAVE_WJUMP], [], []) +AC_SUBST(HAVE_WJUMP) + ################################################################################ dnl -- Override optimisation AC_MSG_CHECKING(for C optimisation flag) diff --git a/make.tmpl.in b/make.tmpl.in index 788eb1d..08560a8 100644 --- a/make.tmpl.in +++ b/make.tmpl.in @@ -138,9 +138,13 @@ WFLAGS += -Wall -Wundef -Wshadow -Wcast-align -Wwrite-strings \ -Wpointer-arith -Wclobbered -Wempty-body -Wignored-qualifiers \ -Wmissing-parameter-type -Wold-style-declaration -Woverride-init \ -Wtype-limits -Wuninitialized -Wmissing-include-dirs -Wsync-nand \ - -Wfloat-equal -Wjump-misses-init -Wlogical-op -Wstrict-prototypes \ + -Wfloat-equal -Wlogical-op -Wstrict-prototypes \ -Wold-style-definition -Wmissing-format-attribute +ifeq ("@HAVE_WJUMP@", "yes") +WFLAGS += -Wjump-misses-init +endif + #WFLAGS += -W -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers #WFLAGS += -Wsign-compare -Wunused-parameter -Wmissing-field-initializers #WFLAGS += -Wconversion -Wbad-function-cast -Wcast-qual -Waggregate-return -Wpacked

1 0

master - clean-up: Remove redundant faulty logic
by Jonathan Brassow 24 Oct '13

24 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=772fa460d1ef7ded… Commit: 772fa460d1ef7dede99478e30de756136ea0c7aa Parent: 4c0db84948ad616930e217f20768a4285ca67c5a Author: Jonathan Brassow <jbrassow(a)redhat.com> AuthorDate: Wed Oct 23 22:44:04 2013 -0500 Committer: Jonathan Brassow <jbrassow(a)redhat.com> CommitterDate: Wed Oct 23 22:44:04 2013 -0500 clean-up: Remove redundant faulty logic Remove conditional that boils down to "if yes or no, then do". The previous condition in the statement is sufficient and the extra (always true) condition is unnecessary. --- tools/lvchange.c | 7 ++----- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/tools/lvchange.c b/tools/lvchange.c index 6ae9720..1931c03 100644 --- a/tools/lvchange.c +++ b/tools/lvchange.c @@ -772,11 +772,8 @@ static int lvchange_writemostly(struct logical_volume *lv) if (!(pv_names[i] = dm_pool_zalloc(lv->vg->vgmem, tmp_str_len + 3))) return_0; - if (tmp_str_len < 3 || - ((tmp_str[tmp_str_len - 2] != ':') && - ((tmp_str[tmp_str_len - 1] != 't') || - (tmp_str[tmp_str_len - 1] != 'y') || - (tmp_str[tmp_str_len - 1] != 'n')))) + if ((tmp_str_len < 3) || + (tmp_str[tmp_str_len - 2] != ':')) /* Default to 'y' if no mode specified */ sprintf(pv_names[i], "%s:y", tmp_str); else

1 0

master - clvmd: fix verify message rejection of REMOTE flag
by David Teigland 24 Oct '13

24 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=4c0db84948ad6169… Commit: 4c0db84948ad616930e217f20768a4285ca67c5a Parent: c9c23d414802ed07c7638ce13d33dba3a629e887 Author: David Teigland <teigland(a)redhat.com> AuthorDate: Thu Oct 24 11:10:09 2013 -0500 Committer: David Teigland <teigland(a)redhat.com> CommitterDate: Thu Oct 24 11:18:22 2013 -0500 clvmd: fix verify message rejection of REMOTE flag This fixes a bug in commit 19baf842 where verify_message was rejecting the CLVMD_FLAG_REMOTE flag. It was missed since the patch was ported from an lvm version where that flag does not exist. --- daemons/clvmd/clvmd.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/daemons/clvmd/clvmd.c b/daemons/clvmd/clvmd.c index bed8e68..d57c0fd 100644 --- a/daemons/clvmd/clvmd.c +++ b/daemons/clvmd/clvmd.c @@ -1124,7 +1124,7 @@ static int verify_message(char *buf, int len) /* TODO: we may be able to narrow len/flags/clientid/arglen checks based on cmd */ - if (h->flags & ~(CLVMD_FLAG_LOCAL | CLVMD_FLAG_SYSTEMLV | CLVMD_FLAG_NODEERRS)) { + if (h->flags & ~(CLVMD_FLAG_LOCAL | CLVMD_FLAG_SYSTEMLV | CLVMD_FLAG_NODEERRS | CLVMD_FLAG_REMOTE)) { log_error("verify_message bad flags %x", h->flags); return -1; }

1 0

master - build: Use additional gcc warning flags.
by Alasdair Kergon 24 Oct '13

24 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=c9c23d414802ed07… Commit: c9c23d414802ed07c7638ce13d33dba3a629e887 Parent: d5896f0afd28096c3dc04a9c7e9627b387f145fb Author: Alasdair G Kergon <agk(a)redhat.com> AuthorDate: Thu Oct 24 17:10:24 2013 +0100 Committer: Alasdair G Kergon <agk(a)redhat.com> CommitterDate: Thu Oct 24 17:10:24 2013 +0100 build: Use additional gcc warning flags. --- WHATS_NEW | 3 ++- make.tmpl.in | 12 +++++++++--- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index 7106867..cb3488b 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ -Version 2.02.104 +Version 2.02.104 - =================================== + Use additional gcc warning flags by default. Add ignore_lvm_mirrors to config file to read/ignore labels on mirrors. Add internal flag for temporary LVs to properly direct udev to not interfere. Fix endless loop in blkdeactivate <device>... if unable to umount/deactivate. diff --git a/make.tmpl.in b/make.tmpl.in index 3218758..788eb1d 100644 --- a/make.tmpl.in +++ b/make.tmpl.in @@ -135,9 +135,15 @@ INSTALL_SCRIPT = $(INSTALL) -p $(M_INSTALL_PROGRAM) WFLAGS += -Wall -Wundef -Wshadow -Wcast-align -Wwrite-strings \ -Wmissing-prototypes -Wmissing-declarations -Wnested-externs \ -Winline -Wmissing-noreturn -Wformat-security -Wredundant-decls \ - -Wpointer-arith - -#WFLAGS += -W -Wconversion -Wbad-function-cast -Wcast-qual + -Wpointer-arith -Wclobbered -Wempty-body -Wignored-qualifiers \ + -Wmissing-parameter-type -Wold-style-declaration -Woverride-init \ + -Wtype-limits -Wuninitialized -Wmissing-include-dirs -Wsync-nand \ + -Wfloat-equal -Wjump-misses-init -Wlogical-op -Wstrict-prototypes \ + -Wold-style-definition -Wmissing-format-attribute + +#WFLAGS += -W -Wno-sign-compare -Wno-unused-parameter -Wno-missing-field-initializers +#WFLAGS += -Wsign-compare -Wunused-parameter -Wmissing-field-initializers +#WFLAGS += -Wconversion -Wbad-function-cast -Wcast-qual -Waggregate-return -Wpacked #WFLAGS += -pedantic -std=gnu99 #DEFS += -DDEBUG_CRC32

1 0

master - Mirror: Fix hangs and lock-ups caused by attempting label reads of mirrors
by Jonathan Brassow 23 Oct '13

23 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=d5896f0afd28096c… Commit: d5896f0afd28096c3dc04a9c7e9627b387f145fb Parent: 039bdad732cd6c0d96b5b8c2c6c7beb528cf338a Author: Jonathan Brassow <jbrassow(a)redhat.com> AuthorDate: Tue Oct 22 19:14:33 2013 -0500 Committer: Jonathan Brassow <jbrassow(a)redhat.com> CommitterDate: Tue Oct 22 19:14:33 2013 -0500 Mirror: Fix hangs and lock-ups caused by attempting label reads of mirrors There is a problem with the way mirrors have been designed to handle failures that is resulting in stuck LVM processes and hung I/O. When mirrors encounter a write failure, they block I/O and notify userspace to reconfigure the mirror to remove failed devices. This process is open to a couple races: 1) Any LVM process other than the one that is meant to deal with the mirror failure can attempt to read the mirror, fail, and block other LVM commands (including the repair command) from proceeding due to holding a lock on the volume group. 2) If there are multiple mirrors that suffer a failure in the same volume group, a repair can block while attempting to read the LVM label from one mirror while trying to repair the other. Mitigation of these races has been attempted by disallowing label reading of mirrors that are either suspended or are indicated as blocking by the kernel. While this has closed the window of opportunity for hitting the above problems considerably, it hasn't closed it completely. This is because it is still possible to start an LVM command, read the status of the mirror as healthy, and then perform the read for the label at the moment after a the failure is discovered by the kernel. I can see two solutions to this problem: 1) Allow users to configure whether mirrors can be candidates for LVM labels (i.e. whether PVs can be created on mirror LVs). If the user chooses to allow label scanning of mirror LVs, it will be at the expense of a possible hang in I/O or LVM processes. 2) Instrument a way to allow asynchronous label reading - allowing blocked label reads to be ignored while continuing to process the LVM command. This would action would allow LVM commands to continue even though they would have otherwise blocked trying to read a mirror. They can then release their lock and allow a repair command to commence. In the event of #2 above, the repair command already in progress can continue and repair the failed mirror. This patch brings solution #1. If solution #2 is developed later on, the configuration option created in #1 can be negated - allowing mirrors to be scanned for labels by default once again. --- WHATS_NEW | 1 + conf/example.conf.in | 29 +++++++++++++++++++++++++++++ lib/activate/dev_manager.c | 16 +++++++++++----- lib/commands/toolcontext.c | 1 + lib/config/config_settings.h | 1 + lib/config/defaults.h | 1 + lib/misc/lvm-globals.c | 11 +++++++++++ lib/misc/lvm-globals.h | 2 ++ 8 files changed, 57 insertions(+), 5 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index 49f37a4..7106867 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ Version 2.02.104 =================================== + Add ignore_lvm_mirrors to config file to read/ignore labels on mirrors. Add internal flag for temporary LVs to properly direct udev to not interfere. Fix endless loop in blkdeactivate <device>... if unable to umount/deactivate. Add dev-block-<major>:<minor>.device systemd alias for complete PV tracking. diff --git a/conf/example.conf.in b/conf/example.conf.in index ae571b6..03e1c45 100644 --- a/conf/example.conf.in +++ b/conf/example.conf.in @@ -186,6 +186,35 @@ devices { # in recovery situations. ignore_suspended_devices = 0 + # ignore_lvm_mirrors: Introduced in version 2.02.104 + # This setting determines whether logical volumes of "mirror" segment + # type are scanned for LVM labels. This affects the ability of + # mirrors to be used as physical volumes. If 'ignore_lvm_mirrors' + # is set to '1', it becomes impossible to create volume groups on top + # of mirror logical volumes - i.e. to stack volume groups on mirrors. + # + # Allowing mirror logical volumes to be scanned (setting the value to '0') + # can potentially cause LVM processes and I/O to the mirror to become + # blocked. This is due to the way that the "mirror" segment type handles + # failures. In order for the hang to manifest itself, an LVM command must + # be run just after a failure and before the automatic LVM repair process + # takes place OR there must be failures in multiple mirrors in the same + # volume group at the same time with write failures occurring moments + # before a scan of the mirror's labels. + # + # Note that these scanning limitations do not apply to the LVM RAID + # types, like "raid1". The RAID segment types handle failures in a + # different way and are not subject to possible process or I/O blocking. + # + # It is encouraged that users set 'ignore_lvm_mirrors' to 1 if they + # are using the "mirror" segment type. Users that require volume group + # stacking on mirrored logical volumes should consider using the "raid1" + # segment type. The "raid1" segment type is not available for + # active/active clustered volume groups. + # + # Set to 1 to disallow stacking and thereby avoid a possible deadlock. + ignore_lvm_mirrors = 1 + # During each LVM operation errors received from each device are counted. # If the counter of a particular device exceeds the limit set here, no # further I/O is sent to that device for the remainder of the respective diff --git a/lib/activate/dev_manager.c b/lib/activate/dev_manager.c index 0b911f2..f8027d4 100644 --- a/lib/activate/dev_manager.c +++ b/lib/activate/dev_manager.c @@ -397,11 +397,17 @@ static int _device_is_usable(struct device *dev, int check_lv_names) next = dm_get_next_target(dmt, next, &start, &length, &target_type, &params); - if (target_type && !strcmp(target_type, "mirror") && - !_ignore_blocked_mirror_devices(dev, start, length, params)) { - log_debug_activation("%s: Mirror device %s not usable.", - dev_name(dev), name); - goto out; + if (target_type && !strcmp(target_type, "mirror")) { + if (ignore_lvm_mirrors()) { + log_debug_activation("%s: Scanning mirror devices is disabled.", dev_name(dev)); + goto out; + } + if (!_ignore_blocked_mirror_devices(dev, start, + length, params)) { + log_debug_activation("%s: Mirror device %s not usable.", + dev_name(dev), name); + goto out; + } } /* diff --git a/lib/commands/toolcontext.c b/lib/commands/toolcontext.c index 2732094..3ed359d 100644 --- a/lib/commands/toolcontext.c +++ b/lib/commands/toolcontext.c @@ -905,6 +905,7 @@ static int _init_filters(struct cmd_context *cmd, unsigned load_persistent_cache goto_bad; init_ignore_suspended_devices(find_config_tree_bool(cmd, devices_ignore_suspended_devices_CFG, NULL)); + init_ignore_lvm_mirrors(find_config_tree_bool(cmd, devices_ignore_lvm_mirrors_CFG, NULL)); /* * If 'cache_dir' or 'cache_file_prefix' is set, ignore 'cache'. diff --git a/lib/config/config_settings.h b/lib/config/config_settings.h index c935920..7fdf763 100644 --- a/lib/config/config_settings.h +++ b/lib/config/config_settings.h @@ -96,6 +96,7 @@ cfg(devices_data_alignment_detection_CFG, "data_alignment_detection", devices_CF cfg(devices_data_alignment_CFG, "data_alignment", devices_CFG_SECTION, 0, CFG_TYPE_INT, 0, vsn(2, 2, 45), NULL) cfg(devices_data_alignment_offset_detection_CFG, "data_alignment_offset_detection", devices_CFG_SECTION, 0, CFG_TYPE_BOOL, DEFAULT_DATA_ALIGNMENT_OFFSET_DETECTION, vsn(2, 2, 50), NULL) cfg(devices_ignore_suspended_devices_CFG, "ignore_suspended_devices", devices_CFG_SECTION, 0, CFG_TYPE_BOOL, DEFAULT_IGNORE_SUSPENDED_DEVICES, vsn(1, 2, 19), NULL) +cfg(devices_ignore_lvm_mirrors_CFG, "ignore_lvm_mirrors", devices_CFG_SECTION, 0, CFG_TYPE_BOOL, DEFAULT_IGNORE_LVM_MIRRORS, vsn(2, 2, 104), NULL) cfg(devices_disable_after_error_count_CFG, "disable_after_error_count", devices_CFG_SECTION, 0, CFG_TYPE_INT, DEFAULT_DISABLE_AFTER_ERROR_COUNT, vsn(2, 2, 75), NULL) cfg(devices_require_restorefile_with_uuid_CFG, "require_restorefile_with_uuid", devices_CFG_SECTION, 0, CFG_TYPE_BOOL, DEFAULT_REQUIRE_RESTOREFILE_WITH_UUID, vsn(2, 2, 73), NULL) cfg(devices_pv_min_size_CFG, "pv_min_size", devices_CFG_SECTION, 0, CFG_TYPE_INT, DEFAULT_PV_MIN_SIZE_KB, vsn(2, 2, 85), NULL) diff --git a/lib/config/defaults.h b/lib/config/defaults.h index bd7a40a..141b7ae 100644 --- a/lib/config/defaults.h +++ b/lib/config/defaults.h @@ -33,6 +33,7 @@ #define DEFAULT_SYSFS_SCAN 1 #define DEFAULT_MD_COMPONENT_DETECTION 1 #define DEFAULT_MD_CHUNK_ALIGNMENT 1 +#define DEFAULT_IGNORE_LVM_MIRRORS 1 #define DEFAULT_MULTIPATH_COMPONENT_DETECTION 1 #define DEFAULT_IGNORE_SUSPENDED_DEVICES 1 #define DEFAULT_DISABLE_AFTER_ERROR_COUNT 0 diff --git a/lib/misc/lvm-globals.c b/lib/misc/lvm-globals.c index 3c5d545..ee192b0 100644 --- a/lib/misc/lvm-globals.c +++ b/lib/misc/lvm-globals.c @@ -40,6 +40,7 @@ static int _mirror_in_sync = 0; static int _dmeventd_monitor = DEFAULT_DMEVENTD_MONITOR; static int _background_polling = DEFAULT_BACKGROUND_POLLING; static int _ignore_suspended_devices = 0; +static int _ignore_lvm_mirrors = DEFAULT_IGNORE_LVM_MIRRORS; static int _error_message_produced = 0; static unsigned _is_static = 0; static int _udev_checking = 1; @@ -123,6 +124,11 @@ void init_ignore_suspended_devices(int ignore) _ignore_suspended_devices = ignore; } +void init_ignore_lvm_mirrors(int scan) +{ + _ignore_lvm_mirrors = scan; +} + void init_cmd_name(int status) { _log_cmd_name = status; @@ -259,6 +265,11 @@ int ignore_suspended_devices(void) return _ignore_suspended_devices; } +int ignore_lvm_mirrors(void) +{ + return _ignore_lvm_mirrors; +} + void init_debug(int level) { _debug_level = level; diff --git a/lib/misc/lvm-globals.h b/lib/misc/lvm-globals.h index f590fd6..4af713b 100644 --- a/lib/misc/lvm-globals.h +++ b/lib/misc/lvm-globals.h @@ -38,6 +38,7 @@ void init_mirror_in_sync(int in_sync); void init_dmeventd_monitor(int reg); void init_background_polling(int polling); void init_ignore_suspended_devices(int ignore); +void init_ignore_lvm_mirrors(int scan); void init_error_message_produced(int produced); void init_is_static(unsigned value); void init_udev_checking(int checking); @@ -66,6 +67,7 @@ int security_level(void); int mirror_in_sync(void); int background_polling(void); int ignore_suspended_devices(void); +int ignore_lvm_mirrors(void); const char *log_command_name(void); unsigned is_static(void); int udev_checking(void);

1 0

master - activation: flag temporary LVs internally
by Peter Rajnoha 23 Oct '13

23 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=039bdad732cd6c0d… Commit: 039bdad732cd6c0d96b5b8c2c6c7beb528cf338a Parent: 1dd6626696b100c8cf176921917418bb8d7f8055 Author: Peter Rajnoha <prajnoha(a)redhat.com> AuthorDate: Wed Oct 23 14:06:39 2013 +0200 Committer: Peter Rajnoha <prajnoha(a)redhat.com> CommitterDate: Wed Oct 23 14:09:37 2013 +0200 activation: flag temporary LVs internally Add LV_TEMPORARY flag for LVs with limited existence during command execution. Such LVs are temporary in way that they need to be activated, some action done and then removed immediately. Such LVs are just like any normal LV - the only difference is that they are removed during LVM command execution. This is also the case for LVs representing future pool metadata spare LVs which we need to initialize by using the usual LV before they are declared as pool metadata spare. We can optimize some other parts like udev to do a better job if it knows that the LV is temporary and any processing on it is just useless. This flag is orthogonal to LV_NOSCAN flag introduced recently as LV_NOSCAN flag is primarily used to mark an LV for the scanning to be avoided before the zeroing of the device happens. The LV_TEMPORARY flag makes a difference between a full-fledged LV visible in the system and the LV just used as a temporary overlay for some action that needs to be done on underlying PVs. For example: lvcreate --thinpool POOL --zero n -L 1G vg - first, the usual LV is created to do a clean up for pool metadata spare. The LV is activated, zeroed, deactivated. - between "activated" and "zeroed" stage, the LV_NOSCAN flag is used to avoid any scanning in udev - betwen "zeroed" and "deactivated" stage, we need to avoid the WATCH udev rule, but since the LV is just a usual LV, we can't make a difference. The LV_TEMPORARY internal LV flag helps here. If we create the LV with this flag, the DM_UDEV_DISABLE_DISK_RULES and DM_UDEV_DISABLE_OTHER_RULES flag are set (just like as it is with "invisible" and non-top-level LVs) - udev is directed to skip WATCH rule use. - if the LV_TEMPORARY flag was not used, there would normally be a WATCH event generated once the LV is closed after "zeroed" stage. This will make problems with immediated deactivation that follows. --- WHATS_NEW | 1 + daemons/clvmd/lvm-functions.c | 2 +- lib/activate/activate.c | 21 +++++++++++++-------- lib/activate/activate.h | 20 ++++++++++++++++---- lib/activate/dev_manager.c | 18 +++++++++++------- lib/format_text/flags.c | 1 + lib/locking/file_locking.c | 6 ++++-- lib/locking/no_locking.c | 6 ++++-- lib/metadata/lv_manip.c | 3 +++ lib/metadata/metadata-exported.h | 8 +++++++- lib/metadata/thin_manip.c | 1 + 11 files changed, 62 insertions(+), 25 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index 7cfa87c..49f37a4 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ Version 2.02.104 =================================== + Add internal flag for temporary LVs to properly direct udev to not interfere. Fix endless loop in blkdeactivate <device>... if unable to umount/deactivate. Add dev-block-<major>:<minor>.device systemd alias for complete PV tracking. Use major:minor as short form of --major and --minor arg for pvscan --cache. diff --git a/daemons/clvmd/lvm-functions.c b/daemons/clvmd/lvm-functions.c index f544218..b15732f 100644 --- a/daemons/clvmd/lvm-functions.c +++ b/daemons/clvmd/lvm-functions.c @@ -401,7 +401,7 @@ static int do_activate_lv(char *resource, unsigned char command, unsigned char l } /* Now activate it */ - if (!lv_activate(cmd, resource, exclusive, 0, NULL)) + if (!lv_activate(cmd, resource, exclusive, 0, 0, NULL)) goto error; return 0; diff --git a/lib/activate/activate.c b/lib/activate/activate.c index 131a093..006681e 100644 --- a/lib/activate/activate.c +++ b/lib/activate/activate.c @@ -338,12 +338,12 @@ int lv_activation_filter(struct cmd_context *cmd, const char *lvid_s, return 1; } int lv_activate(struct cmd_context *cmd, const char *lvid_s, int exclusive, int noscan, - struct logical_volume *lv) + int temporary, struct logical_volume *lv) { return 1; } int lv_activate_with_filter(struct cmd_context *cmd, const char *lvid_s, int exclusive, - int noscan, struct logical_volume *lv) + int noscan, int temporary, struct logical_volume *lv) { return 1; } @@ -2055,10 +2055,11 @@ static int _lv_activate(struct cmd_context *cmd, const char *lvid_s, if (filter) laopts->read_only = _passes_readonly_filter(cmd, lv); - log_debug_activation("Activating %s/%s%s%s%s.", lv->vg->name, lv->name, + log_debug_activation("Activating %s/%s%s%s%s%s.", lv->vg->name, lv->name, laopts->exclusive ? " exclusively" : "", laopts->read_only ? " read-only" : "", - laopts->noscan ? " noscan" : ""); + laopts->noscan ? " noscan" : "", + laopts->temporary ? " temporary" : ""); if (!lv_info(cmd, lv, 0, &info, 0, 0)) goto_out; @@ -2096,9 +2097,11 @@ out: /* Activate LV */ int lv_activate(struct cmd_context *cmd, const char *lvid_s, int exclusive, - int noscan, struct logical_volume *lv) + int noscan, int temporary, struct logical_volume *lv) { - struct lv_activate_opts laopts = { .exclusive = exclusive, .noscan = noscan }; + struct lv_activate_opts laopts = { .exclusive = exclusive, + .noscan = noscan, + .temporary = temporary }; if (!_lv_activate(cmd, lvid_s, &laopts, 0, lv)) return_0; @@ -2108,9 +2111,11 @@ int lv_activate(struct cmd_context *cmd, const char *lvid_s, int exclusive, /* Activate LV only if it passes filter */ int lv_activate_with_filter(struct cmd_context *cmd, const char *lvid_s, int exclusive, - int noscan, struct logical_volume *lv) + int noscan, int temporary, struct logical_volume *lv) { - struct lv_activate_opts laopts = { .exclusive = exclusive, .noscan = noscan }; + struct lv_activate_opts laopts = { .exclusive = exclusive, + .noscan = noscan, + .temporary = temporary }; if (!_lv_activate(cmd, lvid_s, &laopts, 1, lv)) return_0; diff --git a/lib/activate/activate.h b/lib/activate/activate.h index 4eac320..df888cd 100644 --- a/lib/activate/activate.h +++ b/lib/activate/activate.h @@ -38,7 +38,18 @@ struct lv_activate_opts { int skip_in_use; unsigned revert; unsigned read_only; - unsigned noscan; + unsigned noscan; /* Mark this LV to avoid its scanning. This also + directs udev to use proper udev flag to avoid + any scanning in udev. This udev flag is automatically + dropped in udev db on any spurious event that follows. */ + unsigned temporary; /* Mark this LV as temporary. It means, the LV + * is created, used and deactivated within single + * LVM command execution. Such LVs are mostly helper + * LVs to do some action or cleanup before the proper + * LV is created. This also directs udev to use proper + * set of flags to avoid any scanning in udev. These udev + * flags are persistent in udev db for any spurious event + * that follows. */ }; /* target attribute flags */ @@ -81,9 +92,10 @@ int lv_suspend_if_active(struct cmd_context *cmd, const char *lvid_s, unsigned o int lv_resume(struct cmd_context *cmd, const char *lvid_s, unsigned origin_only, struct logical_volume *lv); int lv_resume_if_active(struct cmd_context *cmd, const char *lvid_s, unsigned origin_only, unsigned exclusive, unsigned revert, struct logical_volume *lv); -int lv_activate(struct cmd_context *cmd, const char *lvid_s, int exclusive, int noscan, struct logical_volume *lv); -int lv_activate_with_filter(struct cmd_context *cmd, const char *lvid_s, - int exclusive, int noscan, struct logical_volume *lv); +int lv_activate(struct cmd_context *cmd, const char *lvid_s, int exclusive, + int noscan, int temporary, struct logical_volume *lv); +int lv_activate_with_filter(struct cmd_context *cmd, const char *lvid_s, int exclusive, + int noscan, int temporary, struct logical_volume *lv); int lv_deactivate(struct cmd_context *cmd, const char *lvid_s, struct logical_volume *lv); int lv_mknodes(struct cmd_context *cmd, const struct logical_volume *lv); diff --git a/lib/activate/dev_manager.c b/lib/activate/dev_manager.c index 0f5a04c..0b911f2 100644 --- a/lib/activate/dev_manager.c +++ b/lib/activate/dev_manager.c @@ -1400,7 +1400,7 @@ static int _check_udev_fallback(struct cmd_context *cmd) #endif /* UDEV_SYNC_SUPPORT */ static uint16_t _get_udev_flags(struct dev_manager *dm, struct logical_volume *lv, - const char *layer, uint16_t flags) + const char *layer, int noscan, int temporary) { uint16_t udev_flags = 0; @@ -1449,9 +1449,14 @@ static uint16_t _get_udev_flags(struct dev_manager *dm, struct logical_volume *l DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG; /* - * Firmly set requested flags. + * LVM subsystem specific flags. */ - udev_flags |= flags; + if (noscan) + udev_flags |= DM_SUBSYSTEM_UDEV_FLAG0; + + if (temporary) + udev_flags |= DM_UDEV_DISABLE_DISK_RULES_FLAG | + DM_UDEV_DISABLE_OTHER_RULES_FLAG; return udev_flags; } @@ -1499,7 +1504,7 @@ static int _add_dev_to_dtree(struct dev_manager *dm, struct dm_tree *dtree, } if (info.exists && !dm_tree_add_dev_with_udev_flags(dtree, info.major, info.minor, - _get_udev_flags(dm, lv, layer, 0))) { + _get_udev_flags(dm, lv, layer, 0, 0))) { log_error("Failed to add device (%" PRIu32 ":%" PRIu32") to dtree", info.major, info.minor); return 0; @@ -2340,7 +2345,7 @@ static int _set_udev_flags_for_children(struct dev_manager *dm, } dm_tree_node_set_udev_flags(child, - _get_udev_flags(dm, lvl->lv, NULL, 0)); + _get_udev_flags(dm, lvl->lv, NULL, 0, 0)); } return 1; @@ -2417,8 +2422,7 @@ static int _add_new_lv_to_dtree(struct dev_manager *dm, struct dm_tree *dtree, read_only_lv(lv, laopts), ((lv->vg->status & PRECOMMITTED) | laopts->revert) ? 1 : 0, lvlayer, - _get_udev_flags(dm, lv, layer, - laopts->noscan ? LVM_UDEV_NOSCAN_FLAG : 0)))) + _get_udev_flags(dm, lv, layer, laopts->noscan, laopts->temporary)))) return_0; /* Store existing name so we can do rename later */ diff --git a/lib/format_text/flags.c b/lib/format_text/flags.c index a27b791..e31429e 100644 --- a/lib/format_text/flags.c +++ b/lib/format_text/flags.c @@ -62,6 +62,7 @@ static const struct flag _lv_flags[] = { {LV_WRITEMOSTLY, "WRITEMOSTLY", STATUS_FLAG}, {LV_ACTIVATION_SKIP, "ACTIVATION_SKIP", COMPATIBLE_FLAG}, {LV_NOSCAN, NULL, 0}, + {LV_TEMPORARY, NULL, 0}, {POOL_METADATA_SPARE, NULL, 0}, {RAID, NULL, 0}, {RAID_META, NULL, 0}, diff --git a/lib/locking/file_locking.c b/lib/locking/file_locking.c index b6b2b10..ab3dabd 100644 --- a/lib/locking/file_locking.c +++ b/lib/locking/file_locking.c @@ -305,7 +305,8 @@ static int _file_lock_resource(struct cmd_context *cmd, const char *resource, break; case LCK_READ: log_very_verbose("Locking LV %s (R)", resource); - if (!lv_activate_with_filter(cmd, resource, 0, lv->status & LV_NOSCAN ? 1 : 0, lv_ondisk(lv))) + if (!lv_activate_with_filter(cmd, resource, 0, lv->status & LV_NOSCAN ? 1 : 0, + lv->status & LV_TEMPORARY ? 1 : 0, lv_ondisk(lv))) return 0; break; case LCK_PREAD: @@ -318,7 +319,8 @@ static int _file_lock_resource(struct cmd_context *cmd, const char *resource, break; case LCK_EXCL: log_very_verbose("Locking LV %s (EX)", resource); - if (!lv_activate_with_filter(cmd, resource, 1, lv->status & LV_NOSCAN ? 1 : 0, lv_ondisk(lv))) + if (!lv_activate_with_filter(cmd, resource, 1, lv->status & LV_NOSCAN ? 1 : 0, + lv->status & LV_TEMPORARY ? 1 : 0, lv_ondisk(lv))) return 0; break; default: diff --git a/lib/locking/no_locking.c b/lib/locking/no_locking.c index 4772706..dac2f80 100644 --- a/lib/locking/no_locking.c +++ b/lib/locking/no_locking.c @@ -48,11 +48,13 @@ static int _no_lock_resource(struct cmd_context *cmd, const char *resource, case LCK_UNLOCK: return lv_resume_if_active(cmd, resource, (flags & LCK_ORIGIN_ONLY) ? 1: 0, 0, (flags & LCK_REVERT) ? 1 : 0, lv_ondisk(lv)); case LCK_READ: - return lv_activate_with_filter(cmd, resource, 0, lv->status & LV_NOSCAN ? 1 : 0, lv_ondisk(lv)); + return lv_activate_with_filter(cmd, resource, 0, lv->status & LV_NOSCAN ? 1 : 0, + lv->status & LV_TEMPORARY ? 1 : 0, lv_ondisk(lv)); case LCK_WRITE: return lv_suspend_if_active(cmd, resource, (flags & LCK_ORIGIN_ONLY) ? 1 : 0, 0, lv_ondisk(lv), lv); case LCK_EXCL: - return lv_activate_with_filter(cmd, resource, 1, lv->status & LV_NOSCAN ? 1 : 0, lv_ondisk(lv)); + return lv_activate_with_filter(cmd, resource, 1, lv->status & LV_NOSCAN ? 1 : 0, + lv->status & LV_TEMPORARY ? 1 : 0, lv_ondisk(lv)); default: break; } diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c index 383bab1..88fceb2 100644 --- a/lib/metadata/lv_manip.c +++ b/lib/metadata/lv_manip.c @@ -5987,6 +5987,9 @@ static struct logical_volume *_lv_create_an_lv(struct volume_group *vg, if (lp->zero) lv->status |= LV_NOSCAN; + if (lp->temporary) + lv->status |= LV_TEMPORARY; + if (lv_is_thin_pool(lv)) { if (is_change_activating(lp->activate)) { if (vg_is_clustered(lv->vg)) { diff --git a/lib/metadata/metadata-exported.h b/lib/metadata/metadata-exported.h index 1e9543a..c00e4e5 100644 --- a/lib/metadata/metadata-exported.h +++ b/lib/metadata/metadata-exported.h @@ -102,7 +102,12 @@ #define LV_WRITEMOSTLY UINT64_C(0x0000020000000000) /* LV (RAID1) */ #define LV_ACTIVATION_SKIP UINT64_C(0x0000040000000000) /* LV */ -#define LV_NOSCAN UINT64_C(0x0000080000000000) /* LV */ +#define LV_NOSCAN UINT64_C(0x0000080000000000) /* LV - internal use only - the LV + should not be scanned */ +#define LV_TEMPORARY UINT64_C(0x0000100000000000) /* LV - internal use only - the LV + is supposed to be created and + removed during single LVM + command execution. */ /* Format features flags */ #define FMT_SEGMENTS 0x00000001U /* Arbitrary segment params? */ @@ -721,6 +726,7 @@ struct lvcreate_params { int log_count; /* mirror */ int nosync; /* mirror */ int poolmetadataspare; /* thin pool */ + int temporary; /* temporary LV */ #define ACTIVATION_SKIP_SET 0x01 /* request to set LV activation skip flag state */ #define ACTIVATION_SKIP_SET_ENABLED 0x02 /* set the LV activation skip flag state to 'enabled' */ #define ACTIVATION_SKIP_IGNORE 0x04 /* request to ignore LV activation skip flag (if any) */ diff --git a/lib/metadata/thin_manip.c b/lib/metadata/thin_manip.c index a6e0fc2..bd5b117 100644 --- a/lib/metadata/thin_manip.c +++ b/lib/metadata/thin_manip.c @@ -809,6 +809,7 @@ static struct logical_volume *_alloc_pool_metadata_spare(struct volume_group *vg .stripes = 1, .vg_name = vg->name, .zero = 1, + .temporary = 1, }; dm_list_init(&lp.tags);

1 0

master - cleanup: for commit 546db1c
by Peter Rajnoha 23 Oct '13

23 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=1dd6626696b100c8… Commit: 1dd6626696b100c8cf176921917418bb8d7f8055 Parent: 9883bffb0403d743641bb0971332b5ff0e373652 Author: Peter Rajnoha <prajnoha(a)redhat.com> AuthorDate: Wed Oct 23 09:48:33 2013 +0200 Committer: Peter Rajnoha <prajnoha(a)redhat.com> CommitterDate: Wed Oct 23 09:48:33 2013 +0200 cleanup: for commit 546db1c - properly clean lvm2-pvscan@.service on distclean - use @sbindir@ for sbin path in ExecStop --- scripts/Makefile.in | 2 +- scripts/lvm2_pvscan_systemd_red_hat@.service.in | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/Makefile.in b/scripts/Makefile.in index fac7e40..3616afa 100644 --- a/scripts/Makefile.in +++ b/scripts/Makefile.in @@ -119,7 +119,7 @@ DISTCLEAN_TARGETS += clvmd_init_red_hat cmirrord_init_red_hat \ dm_event_systemd_red_hat.socket dm_event_systemd_red_hat.service \ lvm2_monitoring_systemd_red_hat.service \ lvm2_lvmetad_systemd_red_hat.socket lvm2_lvmetad_systemd_red_hat.service \ - lvm2_lvmetad_systemd_red_hat@.service \ + lvm2_pvscan_systemd_red_hat@.service \ lvm2_tmpfiles_red_hat.conf blk_availability_init_red_hat \ blk_availability_systemd_red_hat.service \ blkdeactivate.sh diff --git a/scripts/lvm2_pvscan_systemd_red_hat@.service.in b/scripts/lvm2_pvscan_systemd_red_hat@.service.in index 07ea5ca..4225982 100644 --- a/scripts/lvm2_pvscan_systemd_red_hat@.service.in +++ b/scripts/lvm2_pvscan_systemd_red_hat@.service.in @@ -11,4 +11,4 @@ Conflicts=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=@sbindir@/pvscan --cache --activate ay /dev/block/%i -ExecStop=/usr/sbin/pvscan --cache %i +ExecStop=@sbindir@/pvscan --cache %i

1 0

master - WHATS_NEW: typo
by Peter Rajnoha 22 Oct '13

22 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=9883bffb0403d743… Commit: 9883bffb0403d743641bb0971332b5ff0e373652 Parent: b109bfc1ef7e20a72170a83dc9244c8b21ea658c Author: Peter Rajnoha <prajnoha(a)redhat.com> AuthorDate: Tue Oct 22 16:37:02 2013 +0200 Committer: Peter Rajnoha <prajnoha(a)redhat.com> CommitterDate: Tue Oct 22 16:37:02 2013 +0200 WHATS_NEW: typo --- WHATS_NEW | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index c2e511f..7cfa87c 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,6 +1,6 @@ Version 2.02.104 =================================== - Fix endless loop for blkdeactivte <device>... if unable to umount/deactivate. + Fix endless loop in blkdeactivate <device>... if unable to umount/deactivate. Add dev-block-<major>:<minor>.device systemd alias for complete PV tracking. Use major:minor as short form of --major and --minor arg for pvscan --cache. Remove 2>/dev/null from three lvm commands executed by vgimportclone.

1 0

master - blkdeactivate: fix endless loop if device(s) given and unable to umount/deactivate
by Peter Rajnoha 22 Oct '13

22 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=b109bfc1ef7e20a7… Commit: b109bfc1ef7e20a72170a83dc9244c8b21ea658c Parent: c3f44a0c66abf8916bebafe2de9da30132cea9e2 Author: Peter Rajnoha <prajnoha(a)redhat.com> AuthorDate: Tue Oct 22 16:10:20 2013 +0200 Committer: Peter Rajnoha <prajnoha(a)redhat.com> CommitterDate: Tue Oct 22 16:24:39 2013 +0200 blkdeactivate: fix endless loop if device(s) given and unable to umount/deactivate The blkdeactivate script iterates over the list of devices if they're given as an argument and it tries to umount/deactivate them one by one. This iteration failed to proceed if any of the umount/deactivation was unsuccessful - there was a missing "shift" call to move to the next argument (device) for processing. As a result of this, the same device was tried again and again, causing an endless loop, never proceeding to the next device given. --- WHATS_NEW | 1 + scripts/blkdeactivate.sh.in | 5 ++++- 2 files changed, 5 insertions(+), 1 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index 2d7792b..c2e511f 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ Version 2.02.104 =================================== + Fix endless loop for blkdeactivte <device>... if unable to umount/deactivate. Add dev-block-<major>:<minor>.device systemd alias for complete PV tracking. Use major:minor as short form of --major and --minor arg for pvscan --cache. Remove 2>/dev/null from three lvm commands executed by vgimportclone. diff --git a/scripts/blkdeactivate.sh.in b/scripts/blkdeactivate.sh.in index b6d0117..f454154 100644 --- a/scripts/blkdeactivate.sh.in +++ b/scripts/blkdeactivate.sh.in @@ -323,7 +323,10 @@ deactivate_all() { $LSBLK_READ <<< "`$LSBLK --nodeps $1`" # check if the device is not on the skip list already - test -z ${SKIP_DEVICE_LIST["$kname"]} || continue + test -z ${SKIP_DEVICE_LIST["$kname"]} || { + shift + continue + } deactivate else

1 0

master - make: correct sed line in udev's Makefile
by Peter Rajnoha 22 Oct '13

22 Oct '13

Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=c3f44a0c66abf891… Commit: c3f44a0c66abf8916bebafe2de9da30132cea9e2 Parent: 3fee6610286f83737d193c9047e24e8d7c85d1ce Author: Peter Rajnoha <prajnoha(a)redhat.com> AuthorDate: Tue Oct 22 15:13:58 2013 +0200 Committer: Peter Rajnoha <prajnoha(a)redhat.com> CommitterDate: Tue Oct 22 15:13:58 2013 +0200 make: correct sed line in udev's Makefile --- udev/Makefile.in | 4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/udev/Makefile.in b/udev/Makefile.in index 8f0dc8a..40a4671 100644 --- a/udev/Makefile.in +++ b/udev/Makefile.in @@ -47,9 +47,7 @@ BLKID_RULE=IMPORT{program}=\"${SBIN}\/blkid -o udev -p \$$tempnode\" endif ifeq ("@UDEV_SYSTEMD_BACKGROUND_JOBS@", "yes") -PVSCAN_RULE="ENV{SYSTEMD_ALIAS}=\"/dev/block/$$major:$$minor\" \ - ENV{ID_MODEL}=\"LVM PV $$env{ID_FS_UUID_ENC} on /dev/$$name\" \ - ENV{SYSTEMD_WANTS}=\"lvm2-pvscan(a)\$$devnode.service\"" +PVSCAN_RULE=ENV{SYSTEMD_ALIAS}=\"\/dev\/block\/\$$major:\$$minor\"\nENV{ID_MODEL}=\"LVM PV \$$env{ID_FS_UUID_ENC} on \/dev\/\$$name\"\nENV{SYSTEMD_WANTS}=\"lvm2-pvscan@\$$major:\$$minor.service\" else PVSCAN_RULE=RUN\+\=\"$(LVM_EXEC)/lvm pvscan --background --cache --activate ay --major \$$major --minor \$$minor\", ENV{LVM_SCANNED}=\"1\" endif

1 0

← Newer
1
2
3
4
5
6
7
8
9
Older →

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

lvm2-commits October 2013