Commit f0646e43 authored by Ingo Molnar's avatar Ingo Molnar

x86: return the page table level in lookup_address()

based on this patch from Andi Kleen:

|  Subject: CPA: Return the page table level in lookup_address()
|  From: Andi Kleen <ak@suse.de>
|
|  Needed for the next change.
|
|  And change all the callers.

and ported it to x86.git.
Signed-off-by: default avatarAndi Kleen <ak@suse.de>
Acked-by: default avatarJan Beulich <jbeulich@novell.com>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent a5a5dc31
...@@ -613,7 +613,8 @@ no_context: ...@@ -613,7 +613,8 @@ no_context:
#ifdef CONFIG_X86_PAE #ifdef CONFIG_X86_PAE
if (error_code & PF_INSTR) { if (error_code & PF_INSTR) {
pte_t *pte = lookup_address(address); int level;
pte_t *pte = lookup_address(address, &level);
if (pte && pte_present(*pte) && !pte_exec(*pte)) if (pte && pte_present(*pte) && !pte_exec(*pte))
printk(KERN_CRIT "kernel tried to execute " printk(KERN_CRIT "kernel tried to execute "
......
...@@ -535,11 +535,12 @@ int __init set_kernel_exec(unsigned long vaddr, int enable) ...@@ -535,11 +535,12 @@ int __init set_kernel_exec(unsigned long vaddr, int enable)
{ {
pte_t *pte; pte_t *pte;
int ret = 1; int ret = 1;
int level;
if (!nx_enabled) if (!nx_enabled)
goto out; goto out;
pte = lookup_address(vaddr); pte = lookup_address(vaddr, &level);
BUG_ON(!pte); BUG_ON(!pte);
if (!pte_exec(*pte)) if (!pte_exec(*pte))
......
...@@ -18,7 +18,7 @@ ...@@ -18,7 +18,7 @@
static DEFINE_SPINLOCK(cpa_lock); static DEFINE_SPINLOCK(cpa_lock);
static struct list_head df_list = LIST_HEAD_INIT(df_list); static struct list_head df_list = LIST_HEAD_INIT(df_list);
pte_t *lookup_address(unsigned long address) pte_t *lookup_address(unsigned long address, int *level)
{ {
pgd_t *pgd = pgd_offset_k(address); pgd_t *pgd = pgd_offset_k(address);
pud_t *pud; pud_t *pud;
...@@ -32,8 +32,10 @@ pte_t *lookup_address(unsigned long address) ...@@ -32,8 +32,10 @@ pte_t *lookup_address(unsigned long address)
pmd = pmd_offset(pud, address); pmd = pmd_offset(pud, address);
if (pmd_none(*pmd)) if (pmd_none(*pmd))
return NULL; return NULL;
*level = 2;
if (pmd_large(*pmd)) if (pmd_large(*pmd))
return (pte_t *)pmd; return (pte_t *)pmd;
*level = 3;
return pte_offset_kernel(pmd, address); return pte_offset_kernel(pmd, address);
} }
...@@ -156,11 +158,12 @@ static int __change_page_attr(struct page *page, pgprot_t prot) ...@@ -156,11 +158,12 @@ static int __change_page_attr(struct page *page, pgprot_t prot)
struct page *kpte_page; struct page *kpte_page;
unsigned long address; unsigned long address;
pte_t *kpte; pte_t *kpte;
int level;
BUG_ON(PageHighMem(page)); BUG_ON(PageHighMem(page));
address = (unsigned long)page_address(page); address = (unsigned long)page_address(page);
kpte = lookup_address(address); kpte = lookup_address(address, &level);
if (!kpte) if (!kpte)
return -EINVAL; return -EINVAL;
......
...@@ -14,7 +14,7 @@ ...@@ -14,7 +14,7 @@
#include <asm/uaccess.h> #include <asm/uaccess.h>
#include <asm/io.h> #include <asm/io.h>
pte_t *lookup_address(unsigned long address) pte_t *lookup_address(unsigned long address, int *level)
{ {
pgd_t *pgd = pgd_offset_k(address); pgd_t *pgd = pgd_offset_k(address);
pud_t *pud; pud_t *pud;
...@@ -29,8 +29,10 @@ pte_t *lookup_address(unsigned long address) ...@@ -29,8 +29,10 @@ pte_t *lookup_address(unsigned long address)
pmd = pmd_offset(pud, address); pmd = pmd_offset(pud, address);
if (!pmd_present(*pmd)) if (!pmd_present(*pmd))
return NULL; return NULL;
*level = 3;
if (pmd_large(*pmd)) if (pmd_large(*pmd))
return (pte_t *)pmd; return (pte_t *)pmd;
*level = 4;
pte = pte_offset_kernel(pmd, address); pte = pte_offset_kernel(pmd, address);
if (pte && !pte_present(*pte)) if (pte && !pte_present(*pte))
...@@ -140,8 +142,9 @@ __change_page_attr(unsigned long address, unsigned long pfn, pgprot_t prot, ...@@ -140,8 +142,9 @@ __change_page_attr(unsigned long address, unsigned long pfn, pgprot_t prot,
struct page *kpte_page; struct page *kpte_page;
pgprot_t ref_prot2; pgprot_t ref_prot2;
pte_t *kpte; pte_t *kpte;
int level;
kpte = lookup_address(address); kpte = lookup_address(address, &level);
if (!kpte) if (!kpte)
return 0; return 0;
......
...@@ -58,7 +58,8 @@ ...@@ -58,7 +58,8 @@
xmaddr_t arbitrary_virt_to_machine(unsigned long address) xmaddr_t arbitrary_virt_to_machine(unsigned long address)
{ {
pte_t *pte = lookup_address(address); int level;
pte_t *pte = lookup_address(address, &level);
unsigned offset = address & PAGE_MASK; unsigned offset = address & PAGE_MASK;
BUG_ON(pte == NULL); BUG_ON(pte == NULL);
...@@ -70,8 +71,9 @@ void make_lowmem_page_readonly(void *vaddr) ...@@ -70,8 +71,9 @@ void make_lowmem_page_readonly(void *vaddr)
{ {
pte_t *pte, ptev; pte_t *pte, ptev;
unsigned long address = (unsigned long)vaddr; unsigned long address = (unsigned long)vaddr;
int level;
pte = lookup_address(address); pte = lookup_address(address, &level);
BUG_ON(pte == NULL); BUG_ON(pte == NULL);
ptev = pte_wrprotect(*pte); ptev = pte_wrprotect(*pte);
...@@ -84,8 +86,9 @@ void make_lowmem_page_readwrite(void *vaddr) ...@@ -84,8 +86,9 @@ void make_lowmem_page_readwrite(void *vaddr)
{ {
pte_t *pte, ptev; pte_t *pte, ptev;
unsigned long address = (unsigned long)vaddr; unsigned long address = (unsigned long)vaddr;
int level;
pte = lookup_address(address); pte = lookup_address(address, &level);
BUG_ON(pte == NULL); BUG_ON(pte == NULL);
ptev = pte_mkwrite(*pte); ptev = pte_mkwrite(*pte);
......
...@@ -182,7 +182,7 @@ static inline void clone_pgd_range(pgd_t *dst, pgd_t *src, int count) ...@@ -182,7 +182,7 @@ static inline void clone_pgd_range(pgd_t *dst, pgd_t *src, int count)
* NOTE: the return type is pte_t but if the pmd is PSE then we return it * NOTE: the return type is pte_t but if the pmd is PSE then we return it
* as a pte too. * as a pte too.
*/ */
extern pte_t *lookup_address(unsigned long address); extern pte_t *lookup_address(unsigned long address, int *level);
/* /*
* Make a given kernel text page executable/non-executable. * Make a given kernel text page executable/non-executable.
......
...@@ -240,7 +240,7 @@ extern struct list_head pgd_list; ...@@ -240,7 +240,7 @@ extern struct list_head pgd_list;
extern int kern_addr_valid(unsigned long addr); extern int kern_addr_valid(unsigned long addr);
pte_t *lookup_address(unsigned long addr); pte_t *lookup_address(unsigned long addr, int *level);
#define io_remap_pfn_range(vma, vaddr, pfn, size, prot) \ #define io_remap_pfn_range(vma, vaddr, pfn, size, prot) \
remap_pfn_range(vma, vaddr, pfn, size, prot) remap_pfn_range(vma, vaddr, pfn, size, prot)
......
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