From 94eda42e7ad7c2dbd7fedfdfbf38032857318fec Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Tue, 31 Jan 2023 22:48:43 +0100 Subject: lv/vgchange: when refreshing whole vg skip snapshot When refreshing all LVs in a VG, skip processing of thick snapshots, since they will be refreshed through its origin LV. Should reduce some unnecessary ioctl(). --- tools/toollib.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tools/toollib.c b/tools/toollib.c index d52c9e342..194088ea6 100644 --- a/tools/toollib.c +++ b/tools/toollib.c @@ -883,7 +883,9 @@ int vg_refresh_visible(struct cmd_context *cmd, struct volume_group *vg) break; } - if (lv_is_visible(lvl->lv) && !lv_refresh(cmd, lvl->lv)) { + if (lv_is_visible(lvl->lv) && + !(lv_is_cow(lvl->lv) && !lv_is_virtual_origin(origin_from_cow(lvl->lv))) && + !lv_refresh(cmd, lvl->lv)) { r = 0; stack; } @@ -3292,7 +3294,9 @@ int process_each_lv_in_vg(struct cmd_context *cmd, struct volume_group *vg, /* Skip availability change for non-virt snaps when processing all LVs */ /* FIXME: pass process_all to process_single_lv() */ - if (process_all && arg_is_set(cmd, activate_ARG) && + if (process_all && + (arg_is_set(cmd, activate_ARG) || + arg_is_set(cmd, refresh_ARG)) && lv_is_cow(lvl->lv) && !lv_is_virtual_origin(origin_from_cow(lvl->lv))) continue; -- cgit v1.2.1