From mboxrd@z Thu Jan 1 00:00:00 1970 From: Robert Ho Subject: [OSSTest Nested v12 20/21] Don't lvextend if actually no more space to extend Date: Fri, 28 Aug 2015 23:08:09 +0800 Message-ID: <1440774490-16725-21-git-send-email-robert.hu@intel.com> References: <1440774490-16725-1-git-send-email-robert.hu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1440774490-16725-1-git-send-email-robert.hu@intel.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xen.org, robert.hu@intel.com Cc: wei.liu2@citrix.com, Ian.Jackson@eu.citrix.com, ian.campbell@citrix.com, di.zheng@intel.com, gordon.jin@intel.com List-Id: xen-devel@lists.xenproject.org Though passes if judgement, the overall_limit_pe(\$vg_more_free_pe); may final judge no more free_pe to extend. So, check if $vg_more_free_pe is 0, if so, we don't lvextend, otherwise lvextend will report error on nonsense operation. Signed-off-by: Robert Ho --- ts-xen-build-prep | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ts-xen-build-prep b/ts-xen-build-prep index 03ad35c..d3f9d01 100755 --- a/ts-xen-build-prep +++ b/ts-xen-build-prep @@ -155,7 +155,8 @@ sub lvextend1 ($$$) { logm("$what: unstriped $vg_more_free_pe PEs"); overall_limit_pe(\$vg_more_free_pe); $more_pe += $vg_more_free_pe; - target_cmd_root($ho, "lvextend -i1 -l +$vg_more_free_pe $lv"); + target_cmd_root($ho, "lvextend -i1 -l +$vg_more_free_pe $lv") + if $vg_more_free_pe != 0; } my $timeout = 2000 + int($pe_size * 0.000003 * $more_pe); -- 1.8.3.1