diff options
author | Arjan van de Ven <arjan@linux.intel.com> | 2008-07-30 15:26:26 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-08-13 13:05:39 -0400 |
commit | 875e40b97571e1f06d1184ad6cbb2acf9cb31a23 (patch) | |
tree | 8908cd7e464b16a363e3f49512f10ba626cfd517 /arch/x86/mm/pageattr.c | |
parent | a726c6009e6eba4acfccf8b683854866eeabb184 (diff) |
x86: use WARN() in arch/x86/mm/pageattr.c
Use WARN() instead of a printk+WARN_ON() pair; this way the message becomes
part of the warning section for better reporting/collection.
Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Acked-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Cc: akpm@linux-foundation.org
Cc: arjan@linux.intel.com
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/x86/mm/pageattr.c')
-rw-r--r-- | arch/x86/mm/pageattr.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/x86/mm/pageattr.c b/arch/x86/mm/pageattr.c index 65c6e46bf059..ba24537d69dd 100644 --- a/arch/x86/mm/pageattr.c +++ b/arch/x86/mm/pageattr.c | |||
@@ -592,10 +592,9 @@ repeat: | |||
592 | if (!pte_val(old_pte)) { | 592 | if (!pte_val(old_pte)) { |
593 | if (!primary) | 593 | if (!primary) |
594 | return 0; | 594 | return 0; |
595 | printk(KERN_WARNING "CPA: called for zero pte. " | 595 | WARN(1, KERN_WARNING "CPA: called for zero pte. " |
596 | "vaddr = %lx cpa->vaddr = %lx\n", address, | 596 | "vaddr = %lx cpa->vaddr = %lx\n", address, |
597 | cpa->vaddr); | 597 | cpa->vaddr); |
598 | WARN_ON(1); | ||
599 | return -EINVAL; | 598 | return -EINVAL; |
600 | } | 599 | } |
601 | 600 | ||