diff options
author | Ingo Molnar <mingo@elte.hu> | 2008-08-22 02:22:23 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-08-22 08:12:31 -0400 |
commit | 9754a5b840a209bc1f192d59f63e81b698a55ac8 (patch) | |
tree | cac68f94cbd80b5b4e06aac3d1f6cdec3452c452 | |
parent | c4bd1fdab0deec0f69aeabab22075cb22ac8ad44 (diff) |
x86: work around MTRR mask setting, v2
improve the debug printout:
- make it actually display something
- print it only once
would be nice to have a WARN_ONCE() facility, to feed such things to
kerneloops.org.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | arch/x86/kernel/cpu/mtrr/generic.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/arch/x86/kernel/cpu/mtrr/generic.c b/arch/x86/kernel/cpu/mtrr/generic.c index 43102e03e2d1..cb7d3b6a80eb 100644 --- a/arch/x86/kernel/cpu/mtrr/generic.c +++ b/arch/x86/kernel/cpu/mtrr/generic.c | |||
@@ -401,7 +401,12 @@ static void generic_get_mtrr(unsigned int reg, unsigned long *base, | |||
401 | tmp |= ~((1<<(hi - 1)) - 1); | 401 | tmp |= ~((1<<(hi - 1)) - 1); |
402 | 402 | ||
403 | if (tmp != mask_lo) { | 403 | if (tmp != mask_lo) { |
404 | WARN_ON("mtrr: your BIOS has set up an incorrect mask, fixing it up.\n"); | 404 | static int once = 1; |
405 | |||
406 | if (once) { | ||
407 | printk(KERN_INFO "mtrr: your BIOS has set up an incorrect mask, fixing it up.\n"); | ||
408 | once = 0; | ||
409 | } | ||
405 | mask_lo = tmp; | 410 | mask_lo = tmp; |
406 | } | 411 | } |
407 | } | 412 | } |