Re: [PATCH] xen/mm: Remove always true ASSERT() in free_heap_pages()

[Julien Grall]

Hi Andrew,

On 23/02/2022 19:30, Andrew Cooper wrote:

On 23/02/2022 18:38, Julien Grall wrote:

From: Julien Grall<jgr...@amazon.com>

free_heap_pages() has an ASSERT() checking that node is >= 0. However
node is defined as an unsigned int. So it cannot be negative.

Therefore remove the check as it will always be true.

Coverity-ID: 1055631

I will add it while committing.

Signed-off-by: Julien Grall<jgr...@amazon.com>

Acked-by: Andrew Cooper <andrew.coop...@citrix.com>

Thanks!

Cheers,

--
Julien Grall