diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2015-10-25 19:13:53 +0100 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2015-10-25 21:01:54 +0100 |
| commit | b780d329aa9ce249bf5164154c81cefd9b276970 (patch) | |
| tree | a79dcd0daf2f6010f8eaf2f44a361daf12fff818 | |
| parent | 8f269697d21035035ffb4954f31bd702a6dd7ca8 (diff) | |
| download | lvm2-b780d329aa9ce249bf5164154c81cefd9b276970.tar.gz | |
thin: fix percentage compare
Since plugin's percentage compare has been fixed,
it's now revealed wrong compare here.
The logic for threshold is - to allow to go as high
as given value e.g. 80% - so if pool is exactlu 80%
full it's still allowed to use it (dmeventd will not
resize it).
| -rw-r--r-- | WHATS_NEW | 1 | ||||
| -rw-r--r-- | lib/metadata/thin_manip.c | 4 |
2 files changed, 3 insertions, 2 deletions
@@ -1,5 +1,6 @@ Version 2.02.133 - ====================================== + Correct percentage evaluation when checking thin-pool over threshold. Fix lvmcache to move PV from VG to orphans if VG is removed and lvmetad used. Fix lvmcache to not cache even invalid info about PV which got removed. Support checking of memlock daemon counter. diff --git a/lib/metadata/thin_manip.c b/lib/metadata/thin_manip.c index 37bf4cbea..6b22ce5db 100644 --- a/lib/metadata/thin_manip.c +++ b/lib/metadata/thin_manip.c @@ -224,7 +224,7 @@ int pool_below_threshold(const struct lv_segment *pool_seg) if (!lv_thin_pool_percent(pool_seg->lv, 0, &percent)) return_0; - if (percent >= threshold) { + if (percent > threshold) { log_debug("Threshold configured for free data space in " "thin pool %s has been reached (%.2f%% >= %.2f%%).", display_lvname(pool_seg->lv), @@ -237,7 +237,7 @@ int pool_below_threshold(const struct lv_segment *pool_seg) if (!lv_thin_pool_percent(pool_seg->lv, 1, &percent)) return_0; - if (percent >= threshold) { + if (percent > threshold) { log_debug("Threshold configured for free metadata space in " "thin pool %s has been reached (%.2f%% > %.2f%%).", display_lvname(pool_seg->lv), |