diff options
-rw-r--r-- | arch/x86/mm/pageattr.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/arch/x86/mm/pageattr.c b/arch/x86/mm/pageattr.c index d3442dfdfced..968f51a2e39b 100644 --- a/arch/x86/mm/pageattr.c +++ b/arch/x86/mm/pageattr.c | |||
@@ -598,7 +598,6 @@ try_preserve_large_page(pte_t *kpte, unsigned long address, | |||
598 | req_prot = pgprot_clear_protnone_bits(req_prot); | 598 | req_prot = pgprot_clear_protnone_bits(req_prot); |
599 | if (pgprot_val(req_prot) & _PAGE_PRESENT) | 599 | if (pgprot_val(req_prot) & _PAGE_PRESENT) |
600 | pgprot_val(req_prot) |= _PAGE_PSE; | 600 | pgprot_val(req_prot) |= _PAGE_PSE; |
601 | req_prot = canon_pgprot(req_prot); | ||
602 | 601 | ||
603 | /* | 602 | /* |
604 | * old_pfn points to the large page base pfn. So we need | 603 | * old_pfn points to the large page base pfn. So we need |
@@ -718,7 +717,7 @@ __split_large_page(struct cpa_data *cpa, pte_t *kpte, unsigned long address, | |||
718 | */ | 717 | */ |
719 | pfn = ref_pfn; | 718 | pfn = ref_pfn; |
720 | for (i = 0; i < PTRS_PER_PTE; i++, pfn += pfninc) | 719 | for (i = 0; i < PTRS_PER_PTE; i++, pfn += pfninc) |
721 | set_pte(&pbase[i], pfn_pte(pfn, canon_pgprot(ref_prot))); | 720 | set_pte(&pbase[i], pfn_pte(pfn, ref_prot)); |
722 | 721 | ||
723 | if (virt_addr_valid(address)) { | 722 | if (virt_addr_valid(address)) { |
724 | unsigned long pfn = PFN_DOWN(__pa(address)); | 723 | unsigned long pfn = PFN_DOWN(__pa(address)); |
@@ -935,7 +934,6 @@ static void populate_pte(struct cpa_data *cpa, | |||
935 | pte = pte_offset_kernel(pmd, start); | 934 | pte = pte_offset_kernel(pmd, start); |
936 | 935 | ||
937 | pgprot = pgprot_clear_protnone_bits(pgprot); | 936 | pgprot = pgprot_clear_protnone_bits(pgprot); |
938 | pgprot = canon_pgprot(pgprot); | ||
939 | 937 | ||
940 | while (num_pages-- && start < end) { | 938 | while (num_pages-- && start < end) { |
941 | set_pte(pte, pfn_pte(cpa->pfn, pgprot)); | 939 | set_pte(pte, pfn_pte(cpa->pfn, pgprot)); |
@@ -1234,7 +1232,7 @@ repeat: | |||
1234 | * after all we're only going to change it's attributes | 1232 | * after all we're only going to change it's attributes |
1235 | * not the memory it points to | 1233 | * not the memory it points to |
1236 | */ | 1234 | */ |
1237 | new_pte = pfn_pte(pfn, canon_pgprot(new_prot)); | 1235 | new_pte = pfn_pte(pfn, new_prot); |
1238 | cpa->pfn = pfn; | 1236 | cpa->pfn = pfn; |
1239 | /* | 1237 | /* |
1240 | * Do we really change anything ? | 1238 | * Do we really change anything ? |