diff options
author | David Teigland <teigland@redhat.com> | 2022-01-12 16:42:01 -0600 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2022-01-12 16:42:01 -0600 |
commit | 5e428d22d9647a294eba65852078947df5ade48f (patch) | |
tree | 87ba7879896b81d19a21991b8caa19cd477399dd | |
parent | 7502f78678db7c7701e543a0a7267e050850ba53 (diff) | |
download | lvm2-5e428d22d9647a294eba65852078947df5ade48f.tar.gz |
vgsplit: don't reread vg_to
The destination vg is first written with the EXPORTED flag,
then the source vg is written, then the destination vg is
written again without the EXPORTED flag. Remove an unnecessary
vg_read of the destination vg just before the second write.
-rw-r--r-- | tools/vgsplit.c | 24 |
1 files changed, 0 insertions, 24 deletions
diff --git a/tools/vgsplit.c b/tools/vgsplit.c index a085ac2ba..5f113b363 100644 --- a/tools/vgsplit.c +++ b/tools/vgsplit.c @@ -727,30 +727,6 @@ int vgsplit(struct cmd_context *cmd, int argc, char **argv) backup(vg_from); } - /* - * Finally, remove the EXPORTED flag from the new VG and write it out. - * We need to unlock vg_to because vg_read_for_update wants to lock it. - */ - if (!test_mode()) { - unlock_vg(cmd, NULL, vg_name_to); - release_vg(vg_to); - - /* - * This command uses the exported vg flag internally, but - * exported VGs are not allowed to be split from the command - * level, so ALLOW_EXPORTED is not set in commands.h. - */ - cmd->include_exported_vgs = 1; - - vg_to = vg_read_for_update(cmd, vg_name_to, NULL, 0, 0); - - if (!vg_to) { - log_error("Volume group \"%s\" became inconsistent: " - "please fix manually", vg_name_to); - goto bad; - } - } - vg_to->status &= ~EXPORTED_VG; if (!vg_write(vg_to) || !vg_commit(vg_to)) |