Hi,
On 23/02/2022 19:03, Andrew Cooper wrote:
On 23/02/2022 18:41, Julien Grall wrote:
From: Julien Grall <[email protected]>
The local variable pg_offlined in free_heap_pages() can only take two
values. So switch it to a bool.
Signed-off-by: Julien Grall <[email protected]>
---
xen/common/page_alloc.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/xen/common/page_alloc.c b/xen/common/page_alloc.c
index e971bf91e0be..72e6cac2365d 100644
--- a/xen/common/page_alloc.c
+++ b/xen/common/page_alloc.c
@@ -1437,8 +1437,9 @@ static void free_heap_pages(
{
unsigned long mask;
mfn_t mfn = page_to_mfn(pg);
- unsigned int i, node = phys_to_nid(mfn_to_maddr(mfn)), pg_offlined = 0;
+ unsigned int i, node = phys_to_nid(mfn_to_maddr(mfn));
unsigned int zone = page_to_zone(pg);
+ bool pg_offlined = true;
= false ?
Whoops. Yes.
I will resend the patch.
Cheers,
--
Julien Grall
