Commit 8e31c2ac authored Feb 11, 2008 by Andi Kleen Committed by Ingo Molnar Feb 19, 2008

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>

parent aa65af3f

arch/x86/mm/pageattr.c

+0 −5

Original line number	Diff line number	Diff line
		@@ -513,7 +513,6 @@ static int __change_page_attr(struct cpa_data *cpa, int primary)
		unsigned long address = cpa->vaddr;
		int do_split, err;
		unsigned int level;
		struct page *kpte_page;
		pte_t *kpte, old_pte;

		repeat:
		@@ -532,10 +531,6 @@ repeat:
		return -EINVAL;
		}

		kpte_page = virt_to_page(kpte);
		BUG_ON(PageLRU(kpte_page));
		BUG_ON(PageCompound(kpte_page));

		if (level == PG_LEVEL_4K) {
		pte_t new_pte;
		pgprot_t new_prot = pte_pgprot(old_pte);

Admin message