diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2023-01-31 22:48:43 +0100 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2023-02-01 11:48:53 +0100 |
| commit | 94eda42e7ad7c2dbd7fedfdfbf38032857318fec (patch) | |
| tree | b1ffc8ed45c190651d1cb83485d9cc9075fc2a4e | |
| parent | 0ec087e8b82848891512f100ff594add8f47cfb2 (diff) | |
| download | lvm2-94eda42e7ad7c2dbd7fedfdfbf38032857318fec.tar.gz | |
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().
| -rw-r--r-- | tools/toollib.c | 8 |
1 files 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; |