Thread (88 messages) flat view 88 messages, 8 authors, 2018-07-11

[PATCH v3 05/20] kvm: arm/arm64: Fix stage2_flush_memslot for 4 level page table

From: eric.auger@redhat.com (Auger Eric)
Date: 2018-06-29 14:51:04
Also in: kvm, kvmarm, lkml, qemu-devel

Hi Suzuki,

On 06/29/2018 01:15 PM, Suzuki K Poulose wrote:
So far we have only supported 3 level page table with fixed IPA of 40bits.
Fix stage2_flush_memslot() to accommodate for 4 level tables.
in 06/30 you add the justification for this change I think. worth to put
in here as well?
quoted hunk ↗ jump to hunk
Cc: Marc Zyngier <redacted>
Acked-by: Christoffer Dall <cdall@kernel.org>
Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com>
---
 virt/kvm/arm/mmu.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/virt/kvm/arm/mmu.c b/virt/kvm/arm/mmu.c
index 1d90d79..061e6b3 100644
--- a/virt/kvm/arm/mmu.c
+++ b/virt/kvm/arm/mmu.c
@@ -379,7 +379,8 @@ static void stage2_flush_memslot(struct kvm *kvm,
 	pgd = kvm->arch.pgd + stage2_pgd_index(addr);
 	do {
 		next = stage2_pgd_addr_end(addr, end);
-		stage2_flush_puds(kvm, pgd, addr, next);
+		if (!stage2_pgd_none(*pgd))
+			stage2_flush_puds(kvm, pgd, addr, next);
 	} while (pgd++, addr = next, addr != end);
 }
 
Besides
Reviewed-by: Eric Auger <eric.auger@redhat.com>

Thanks

Eric
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help