diff options
author | Ingo Molnar <mingo@elte.hu> | 2008-01-30 07:33:56 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-01-30 07:33:56 -0500 |
commit | 5508a7489659f1eed108d3ae7c2d36c8794ee330 (patch) | |
tree | 518702b4f68c75c83bb2976e275cb53c5138488a /arch/x86/mm/pageattr-test.c | |
parent | bb5c2dbd57d93a36b0386dd783dd95e0cbaaa23f (diff) |
x86: cpa self-test fixes
cpa self-test fixes. change_page_attr_addr() was buggy, it
passed in a virtual address as a physical one.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'arch/x86/mm/pageattr-test.c')
-rw-r--r-- | arch/x86/mm/pageattr-test.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/arch/x86/mm/pageattr-test.c b/arch/x86/mm/pageattr-test.c index 91e05a26004d..a12dabbd5c33 100644 --- a/arch/x86/mm/pageattr-test.c +++ b/arch/x86/mm/pageattr-test.c | |||
@@ -106,12 +106,6 @@ static __init int print_split(struct split_state *s) | |||
106 | return err; | 106 | return err; |
107 | } | 107 | } |
108 | 108 | ||
109 | static __init int state_same(struct split_state *a, struct split_state *b) | ||
110 | { | ||
111 | return a->lpg == b->lpg && a->gpg == b->gpg && a->spg == b->spg && | ||
112 | a->exec == b->exec; | ||
113 | } | ||
114 | |||
115 | static unsigned long __initdata addr[NTEST]; | 109 | static unsigned long __initdata addr[NTEST]; |
116 | static unsigned int __initdata len[NTEST]; | 110 | static unsigned int __initdata len[NTEST]; |
117 | 111 | ||
@@ -229,8 +223,6 @@ static __init int exercise_pageattr(void) | |||
229 | global_flush_tlb(); | 223 | global_flush_tlb(); |
230 | 224 | ||
231 | failed += print_split(&sc); | 225 | failed += print_split(&sc); |
232 | if (!state_same(&sa, &sc)) | ||
233 | failed++; | ||
234 | 226 | ||
235 | if (failed) | 227 | if (failed) |
236 | printk(KERN_ERR "CPA selftests NOT PASSED. Please report.\n"); | 228 | printk(KERN_ERR "CPA selftests NOT PASSED. Please report.\n"); |