Re: [Xen-devel] [PATCH] remove dead code in allocate_memory

2018-01-18 Thread Stefano Stabellini
On Mon, 15 Jan 2018, Julien Grall wrote: > Hi Stefano, > > On 01/13/2018 12:29 AM, Stefano Stabellini wrote: > > lowmem is unchanged until after this loop, there is no point in testing > > for its value. Moreover, Coverity complains about dead code. > > > > Remove the lowmem test in the first

Re: [Xen-devel] [PATCH] remove dead code in allocate_memory

2018-01-15 Thread Julien Grall
Hi Stefano, On 01/13/2018 12:29 AM, Stefano Stabellini wrote: lowmem is unchanged until after this loop, there is no point in testing for its value. Moreover, Coverity complains about dead code. Remove the lowmem test in the first loop. While I understand Coverity complains about dead code,

[Xen-devel] [PATCH] remove dead code in allocate_memory

2018-01-12 Thread Stefano Stabellini
lowmem is unchanged until after this loop, there is no point in testing for its value. Moreover, Coverity complains about dead code. Remove the lowmem test in the first loop. CID: 1381832 Signed-off-by: Stefano Stabellini diff --git a/xen/arch/arm/domain_build.c