From: Thomas Bogendoerfer <tsbog...@alpha.franken.de>

[ Upstream commit 411406a8c758d9ad6f908fab3a6cf1d3d89e1d08 ]

MIPS protection bits are setup during runtime so using defines like
PAGE_SHARED ignores this runtime changes. Using vm_get_page_prot
to get correct page protection fixes this.

Signed-off-by: Thomas Bogendoerfer <tsbog...@alpha.franken.de>
Signed-off-by: Sasha Levin <sas...@kernel.org>
---
 arch/mips/kvm/mmu.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/mips/kvm/mmu.c b/arch/mips/kvm/mmu.c
index 97f63a84aa51f..b5974ff2d9162 100644
--- a/arch/mips/kvm/mmu.c
+++ b/arch/mips/kvm/mmu.c
@@ -1100,6 +1100,7 @@ int kvm_mips_handle_commpage_tlb_fault(unsigned long 
badvaddr,
 {
        kvm_pfn_t pfn;
        pte_t *ptep;
+       pgprot_t prot;
 
        ptep = kvm_trap_emul_pte_for_gva(vcpu, badvaddr);
        if (!ptep) {
@@ -1109,7 +1110,8 @@ int kvm_mips_handle_commpage_tlb_fault(unsigned long 
badvaddr,
 
        pfn = PFN_DOWN(virt_to_phys(vcpu->arch.kseg0_commpage));
        /* Also set valid and dirty, so refill handler doesn't have to */
-       *ptep = pte_mkyoung(pte_mkdirty(pfn_pte(pfn, PAGE_SHARED)));
+       prot = vm_get_page_prot(VM_READ|VM_WRITE|VM_SHARED);
+       *ptep = pte_mkyoung(pte_mkdirty(pfn_pte(pfn, prot)));
 
        /* Invalidate this entry in the TLB, guest kernel ASID only */
        kvm_mips_host_tlb_inv(vcpu, badvaddr, false, true);
-- 
2.27.0

Reply via email to