[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH] xen/mm: pg_offlined can be defined as bool in free_heap_pages()
Hi, On 23/02/2022 19:03, Andrew Cooper wrote: On 23/02/2022 18:41, Julien Grall wrote:From: Julien Grall <jgrall@xxxxxxxxxx> 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 <jgrall@xxxxxxxxxx> --- 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
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |