x86: remove unnecessary save_mr() sanity check
authorPekka Enberg <penberg@cs.helsinki.fi>
Thu, 5 Mar 2009 12:54:57 +0000 (14:54 +0200)
committerIngo Molnar <mingo@elte.hu>
Thu, 5 Mar 2009 13:17:13 +0000 (14:17 +0100)
Impact: cleanup

The save_mr() function already checks that start_pfn is less than
end_pfn so we can remove the unnecessary check which reduces the
diff between the 32-bit and the 64-bit versions of init_memory_mapping().

Signed-off-by: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: Yinghai Lu <yinghai@kernel.org>
LKML-Reference: <1236257708-27269-6-git-send-email-penberg@cs.helsinki.fi>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/mm/init_32.c

index eb98cb90cb39f367687f525d208957c47d4d3ee4..559715b488bbc811f8cd626f631bc137e0234a1e 100644 (file)
@@ -1008,8 +1008,7 @@ unsigned long __init_refok init_memory_mapping(unsigned long start,
        /* tail is not big page (2M) alignment */
        start_pfn = pos>>PAGE_SHIFT;
        end_pfn = end>>PAGE_SHIFT;
-       if (start_pfn < end_pfn)
-               nr_range = save_mr(mr, nr_range, start_pfn, end_pfn, 0);
+       nr_range = save_mr(mr, nr_range, start_pfn, end_pfn, 0);
 
        /* try to merge same page size and continuous */
        for (i = 0; nr_range > 1 && i < nr_range - 1; i++) {