On Fri, Aug 24, 2012 at 06:13:02PM -0700, H. Peter Anvin wrote: > On 08/24/2012 05:49 PM, Jacob Shin wrote: > > > > Right, I think what I was attempting to do was to merge the 1MB > > with E820_RAM right above 1MB: > > > > So instead of: > > > > init_memory_mapping(0, 1MB) > > init_memory_mapping(1MB, 2GB) > > > > It would be: > > > > init_memory_mapping(0, 2GB) > > > > While taking care of the odd case where there is a gap right after > > 1MB. > > > > But if its not worth it, I can move it out of the loop. > > > > What is the benefit?
So that in the case where we have E820_RAM right above 1MB, we don't call init_memory_mapping twice, first on 0 ~ 1MB and then 1MB ~ something we only call it once. 0 ~ something. I'll get it out of the loop if you don't think its a good idea. > > -hpa > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
