Commit 7afe15b9 authored by Ingo Molnar's avatar Ingo Molnar

x86: simplify cpa largepage split, #3

simplify cpa largepage split: push the reference protection bits
into the largepage-splitting function.
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent 5508a748
...@@ -61,13 +61,13 @@ static void set_pmd_pte(pte_t *kpte, unsigned long address, pte_t pte) ...@@ -61,13 +61,13 @@ static void set_pmd_pte(pte_t *kpte, unsigned long address, pte_t pte)
spin_unlock_irqrestore(&pgd_lock, flags); spin_unlock_irqrestore(&pgd_lock, flags);
} }
static int static int split_large_page(pte_t *kpte, unsigned long address)
split_large_page(pte_t *kpte, unsigned long address, pgprot_t ref_prot)
{ {
int i, level; pgprot_t ref_prot = pte_pgprot(pte_clrhuge(*kpte));
unsigned long addr; unsigned long addr;
pte_t *pbase, *tmp; pte_t *pbase, *tmp;
struct page *base; struct page *base;
int i, level;
base = alloc_pages(GFP_KERNEL, 0); base = alloc_pages(GFP_KERNEL, 0);
if (!base) if (!base)
...@@ -109,11 +109,9 @@ split_large_page(pte_t *kpte, unsigned long address, pgprot_t ref_prot) ...@@ -109,11 +109,9 @@ split_large_page(pte_t *kpte, unsigned long address, pgprot_t ref_prot)
static int __change_page_attr(struct page *page, pgprot_t prot) static int __change_page_attr(struct page *page, pgprot_t prot)
{ {
pgprot_t ref_prot = PAGE_KERNEL;
struct page *kpte_page; struct page *kpte_page;
unsigned long address; unsigned long address;
int level, err = 0; int level, err = 0;
pgprot_t oldprot;
pte_t *kpte; pte_t *kpte;
BUG_ON(PageHighMem(page)); BUG_ON(PageHighMem(page));
...@@ -124,7 +122,6 @@ static int __change_page_attr(struct page *page, pgprot_t prot) ...@@ -124,7 +122,6 @@ static int __change_page_attr(struct page *page, pgprot_t prot)
if (!kpte) if (!kpte)
return -EINVAL; return -EINVAL;
oldprot = pte_pgprot(*kpte);
kpte_page = virt_to_page(kpte); kpte_page = virt_to_page(kpte);
BUG_ON(PageLRU(kpte_page)); BUG_ON(PageLRU(kpte_page));
BUG_ON(PageCompound(kpte_page)); BUG_ON(PageCompound(kpte_page));
...@@ -137,16 +134,10 @@ static int __change_page_attr(struct page *page, pgprot_t prot) ...@@ -137,16 +134,10 @@ static int __change_page_attr(struct page *page, pgprot_t prot)
address < (unsigned long)&_etext && address < (unsigned long)&_etext &&
(pgprot_val(prot) & _PAGE_NX)); (pgprot_val(prot) & _PAGE_NX));
if ((address & LARGE_PAGE_MASK) < (unsigned long)&_etext)
ref_prot = PAGE_KERNEL_EXEC;
ref_prot = canon_pgprot(ref_prot);
prot = canon_pgprot(prot);
if (level == 3) { if (level == 3) {
set_pte_atomic(kpte, mk_pte(page, prot)); set_pte_atomic(kpte, mk_pte(page, canon_pgprot(prot)));
} else { } else {
err = split_large_page(kpte, address, ref_prot); err = split_large_page(kpte, address);
if (!err) if (!err)
goto repeat; goto repeat;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment