diff options
author | Andi Kleen <ak@suse.de> | 2008-02-11 04:50:21 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-02-19 10:18:29 -0500 |
commit | 8e31c2ac1101b09ad6c6d3539b032d312cc3302b (patch) | |
tree | 92acaaa55da1170bf738e0afdb53ce36c984d84e /arch/x86/mm/pageattr.c | |
parent | aa65af3f92da39a686c8a3479426d15854fa27ab (diff) |
x86: CPA: remove BUG_ON for LRU/Compound pages
New implementation does not use lru for anything so there is no need
to reject pages that are in the LRU. Similar for compound pages (which
were checked because they also use page->lru)
[ tglx@linutronix.de: removed unused variable ]
Signed-off-by: Andi Kleen <ak@suse.de>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'arch/x86/mm/pageattr.c')
-rw-r--r-- | arch/x86/mm/pageattr.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/arch/x86/mm/pageattr.c b/arch/x86/mm/pageattr.c index 3ee14996c829..e2a74ea11a53 100644 --- a/arch/x86/mm/pageattr.c +++ b/arch/x86/mm/pageattr.c | |||
@@ -513,7 +513,6 @@ static int __change_page_attr(struct cpa_data *cpa, int primary) | |||
513 | unsigned long address = cpa->vaddr; | 513 | unsigned long address = cpa->vaddr; |
514 | int do_split, err; | 514 | int do_split, err; |
515 | unsigned int level; | 515 | unsigned int level; |
516 | struct page *kpte_page; | ||
517 | pte_t *kpte, old_pte; | 516 | pte_t *kpte, old_pte; |
518 | 517 | ||
519 | repeat: | 518 | repeat: |
@@ -532,10 +531,6 @@ repeat: | |||
532 | return -EINVAL; | 531 | return -EINVAL; |
533 | } | 532 | } |
534 | 533 | ||
535 | kpte_page = virt_to_page(kpte); | ||
536 | BUG_ON(PageLRU(kpte_page)); | ||
537 | BUG_ON(PageCompound(kpte_page)); | ||
538 | |||
539 | if (level == PG_LEVEL_4K) { | 534 | if (level == PG_LEVEL_4K) { |
540 | pte_t new_pte; | 535 | pte_t new_pte; |
541 | pgprot_t new_prot = pte_pgprot(old_pte); | 536 | pgprot_t new_prot = pte_pgprot(old_pte); |