Hi Will, On 9/11/20 2:25 PM, Will Deacon wrote: > It's me again, with version five of the KVM page-table rework previously > seen at: > > v1: https://lore.kernel.org/r/20200730153406.25136-1-will@xxxxxxxxxx > v2: https://lore.kernel.org/r/20200818132818.16065-1-will@xxxxxxxxxx > v3: https://lore.kernel.org/r/20200825093953.26493-1-will@xxxxxxxxxx > v4: https://lore.kernel.org/r/20200907152344.12978-1-will@xxxxxxxxxx > > Changes since v4 include: > > * Add comments to the kerneldoc describing alignment behaviour for > addresses and size parameters > * Fix formatting of IPA size messages > * Fix handling of unaligned addresses in kvm_phys_addr_ioremap() > * Add DSB after zeroing stage-2 PGD pages > * Add reviewer tags > > Once again, thanks to Alex, Gavin and Andrew for their comments. I didn't realize that there won't be another iteration of the series. I reviewed all the patches with the exception of patch #5 ("KVM: arm64: Use generic allocator for hyp stage-1 page-tables") because I don't know much about what needs to mapped at EL2 stage 1, everything else looks alright to me. If the commit message can be modified, you can add my Reviewed-by tag to the series (sans #5); it's fine if that's not possible. Thanks, Alex > > Will > > Cc: Marc Zyngier <maz@xxxxxxxxxx> > Cc: Quentin Perret <qperret@xxxxxxxxxx> > Cc: James Morse <james.morse@xxxxxxx> > Cc: Suzuki Poulose <suzuki.poulose@xxxxxxx> > Cc: Catalin Marinas <catalin.marinas@xxxxxxx> > Cc: Gavin Shan <gshan@xxxxxxxxxx> > Cc: Alexandru Elisei <alexandru.elisei@xxxxxxx> > Cc: Andrew Scull <ascull@xxxxxxxxxx> > Cc: kernel-team@xxxxxxxxxxx > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > > --->8 > > Quentin Perret (4): > KVM: arm64: Add support for stage-2 write-protect in generic > page-table > KVM: arm64: Convert write-protect operation to generic page-table API > KVM: arm64: Add support for stage-2 cache flushing in generic > page-table > KVM: arm64: Convert memslot cache-flushing code to generic page-table > API > > Will Deacon (17): > KVM: arm64: Remove kvm_mmu_free_memory_caches() > KVM: arm64: Add stand-alone page-table walker infrastructure > KVM: arm64: Add support for creating kernel-agnostic stage-1 page > tables > KVM: arm64: Use generic allocator for hyp stage-1 page-tables > KVM: arm64: Add support for creating kernel-agnostic stage-2 page > tables > KVM: arm64: Add support for stage-2 map()/unmap() in generic > page-table > KVM: arm64: Convert kvm_phys_addr_ioremap() to generic page-table API > KVM: arm64: Convert kvm_set_spte_hva() to generic page-table API > KVM: arm64: Convert unmap_stage2_range() to generic page-table API > KVM: arm64: Add support for stage-2 page-aging in generic page-table > KVM: arm64: Convert page-aging and access faults to generic page-table > API > KVM: arm64: Add support for relaxing stage-2 perms in generic > page-table code > KVM: arm64: Convert user_mem_abort() to generic page-table API > KVM: arm64: Check the pgt instead of the pgd when modifying page-table > KVM: arm64: Remove unused page-table code > KVM: arm64: Remove unused 'pgd' field from 'struct kvm_s2_mmu' > KVM: arm64: Don't constrain maximum IPA size based on host > configuration > > arch/arm64/include/asm/kvm_host.h | 2 +- > arch/arm64/include/asm/kvm_mmu.h | 251 +--- > arch/arm64/include/asm/kvm_pgtable.h | 309 ++++ > arch/arm64/include/asm/pgtable-hwdef.h | 24 - > arch/arm64/include/asm/pgtable-prot.h | 19 - > arch/arm64/include/asm/stage2_pgtable.h | 215 --- > arch/arm64/kvm/arm.c | 2 +- > arch/arm64/kvm/hyp/Makefile | 2 +- > arch/arm64/kvm/hyp/pgtable.c | 883 ++++++++++++ > arch/arm64/kvm/mmu.c | 1748 ++++------------------- > arch/arm64/kvm/reset.c | 40 +- > 11 files changed, 1479 insertions(+), 2016 deletions(-) > create mode 100644 arch/arm64/include/asm/kvm_pgtable.h > create mode 100644 arch/arm64/kvm/hyp/pgtable.c > _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/mailman/listinfo/kvmarm