|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [xen staging] xen/mm: Remove always true ASSERT() in free_heap_pages()
commit f1097988b9baeb4cae38a1a71e027794291213ff
Author: Julien Grall <jgrall@xxxxxxxxxx>
AuthorDate: Wed Feb 23 18:38:31 2022 +0000
Commit: Julien Grall <jgrall@xxxxxxxxxx>
CommitDate: Thu Feb 24 14:57:55 2022 +0000
xen/mm: Remove always true ASSERT() in free_heap_pages()
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
Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
Acked-by: Andrew Cooper
<andrew.cooper3@xxxxxxxxxx><mailto:andrew.cooper3@xxxxxxxxxx>
Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
---
xen/common/page_alloc.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/xen/common/page_alloc.c b/xen/common/page_alloc.c
index 4635718237..e971bf91e0 100644
--- a/xen/common/page_alloc.c
+++ b/xen/common/page_alloc.c
@@ -1441,7 +1441,6 @@ static void free_heap_pages(
unsigned int zone = page_to_zone(pg);
ASSERT(order <= MAX_ORDER);
- ASSERT(node >= 0);
spin_lock(&heap_lock);
--
generated by git-patchbot for /home/xen/git/xen.git#staging
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |