diff options
author | David Teigland <teigland@redhat.com> | 2016-02-09 13:32:26 -0600 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2016-02-11 16:48:55 -0600 |
commit | d518977073e9cb05c9d3a5ff53900860f1eea4ee (patch) | |
tree | a0c83eacccd90861cdaa5ce0e83ab977d6af3da9 | |
parent | 762b0d697ff7456d0a450121e41eff00cd5a7f08 (diff) | |
download | lvm2-d518977073e9cb05c9d3a5ff53900860f1eea4ee.tar.gz |
rename function read_vgname to read_vgsummary
The name did not clearly represent what it does.
-rw-r--r-- | lib/format_text/format-text.c | 2 | ||||
-rw-r--r-- | lib/format_text/import-export.h | 8 | ||||
-rw-r--r-- | lib/format_text/import.c | 4 | ||||
-rw-r--r-- | lib/format_text/import_vsn1.c | 12 |
4 files changed, 15 insertions, 11 deletions
diff --git a/lib/format_text/format-text.c b/lib/format_text/format-text.c index b1e2ebfc9..b2b66df3f 100644 --- a/lib/format_text/format-text.c +++ b/lib/format_text/format-text.c @@ -1210,7 +1210,7 @@ int vgname_from_mda(const struct format_type *fmt, used_cached_metadata = 1; /* FIXME 64-bit */ - if (!text_vgname_import(fmt, dev_area->dev, + if (!text_vgsummary_import(fmt, dev_area->dev, (off_t) (dev_area->start + rlocn->offset), (uint32_t) (rlocn->size - wrap), (off_t) (dev_area->start + MDA_HEADER_SIZE), diff --git a/lib/format_text/import-export.h b/lib/format_text/import-export.h index 074b1181a..c081c511c 100644 --- a/lib/format_text/import-export.h +++ b/lib/format_text/import-export.h @@ -51,9 +51,9 @@ struct text_vg_version_ops { unsigned allow_lvmetad_extensions); void (*read_desc) (struct dm_pool * mem, const struct dm_config_tree *cf, time_t *when, char **desc); - int (*read_vgname) (const struct format_type *fmt, - const struct dm_config_tree *cft, - struct lvmcache_vgsummary *vgsummary); + int (*read_vgsummary) (const struct format_type *fmt, + const struct dm_config_tree *cft, + struct lvmcache_vgsummary *vgsummary); }; struct text_vg_version_ops *text_vg_vsn1_init(void); @@ -78,7 +78,7 @@ struct volume_group *text_vg_import_fd(struct format_instance *fid, uint32_t checksum, time_t *when, char **desc); -int text_vgname_import(const struct format_type *fmt, +int text_vgsummary_import(const struct format_type *fmt, struct device *dev, off_t offset, uint32_t size, off_t offset2, uint32_t size2, diff --git a/lib/format_text/import.c b/lib/format_text/import.c index 5b4f4a90a..629884bbb 100644 --- a/lib/format_text/import.c +++ b/lib/format_text/import.c @@ -35,7 +35,7 @@ static void _init_text_import(void) /* * Find out vgname on a given device. */ -int text_vgname_import(const struct format_type *fmt, +int text_vgsummary_import(const struct format_type *fmt, struct device *dev, off_t offset, uint32_t size, off_t offset2, uint32_t size2, @@ -74,7 +74,7 @@ int text_vgname_import(const struct format_type *fmt, if (!(*vsn)->check_version(cft)) continue; - if (!(*vsn)->read_vgname(fmt, cft, vgsummary)) + if (!(*vsn)->read_vgsummary(fmt, cft, vgsummary)) goto_out; r = 1; diff --git a/lib/format_text/import_vsn1.c b/lib/format_text/import_vsn1.c index 71181bde4..76442916f 100644 --- a/lib/format_text/import_vsn1.c +++ b/lib/format_text/import_vsn1.c @@ -1032,12 +1032,16 @@ static void _read_desc(struct dm_pool *mem, } /* - * It would be more accurate to call this _read_vgsummary(). * It is used to read vgsummary information about a VG * before locking and reading the VG via vg_read(). + * read_vgsummary: read VG metadata before VG is locked + * and save the data in struct vgsummary + * read_vg: read VG metadata after VG is locked + * and save the data in struct volume_group + * FIXME: why are these separate? */ -static int _read_vgname(const struct format_type *fmt, const struct dm_config_tree *cft, - struct lvmcache_vgsummary *vgsummary) +static int _read_vgsummary(const struct format_type *fmt, const struct dm_config_tree *cft, + struct lvmcache_vgsummary *vgsummary) { const struct dm_config_node *vgn; struct dm_pool *mem = fmt->cmd->mem; @@ -1088,7 +1092,7 @@ static struct text_vg_version_ops _vsn1_ops = { .check_version = _vsn1_check_version, .read_vg = _read_vg, .read_desc = _read_desc, - .read_vgname = _read_vgname, + .read_vgsummary = _read_vgsummary }; struct text_vg_version_ops *text_vg_vsn1_init(void) |