diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2016-01-08 11:07:47 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2016-01-14 11:34:05 +0100 |
commit | 4310dfd4e141bb83320148d245f3ab2cae778f6e (patch) | |
tree | ba0de3652cd0fa32a3ec6eed75b1a3f48c12e06b | |
parent | ebcfd09ba9631b52762b4932fbe06260326b9827 (diff) | |
download | lvm2-4310dfd4e141bb83320148d245f3ab2cae778f6e.tar.gz |
cleanup: simplier formula
-rw-r--r-- | lib/metadata/lv_manip.c | 21 |
1 files changed, 7 insertions, 14 deletions
diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c index 62564c572..1396b867a 100644 --- a/lib/metadata/lv_manip.c +++ b/lib/metadata/lv_manip.c @@ -4950,21 +4950,14 @@ static int _lvresize_adjust_extents(struct cmd_context *cmd, struct logical_volu log_print_unless_silent("Ignoring stripes, stripesize and mirrors " "arguments when reducing."); - if (lp->sign == SIGN_MINUS) - if (lp->extents_are_pes) { - if (lp->extents >= existing_physical_extents) { - log_error("Unable to reduce %s below 1 extent.", lp->lv_name); - return 0; - } - new_extents = existing_physical_extents - lp->extents; - } else { - new_extents = existing_logical_extents - lp->extents; - if (lp->extents >= existing_logical_extents) { - log_error("Unable to reduce %s below 1 extent.", lp->lv_name); - return 0; - } + if (lp->sign == SIGN_MINUS) { + new_extents = lp->extents_are_pes ? existing_physical_extents : existing_logical_extents; + if (lp->extents >= new_extents) { + log_error("Unable to reduce %s below 1 extent.", lp->lv_name); + return 0; } - else + new_extents -= lp->extents; + } else new_extents = lp->extents; dm_list_iterate_items(seg, &lv->segments) { |