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

[Julien Grall <julien@xxxxxxx>]

Hi Andrew,

On 23/02/2022 19:30, Andrew Cooper wrote:
> On 23/02/2022 18:38, Julien Grall wrote:
> > From: Julien Grall<jgrall@xxxxxxxxxx>
> >
> > 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<jgrall@xxxxxxxxxx>
> Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Thanks!

Cheers,

--
Julien Grall