Re: [Xen-devel] [PATCH] x86/mm: Fix infinite loop in get_spage_pages()

2017-06-26 Thread Jan Beulich
>>> Andrew Cooper 06/26/17 1:59 PM >>> >c/s 2b8eb37 switched int i to being unsigned, but the undo logic on failure >relied in i being signed. As i being unsigned in still preforable, adjust the >undo logic to work with an unsigned i. > >Coverity-ID: 1413017 >Signed-off-by: Andrew Cooper Review

Re: [Xen-devel] [PATCH] x86/mm: Fix infinite loop in get_spage_pages()

2017-06-26 Thread Wei Liu
On Mon, Jun 26, 2017 at 12:59:02PM +0100, Andrew Cooper wrote: > c/s 2b8eb37 switched int i to being unsigned, but the undo logic on failure > relied in i being signed. As i being unsigned in still preforable, adjust the > undo logic to work with an unsigned i. > > Coverity-ID: 1413017 > Signed-o

Re: [Xen-devel] [PATCH] x86/mm: Fix infinite loop in get_spage_pages()

2017-06-26 Thread Konrad Rzeszutek Wilk
On June 26, 2017 7:59:02 AM EDT, Andrew Cooper wrote: >c/s 2b8eb37 switched int i to being unsigned, but the undo logic on >failure >relied in i being signed. As i being unsigned in still preforable, >adjust the >undo logic to work with an unsigned i. > >Coverity-ID: 1413017 >Signed-off-by: Andr