Rename vg_change_pesize to vg_set_extent_size and use vg_t.

In liblvm, we will reserve the word 'change' to mean an API that
both sets one or more values, and commits to disk.  This will be
consistent with the LVM commandline.  The existing vg_change_pesize()
function does not commit to disk, but just changes the extent_size
and ensures all internal structures are updated.  This logic should
be contained in a function that sets the value.

Signed-off-by: Dave Wysochanski <dwysocha@redhat.com>

diff --git a/lib/metadata/metadata-exported.h b/lib/metadata/metadata-exported.h
index 07b7eae199796a2883c4165b6ceb8cca5dbb76d3..bdad97f1857febcac802cc0d67a21a081a1b766c 100644 (file)
--- a/lib/metadata/metadata-exported.h
+++ b/lib/metadata/metadata-exported.h
@@ -434,7 +434,7 @@ int vg_remove_single(struct cmd_context *cmd, const char *vg_name,
 int vg_rename(struct cmd_context *cmd, struct volume_group *vg,
              const char *new_name);
 int vg_extend(struct volume_group *vg, int pv_count, char **pv_names);
-int vg_change_pesize(struct volume_group *vg, uint32_t new_extent_size);
+int vg_set_extent_size(vg_t *vg, uint32_t new_extent_size);
 int vg_split_mdas(struct cmd_context *cmd, struct volume_group *vg_from,
                  struct volume_group *vg_to);
 

diff --git a/lib/metadata/metadata.c b/lib/metadata/metadata.c
index 41ca6645f1d37c5bdcdf85c24d2e6e0e28df38b8..c2dbf54ed952001d1d086d4b326803c2a85347cf 100644 (file)
--- a/lib/metadata/metadata.c
+++ b/lib/metadata/metadata.c
@@ -624,7 +624,7 @@ static int _recalc_extents(uint32_t *extents, const char *desc1,
        return 1;
 }
 
-int vg_change_pesize(struct volume_group *vg, uint32_t new_size)
+int vg_set_extent_size(vg_t *vg, uint32_t new_size)
 {
        uint32_t old_size = vg->extent_size;
        struct pv_list *pvl;

diff --git a/tools/vgchange.c b/tools/vgchange.c
index ade0735d1546421fa247d45af5b4db91f436de14..a2b8aa9b699e6e77347afdac6961604f1ff840e1 100644 (file)
--- a/tools/vgchange.c
+++ b/tools/vgchange.c
@@ -397,7 +397,7 @@ static int _vgchange_pesize(struct cmd_context *cmd, struct volume_group *vg)
        if (!archive(vg))
                return ECMD_FAILED;
 
-       if (!vg_change_pesize(vg, extent_size)) {
+       if (!vg_set_extent_size(vg, extent_size)) {
                stack;
                return EINVALID_CMD_LINE;
        }