Commit 20bf062c authored by Alexander Holler's avatar Alexander Holler Committed by Ingo Molnar

x86/memtest: Shorten time for tests

By just reversing the order memtest is using the test patterns,
an additional round to zero the memory is not necessary.

This might save up to a second or even more for setups which are
doing tests on every boot.
Signed-off-by: Alexander Holler's avatarAlexander Holler <[email protected]>
Cc: Yinghai Lu <[email protected]>
Cc: Linus Torvalds <[email protected]>
Cc: Andrew Morton <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Thomas Gleixner <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]Signed-off-by: default avatarIngo Molnar <[email protected]>
parent 9611dc7a
...@@ -9,6 +9,7 @@ ...@@ -9,6 +9,7 @@
#include <linux/memblock.h> #include <linux/memblock.h>
static u64 patterns[] __initdata = { static u64 patterns[] __initdata = {
/* The first entry has to be 0 to leave memtest with zeroed memory */
0, 0,
0xffffffffffffffffULL, 0xffffffffffffffffULL,
0x5555555555555555ULL, 0x5555555555555555ULL,
...@@ -110,15 +111,8 @@ void __init early_memtest(unsigned long start, unsigned long end) ...@@ -110,15 +111,8 @@ void __init early_memtest(unsigned long start, unsigned long end)
return; return;
printk(KERN_INFO "early_memtest: # of tests: %d\n", memtest_pattern); printk(KERN_INFO "early_memtest: # of tests: %d\n", memtest_pattern);
for (i = 0; i < memtest_pattern; i++) { for (i = memtest_pattern-1; i < UINT_MAX; --i) {
idx = i % ARRAY_SIZE(patterns); idx = i % ARRAY_SIZE(patterns);
do_one_pass(patterns[idx], start, end); do_one_pass(patterns[idx], start, end);
} }
if (idx > 0) {
printk(KERN_INFO "early_memtest: wipe out "
"test pattern from memory\n");
/* additional test with pattern 0 will do this */
do_one_pass(0, start, end);
}
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment