From 097a724bda160e4c0b5c7760e7a710d3e00b898c Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Thu, 7 Apr 2016 11:12:49 +0200 Subject: [PATCH] lvcreate: %FREE -> %PVS This is hotfix for RHBZ: https://bugzilla.redhat.com/1324537 However already the %FREE is not a good fit and we need something better. Meanwhile make -l%PVS work at least as good as %FREE for thin-pool. TODO: this needs rework - it should be allocator to do all the size decisions at one place. --- WHATS_NEW | 1 + tools/lvcreate.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/WHATS_NEW b/WHATS_NEW index 0c940b60e..8387eabc6 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,5 +1,6 @@ Version 2.02.150 - ================================= + Reuse %FREE size aproximation with lvcreate -l%PVS thin-pool. Allow the lvmdump directory to exist already provided it is empty. Show lvconverted percentage with 2 decimal digits. Fix regression in suspend when repairing --type mirror (2.02.133). diff --git a/tools/lvcreate.c b/tools/lvcreate.c index 283e52a79..97147899a 100644 --- a/tools/lvcreate.c +++ b/tools/lvcreate.c @@ -368,7 +368,7 @@ static int _update_extents_params(struct volume_group *vg, &lp->discards, &lp->zero)) return_0; - if (lcp->percent == PERCENT_FREE) { + if (lcp->percent == PERCENT_FREE || lcp->percent == PERCENT_PVS) { if (lp->extents <= (2 * lp->pool_metadata_extents)) { log_error("Not enough space for thin pool creation."); return 0;