RE: [Xen-devel] [patch] PFN_PHYS PAE overflow (fixes bug 242)

["Ian Pratt" <m+Ian.Pratt@xxxxxxxxxxxx>]

> machine_specific_memory_setup() makes the following call to 
> setup e820:
> 
>     add_memory_region(0, PFN_PHYS(max_pfn), E820_RAM);
> 
> Since max_pfn is (rightly) an unsigned long, we overflow for 
> the current definition of PFN_PHYS. Attached is my preferred 
> solution as it will hopefully avoid similar situations that 
> may come up in the future.

Good catch!

Ian

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel