Commit 6eade8ff authored by Thomas Gleixner's avatar Thomas Gleixner Committed by Ingo Molnar

x86: cpa: clean up change_page_attr_set/clear()

Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent 4692a145
...@@ -312,10 +312,10 @@ static int change_page_attr_set(unsigned long addr, int numpages, ...@@ -312,10 +312,10 @@ static int change_page_attr_set(unsigned long addr, int numpages,
for (i = 0; i < numpages ; i++) { for (i = 0; i < numpages ; i++) {
pte = lookup_address(addr, &level); pte = lookup_address(addr, &level);
if (pte) if (!pte)
current_prot = pte_pgprot(*pte); return -EINVAL;
else
pgprot_val(current_prot) = 0; current_prot = pte_pgprot(*pte);
pgprot_val(new_prot) = pgprot_val(new_prot) =
pgprot_val(current_prot) | pgprot_val(prot); pgprot_val(current_prot) | pgprot_val(prot);
...@@ -356,11 +356,12 @@ static int change_page_attr_clear(unsigned long addr, int numpages, ...@@ -356,11 +356,12 @@ static int change_page_attr_clear(unsigned long addr, int numpages,
int i, ret; int i, ret;
for (i = 0; i < numpages; i++) { for (i = 0; i < numpages; i++) {
pte = lookup_address(addr, &level); pte = lookup_address(addr, &level);
if (pte) if (!pte)
current_prot = pte_pgprot(*pte); return -EINVAL;
else
pgprot_val(current_prot) = 0; current_prot = pte_pgprot(*pte);
pgprot_val(new_prot) = pgprot_val(new_prot) =
pgprot_val(current_prot) & ~pgprot_val(prot); pgprot_val(current_prot) & ~pgprot_val(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