Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=4047a321285fc0d1a70fd…
Commit: 4047a321285fc0d1a70fd519fb53fc778fe6093b
Parent: a509776588a5c0c0bfc2394e4d1ed717531b0257
Author: David Teigland <teigland(a)redhat.com>
AuthorDate: Wed Apr 22 14:08:54 2020 -0500
Committer: David Teigland <teigland(a)redhat.com>
CommitterDate: Wed Apr 22 14:08:54 2020 -0500
use refresh_filters only where needed
Filters are changed and need refresh in only one
place (vgimportclone), so avoid doing the refresh
for every other command that doesn't need it.
---
lib/cache/lvmcache.c | 10 ----------
tools/vgimportclone.c | 4 +++-
2 files changed, 3 insertions(+), 11 deletions(-)
diff --git a/lib/cache/lvmcache.c b/lib/cache/lvmcache.c
index bcb893346..b8575ff62 100644
--- a/lib/cache/lvmcache.c
+++ b/lib/cache/lvmcache.c
@@ -1032,15 +1032,6 @@ int lvmcache_label_scan(struct cmd_context *cmd)
log_debug_cache("Finding VG info");
- /* FIXME: can this happen? */
- if (!cmd->filter) {
- log_error("label scan is missing filter");
- goto out;
- }
-
- if (!refresh_filters(cmd))
- log_error("Scan failed to refresh device filter.");
-
/*
* Duplicates found during this label scan are added to _initial_duplicates.
*/
@@ -1103,7 +1094,6 @@ int lvmcache_label_scan(struct cmd_context *cmd)
r = 1;
- out:
dm_list_iterate_items(vginfo, &_vginfos) {
if (is_orphan_vg(vginfo->vgname))
continue;
diff --git a/tools/vgimportclone.c b/tools/vgimportclone.c
index be0186101..ee1c28fae 100644
--- a/tools/vgimportclone.c
+++ b/tools/vgimportclone.c
@@ -315,6 +315,8 @@ retry_name:
goto_out;
log_debug("Using new VG name %s.", vp.new_vgname);
+ lvmcache_destroy(cmd, 1, 0);
+
/*
* Create a device filter so that we are only working with the devices
* in arg_import. With the original devs hidden (that arg_import were
@@ -325,7 +327,7 @@ retry_name:
init_internal_filtering(1);
dm_list_iterate_items(vd, &vp.arg_import)
internal_filter_allow(cmd->mem, vd->dev);
- lvmcache_destroy(cmd, 1, 0);
+ refresh_filters(cmd);
log_debug("Changing VG %s to %s.", vp.old_vgname, vp.new_vgname);
Gitweb: https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=a509776588a5c0c0bfc23…
Commit: a509776588a5c0c0bfc2394e4d1ed717531b0257
Parent: d79afd408446e3eb922d16954ae630a7af995e66
Author: Maxim Plotnikov <wgh(a)torlan.ru>
AuthorDate: Wed Apr 22 00:16:29 2020 +0300
Committer: David Teigland <teigland(a)redhat.com>
CommitterDate: Tue Apr 21 16:52:45 2020 -0500
Fix scripts/lvmlocks.service.in using nonexistent --lock-opt autowait
The --lock-opt autowait was dropped back in 9ab6bdce01,
and attempting to specify it has quite an opposite effect:
no waiting is done, which makes the unit almost useless.
---
scripts/lvmlocks.service.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/lvmlocks.service.in b/scripts/lvmlocks.service.in
index f6a951f19..a3d0bd48d 100644
--- a/scripts/lvmlocks.service.in
+++ b/scripts/lvmlocks.service.in
@@ -8,7 +8,7 @@ Type=oneshot
RemainAfterExit=yes
# start lockspaces and wait for them to finish starting
-ExecStart=@SBINDIR@/lvm vgchange --lock-start --lock-opt autowait
+ExecStart=@SBINDIR@/lvm vgchange --lock-start --lock-opt auto
# stop lockspaces and wait for them to finish stopping
ExecStop=@SBINDIR@/lvmlockctl --stop-lockspaces --wait 1