cache-pool: Change segtype name from cache_pool to cache-pool

Thin pools use "thin-pool" for the segment type name.  To be consistent,
we use "cache-pool" instead of "cache_pool".

diff --git a/lib/cache_segtype/cache.c b/lib/cache_segtype/cache.c
index 90ed40e75f27ad78b9dc4f0dc390f281e0a78bb2..57c7a5cc1b3b66a5db0f56334afeaa1eb2164d9a 100644 (file)
--- a/lib/cache_segtype/cache.c
+++ b/lib/cache_segtype/cache.c
@@ -418,7 +418,7 @@ int init_cache_segtypes(struct cmd_context *cmd,
        }
        segtype->cmd = cmd;
 
-       segtype->name = "cache_pool";
+       segtype->name = "cache-pool";
        segtype->flags = SEG_CACHE_POOL;
        segtype->ops = &_cache_pool_ops;
        segtype->private = NULL;

diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c
index 5d49a7039da52a4dfd92cb6193e172a3471851e4..2b1cdccb292842179645a51916d0efe99b4bec33 100644 (file)
--- a/lib/metadata/lv_manip.c
+++ b/lib/metadata/lv_manip.c
@@ -5917,11 +5917,12 @@ static struct logical_volume *_lv_create_an_lv(struct volume_group *vg,
                }
 
                if (lp->extents > org->le_count) {
-                       log_error("Cache_Pool size cannot be larger than"
+                       log_error("cache-pool size cannot be larger than"
                                  " the origin");
                        return NULL;
                }
-               if (!(lp->segtype = get_segtype_from_string(vg->cmd, "cache_pool")))
+               if (!(lp->segtype = get_segtype_from_string(vg->cmd,
+                                                           "cache-pool")))
                        return_0;
        } else if (seg_is_thin(lp) && lp->snapshot) {
                if (!(org = find_lv(vg, lp->origin))) {

diff --git a/man/lvconvert.8.in b/man/lvconvert.8.in
index 32b251ca2ab3f696439db4767ae55779a298e42f..c496d7478abe4da5594dbc40d379ae171c04cb50 100644 (file)
--- a/man/lvconvert.8.in
+++ b/man/lvconvert.8.in
@@ -113,7 +113,7 @@ lvconvert \- convert a logical volume from linear to mirror or snapshot
 .RB [ \-v | \-\-verbose ]
 .RB [ \-\-version ]
 .sp
-.B lvconvert \-\-type cache_pool
+.B lvconvert \-\-type cache-pool
 .RB [ \-c | \-\-chunksize
 .IR ChunkSize [ bBsSkKmMgG ]]
 .RB [ \-\-cachemode
@@ -207,7 +207,7 @@ A mirror is divided into regions of this size (in MB), and the mirror log
 uses this granularity to track which regions are in sync.
 .TP
 .B \-\-type \fISegmentType
-Used to convert a logical volume to another segment type, like cache_pool,
+Used to convert a logical volume to another segment type, like cache-pool,
 cache, raid1, mirror.  When converting a logical volume to a cached logical
 volume, the \-\-cachepool argument is required.
 .TP
@@ -481,7 +481,7 @@ For the read-only external origin use the new name "vg00/external".
 Convert an existing logical volume to a cache pool logical volume using the
 given pool metadata LV.
 .sp
-.B lvconvert --type cache_pool vg00/my_lv_data \-\-poolmetadata vg00/my_lv_meta
+.B lvconvert --type cache-pool vg00/my_lv_data \-\-poolmetadata vg00/my_lv_meta
 .B lvrename vg00/my_lv_data vg00/my_lv_cachepool
 
 Convert an existing logical volume to a cached logical volume using the given

diff --git a/man/lvcreate.8.in b/man/lvcreate.8.in
index f03fd487fa897fc959c0592d08285a782aba8984..682ff3a8445dc2e1107684f535153686d587bd9d 100644 (file)
--- a/man/lvcreate.8.in
+++ b/man/lvcreate.8.in
@@ -532,7 +532,7 @@ in vg00 that will use an existing thin pool "vg00/pool":
 Create a cache pool logical volume that can later be used to cache one
 logical volume.
 .sp
-.B lvcreate --type cache_pool -L 1G -n my_lv_cachepool vg /dev/fast1
+.B lvcreate --type cache-pool -L 1G -n my_lv_cachepool vg /dev/fast1
 
 If there is an existing cache pool logical volume, create the large slow
 device (i.e. the origin) and link it to the supplied cache pool LV,

diff --git a/man/lvm.8.in b/man/lvm.8.in
index e5cfb825482be6ecc61ead087f3675aa48fa8b77..363f6af35f368e422bd1dbacc43bd0ed8512b4b3 100644 (file)
--- a/man/lvm.8.in
+++ b/man/lvm.8.in
@@ -465,7 +465,7 @@ Step-1> lvcreate -L 1G -n cachepool my_vg /dev/fast_dev
 Step-2> lvcreate -L 8M -n meta my_vg /dev/fast_dev
 
 # Combine cache pool sub-LVs into a cache pool.
-Step-3> lvconvert --type cache_pool vg/cachepool --poolmetadata vg/meta
+Step-3> lvconvert --type cache-pool vg/cachepool --poolmetadata vg/meta
 
 # Combine the cache pool and origin to create a cached LV
 Step-4> lvconvert --type cache my_vg/my_lv --cachepool vg/cachepool
@@ -490,7 +490,7 @@ Step-01> lvcreate --type raid1 -L 1G -n cachepool my_vg /dev/fast[12]
 Step-2> lvcreate --type raid1 -L 8M -n meta my_vg /dev/fast1 /dev/fast2
 
 # Combine cache pool sub-LVs into a cache pool.
-Step-3> lvconvert --type cache_pool vg/cachepool --poolmetadata vg/meta
+Step-3> lvconvert --type cache-pool vg/cachepool --poolmetadata vg/meta
 
 # Combine the cache pool and origin to create a cached LV
 Step-4> lvconvert --type cache my_vg/my_lv --cachepool vg/cachepool
@@ -517,7 +517,7 @@ Step-1> lvcreate -L 1G -n cachepool my_vg /dev/fast_dev
 Step-2> lvcreate -L 8M -n meta my_vg /dev/fast_dev
 
 # Combine cache pool sub-LVs into a cache pool and specify "writethough".
-Step-3> lvconvert --type cache_pool vg/cachepool \\
+Step-3> lvconvert --type cache-pool vg/cachepool \\
        --poolmetadata vg/meta --cachemode writethrough -n cachepool
 
 # Combine the cache pool and origin to create a cached LV

diff --git a/test/shell/lvcreate-cache.sh b/test/shell/lvcreate-cache.sh
index 01e6f510b25b9bc14b558fa38011788c76d04ee6..c4c07941b0cd759ab9a91f3d84db65aba9ec9080 100644 (file)
--- a/test/shell/lvcreate-cache.sh
+++ b/test/shell/lvcreate-cache.sh
@@ -23,7 +23,7 @@ aux prepare_vg 5 80
 ####################
 
 # Full CLI (the advertised form)
-lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 lvremove -ff $vg/${lv}_cache_pool
 
 # Shorthand CLI (not advertised) -- not yet implemented
@@ -42,34 +42,34 @@ lvremove -ff $vg/${lv}_cache_pool
 ################
 
 # Create/remove cache_pool
-lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 lvremove -ff $vg
 
 # Create cache_pool, then origin with cache, then remove all
-lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 lvcreate --type cache -l 2 $vg/${lv}_cache_pool -n $lv1
 dmsetup table ${vg}-$lv1 | grep cache  # ensure it is loaded in kernel
 lvremove -ff $vg
 
 # Create cache_pool, then origin with cache, then remove cache_pool/cache
-lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 lvcreate --type cache -l 2 $vg/${lv}_cache_pool -n $lv1
 lvremove -ff $vg/${lv}_cache_pool
 lvremove -ff $vg/$lv1
 
 # Create cache_pool, then origin with cache, then remove origin
-lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 lvcreate --type cache -l 2 $vg/${lv}_cache_pool -n $lv1
 lvremove -ff $vg/$lv1
 lvremove -ff $vg/${lv}_cache_pool
 
 # Shorthand CLI (cache_pool exists, create origin w/ cache)
-#lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+#lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 #lvcreate --cache -l 2 $vg/${lv}_cache_pool -n $lv1
 #lvremove -ff $vg
 
 # Shorthand CLI (cache_pool exists, create origin w/ cache)
-#lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+#lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 #lvcreate -H -l 2 $vg/${lv}_cache_pool -n $lv1
 #lvremove -ff $vg
 
@@ -94,22 +94,22 @@ lvremove -ff $vg
 # Repeat key tests with 'writethrough' cachemode
 ################################################
 # Create/remove cache_pool
-lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg --cachemode writethrough
+lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg --cachemode writethrough
 lvremove -ff $vg
 
 # Create cache_pool, then origin with cache, then remove all
-lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 lvcreate --type cache -l 2 $vg/${lv}_cache_pool -n $lv1 --cachemode writethrough
 lvremove -ff $vg
 
 # Create cache_pool, then origin with cache, then remove cache_pool/cache
-lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 lvcreate --type cache -l 2 $vg/${lv}_cache_pool -n $lv1 --cachemode writethrough
 lvremove -ff $vg/${lv}_cache_pool
 lvremove -ff $vg/$lv1
 
 # Create cache_pool, then origin with cache, then remove origin
-lvcreate --type cache_pool -l 1 -n ${lv}_cache_pool $vg
+lvcreate --type cache-pool -l 1 -n ${lv}_cache_pool $vg
 lvcreate --type cache -l 2 $vg/${lv}_cache_pool -n $lv1 --cachemode writethrough
 lvremove -ff $vg/$lv1
 lvremove -ff $vg/${lv}_cache_pool

diff --git a/tools/lvconvert.c b/tools/lvconvert.c
index 7fc1949127884aa3c3e1297c4fe0f46f8219c415..814ac65bc9e2a4eb06f6eb6e2f83816d4869f99d 100644 (file)
--- a/tools/lvconvert.c
+++ b/tools/lvconvert.c
@@ -202,7 +202,7 @@ static int _check_conversion_type(struct cmd_context *cmd, const char *type_str)
        /* FIXME: Check thin-pool and thin more thoroughly! */
        if (!strcmp(type_str, "snapshot") ||
            !strncmp(type_str, "raid", 4) ||
-           !strcmp(type_str, "cache_pool") || !strcmp(type_str, "cache") ||
+           !strcmp(type_str, "cache-pool") || !strcmp(type_str, "cache") ||
            !strcmp(type_str, "thin-pool") || !strcmp(type_str, "thin"))
                return 1;
 
@@ -255,7 +255,7 @@ static int _read_params(struct lvconvert_params *lp, struct cmd_context *cmd,
                return 0;
        }
 
-       if (!strcmp(type_str, "cache_pool")) {
+       if (!strcmp(type_str, "cache-pool")) {
                cache_pool = 1;
                if ((tmp_str = arg_str_value(cmd, cachemode_ARG, NULL))) {
                        if (!strcmp(tmp_str, "writeback"))
@@ -520,7 +520,7 @@ static int _read_params(struct lvconvert_params *lp, struct cmd_context *cmd,
                        }
                }
 
-               lp->segtype = get_segtype_from_string(cmd, arg_str_value(cmd, type_ARG, cache_pool ? "cache_pool" : "thin-pool"));
+               lp->segtype = get_segtype_from_string(cmd, arg_str_value(cmd, type_ARG, cache_pool ? "cache-pool" : "thin-pool"));
                if (!lp->segtype)
                        return_0;
        } else { /* Mirrors (and some RAID functions) */

diff --git a/tools/toollib.c b/tools/toollib.c
index 9662aa8bb1801042355aa2dce64d8c088b54769a..31033e61387cbb07376e9460334cabe7966f2d65 100644 (file)
--- a/tools/toollib.c
+++ b/tools/toollib.c
@@ -1662,7 +1662,7 @@ int get_pool_params(struct cmd_context *cmd,
 {
        int cache_pool = 0;
 
-       if (!strcmp("cache_pool", arg_str_value(cmd, type_ARG, "none")))
+       if (!strcmp("cache-pool", arg_str_value(cmd, type_ARG, "none")))
                cache_pool = 1;
 
        if (!cache_pool && !arg_count(cmd, thinpool_ARG)) {