Hi Steve

On 08/13/2018 05:04 PM, Steven Price wrote:
The lock has never been used and the page tables are protected by
mmu_lock in struct kvm.

Signed-off-by: Steven Price <steven.pr...@arm.com>
CC: Marc Zyngier <marc.zyng...@arm.com>
CC: Christoffer Dall <christoffer.d...@arm.com>

  arch/arm64/include/asm/kvm_host.h | 3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm64/include/asm/kvm_host.h 
b/arch/arm64/include/asm/kvm_host.h
index fe8777b12f86..8150a7f8a50f 100644
--- a/arch/arm64/include/asm/kvm_host.h
+++ b/arch/arm64/include/asm/kvm_host.h
@@ -61,8 +61,7 @@ struct kvm_arch {
        u64    vmid_gen;
        u32    vmid;
- /* 1-level 2nd stage table and lock */
-       spinlock_t pgd_lock;
+       /* 1-level 2nd stage table, protected by kvm->mmu_lock */
        pgd_t *pgd;
/* VTTBR value associated with above pgd and vmid */


I have a patch which already does this, though not a dedicated patch [1] in the dynamic IPA series. As such I am fine with this patch being
merged if it gets there first, as we are still working out the userspace
API for dynamic IPA.

FWIW,

Reviewed-by: Suzuki K Poulose <suzuki.poul...@arm.com>



[1] http://lists.infradead.org/pipermail/linux-arm-kernel/2018-July/591778.html
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

Reply via email to