[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Minios-devel] [PATCH v3 01/43] mini-os: fix the wrong parameter for map_free() in init_page_allocator()



The @r_min has become to virtual address, so it is wrong to use PHYS_PFN()
for @r_min.

This patch uses the virt_to_pfn() to fix it.

The bug is introduced by:
 "2526e9bbe52328a7 mini-os: get physical memory map"

Reviewed-by: Juergen Gross <jgross@xxxxxxxx>
Acked-by: Samuel Thibault <samuel.thibault@xxxxxxxxxxxx>
Signed-off-by: Huang Shijie <shijie.huang@xxxxxxx>
---
 mm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mm.c b/mm.c
index 932ceeb..4880b5e 100644
--- a/mm.c
+++ b/mm.c
@@ -199,7 +199,7 @@ static void init_page_allocator(unsigned long min, unsigned 
long max)
         range = r_max - r_min;
 
         /* Free up the memory we've been given to play with. */
-        map_free(PHYS_PFN(r_min), range >> PAGE_SHIFT);
+        map_free(virt_to_pfn(r_min), range >> PAGE_SHIFT);
 
         while ( range != 0 )
         {
-- 
2.7.4


_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.