x86, 32-bit: refactor find_low_pfn_range()
authorIngo Molnar <mingo@elte.hu>
Thu, 12 Feb 2009 14:16:03 +0000 (15:16 +0100)
committerIngo Molnar <mingo@elte.hu>
Thu, 12 Feb 2009 14:21:17 +0000 (15:21 +0100)
Impact: cleanup

Make the max_low_pfn logic a bit more standard between
lowmem_pfn_init() and highmem_pfn_init().

Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/mm/init_32.c

index 9d36eb9..1a96124 100644 (file)
@@ -687,6 +687,9 @@ early_param("highmem", parse_highmem);
  */
 void __init lowmem_pfn_init(void)
 {
+       /* max_low_pfn is 0, we already have early_res support */
+       max_low_pfn = max_pfn;
+
        if (highmem_pages == -1)
                highmem_pages = 0;
 #ifdef CONFIG_HIGHMEM
@@ -720,6 +723,8 @@ void __init lowmem_pfn_init(void)
  */
 void __init highmem_pfn_init(void)
 {
+       max_low_pfn = MAXMEM_PFN;
+
        if (highmem_pages == -1)
                highmem_pages = max_pfn - MAXMEM_PFN;
 
@@ -732,7 +737,6 @@ void __init highmem_pfn_init(void)
                        pages_to_mb(highmem_pages));
                highmem_pages = 0;
        }
-       max_low_pfn = MAXMEM_PFN;
 #ifndef CONFIG_HIGHMEM
        /* Maximum memory usable is what is directly addressable */
        printk(KERN_WARNING "Warning only %ldMB will be used.\n", MAXMEM>>20);
@@ -758,13 +762,10 @@ void __init find_low_pfn_range(void)
 {
        /* it could update max_pfn */
 
-       /* max_low_pfn is 0, we already have early_res support */
-       max_low_pfn = max_pfn;
-
-       if (max_low_pfn > MAXMEM_PFN)
-               highmem_pfn_init();
-       else
+       if (max_pfn <= MAXMEM_PFN)
                lowmem_pfn_init();
+       else
+               highmem_pfn_init();
 }
 
 #ifndef CONFIG_NEED_MULTIPLE_NODES