Re: [Xen-devel] [OSSTest Nested v12 20/21] Don't lvextend if actually no more space to extend

2015-09-22 Thread Ian Campbell
On Wed, 2015-09-16 at 15:27 +0100, Ian Jackson wrote: > Robert Ho writes ("[OSSTest Nested v12 20/21] Don't lvextend if actually > no more space to extend"): > > Though passes if judgement, the > > overall_limit_pe(\$vg_more_free_pe); > > may final judge no more free_pe to extend. > > So,

Re: [Xen-devel] [OSSTest Nested v12 20/21] Don't lvextend if actually no more space to extend

2015-09-22 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTest Nested v12 20/21] Don't lvextend if actually no more space to extend"): > On Wed, 2015-09-16 at 15:27 +0100, Ian Jackson wrote: > > I think this should be done by moving the start of the if block to > > after overall_limit_pe. That would avoid a tested if (and

Re: [Xen-devel] [OSSTest Nested v12 20/21] Don't lvextend if actually no more space to extend

2015-09-16 Thread Ian Jackson
Robert Ho writes ("[OSSTest Nested v12 20/21] Don't lvextend if actually no more space to extend"): > 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, >