From: Gavin Shan <gshan@redhat.com> To: Will Deacon <will@kernel.org>, kvmarm@lists.cs.columbia.edu Cc: Marc Zyngier <maz@kernel.org>, Catalin Marinas <catalin.marinas@arm.com>, kernel-team@android.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 14/21] KVM: arm64: Add support for stage-2 cache flushing in generic page-table Date: Thu, 3 Sep 2020 14:51:30 +1000 [thread overview] Message-ID: <4198f2f9-abaa-2755-0113-38384150b595@redhat.com> (raw) In-Reply-To: <20200825093953.26493-15-will@kernel.org> Hi Will, On 8/25/20 7:39 PM, Will Deacon wrote: > From: Quentin Perret <qperret@google.com> > > Add support for cache flushing a range of the stage-2 address space to > the generic page-table code. > > Cc: Marc Zyngier <maz@kernel.org> > Signed-off-by: Quentin Perret <qperret@google.com> > Signed-off-by: Will Deacon <will@kernel.org> > --- Reviewed-by: Gavin Shan <gshan@redhat.com> > arch/arm64/include/asm/kvm_pgtable.h | 12 ++++++++++++ > arch/arm64/kvm/hyp/pgtable.c | 26 ++++++++++++++++++++++++++ > 2 files changed, 38 insertions(+) > > diff --git a/arch/arm64/include/asm/kvm_pgtable.h b/arch/arm64/include/asm/kvm_pgtable.h > index 0c96b78d791d..ea823fe31913 100644 > --- a/arch/arm64/include/asm/kvm_pgtable.h > +++ b/arch/arm64/include/asm/kvm_pgtable.h > @@ -226,6 +226,18 @@ kvm_pte_t kvm_pgtable_stage2_mkold(struct kvm_pgtable *pgt, u64 addr); > */ > bool kvm_pgtable_stage2_is_young(struct kvm_pgtable *pgt, u64 addr); > > +/** > + * kvm_pgtable_stage2_flush_range() - Clean and invalidate data cache to Point > + * of Coherency for guest stage-2 address > + * range. > + * @pgt: Page-table structure initialised by kvm_pgtable_stage2_init(). > + * @addr: Intermediate physical address from which to flush. > + * @size: Size of the range. > + * > + * Return: 0 on success, negative error code on failure. > + */ > +int kvm_pgtable_stage2_flush(struct kvm_pgtable *pgt, u64 addr, u64 size); > + > /** > * kvm_pgtable_walk() - Walk a page-table. > * @pgt: Page-table structure initialised by kvm_pgtable_*_init(). > diff --git a/arch/arm64/kvm/hyp/pgtable.c b/arch/arm64/kvm/hyp/pgtable.c > index c218651f8eba..75887185f1e2 100644 > --- a/arch/arm64/kvm/hyp/pgtable.c > +++ b/arch/arm64/kvm/hyp/pgtable.c > @@ -762,6 +762,32 @@ bool kvm_pgtable_stage2_is_young(struct kvm_pgtable *pgt, u64 addr) > return pte & KVM_PTE_LEAF_ATTR_LO_S2_AF; > } > > +static int stage2_flush_walker(u64 addr, u64 end, u32 level, kvm_pte_t *ptep, > + enum kvm_pgtable_walk_flags flag, > + void * const arg) > +{ > + kvm_pte_t pte = *ptep; > + > + if (!kvm_pte_valid(pte) || !stage2_pte_cacheable(pte)) > + return 0; > + > + stage2_flush_dcache(kvm_pte_follow(pte), kvm_granule_size(level)); > + return 0; > +} > + > +int kvm_pgtable_stage2_flush(struct kvm_pgtable *pgt, u64 addr, u64 size) > +{ > + struct kvm_pgtable_walker walker = { > + .cb = stage2_flush_walker, > + .flags = KVM_PGTABLE_WALK_LEAF, > + }; > + > + if (cpus_have_const_cap(ARM64_HAS_STAGE2_FWB)) > + return 0; > + > + return kvm_pgtable_walk(pgt, addr, size, &walker); > +} > + > int kvm_pgtable_stage2_init(struct kvm_pgtable *pgt, struct kvm *kvm) > { > size_t pgd_sz; > Thanks, Gavin _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
WARNING: multiple messages have this Message-ID (diff)
From: Gavin Shan <gshan@redhat.com> To: Will Deacon <will@kernel.org>, kvmarm@lists.cs.columbia.edu Cc: Suzuki Poulose <suzuki.poulose@arm.com>, Marc Zyngier <maz@kernel.org>, Quentin Perret <qperret@google.com>, James Morse <james.morse@arm.com>, Catalin Marinas <catalin.marinas@arm.com>, kernel-team@android.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 14/21] KVM: arm64: Add support for stage-2 cache flushing in generic page-table Date: Thu, 3 Sep 2020 14:51:30 +1000 [thread overview] Message-ID: <4198f2f9-abaa-2755-0113-38384150b595@redhat.com> (raw) In-Reply-To: <20200825093953.26493-15-will@kernel.org> Hi Will, On 8/25/20 7:39 PM, Will Deacon wrote: > From: Quentin Perret <qperret@google.com> > > Add support for cache flushing a range of the stage-2 address space to > the generic page-table code. > > Cc: Marc Zyngier <maz@kernel.org> > Signed-off-by: Quentin Perret <qperret@google.com> > Signed-off-by: Will Deacon <will@kernel.org> > --- Reviewed-by: Gavin Shan <gshan@redhat.com> > arch/arm64/include/asm/kvm_pgtable.h | 12 ++++++++++++ > arch/arm64/kvm/hyp/pgtable.c | 26 ++++++++++++++++++++++++++ > 2 files changed, 38 insertions(+) > > diff --git a/arch/arm64/include/asm/kvm_pgtable.h b/arch/arm64/include/asm/kvm_pgtable.h > index 0c96b78d791d..ea823fe31913 100644 > --- a/arch/arm64/include/asm/kvm_pgtable.h > +++ b/arch/arm64/include/asm/kvm_pgtable.h > @@ -226,6 +226,18 @@ kvm_pte_t kvm_pgtable_stage2_mkold(struct kvm_pgtable *pgt, u64 addr); > */ > bool kvm_pgtable_stage2_is_young(struct kvm_pgtable *pgt, u64 addr); > > +/** > + * kvm_pgtable_stage2_flush_range() - Clean and invalidate data cache to Point > + * of Coherency for guest stage-2 address > + * range. > + * @pgt: Page-table structure initialised by kvm_pgtable_stage2_init(). > + * @addr: Intermediate physical address from which to flush. > + * @size: Size of the range. > + * > + * Return: 0 on success, negative error code on failure. > + */ > +int kvm_pgtable_stage2_flush(struct kvm_pgtable *pgt, u64 addr, u64 size); > + > /** > * kvm_pgtable_walk() - Walk a page-table. > * @pgt: Page-table structure initialised by kvm_pgtable_*_init(). > diff --git a/arch/arm64/kvm/hyp/pgtable.c b/arch/arm64/kvm/hyp/pgtable.c > index c218651f8eba..75887185f1e2 100644 > --- a/arch/arm64/kvm/hyp/pgtable.c > +++ b/arch/arm64/kvm/hyp/pgtable.c > @@ -762,6 +762,32 @@ bool kvm_pgtable_stage2_is_young(struct kvm_pgtable *pgt, u64 addr) > return pte & KVM_PTE_LEAF_ATTR_LO_S2_AF; > } > > +static int stage2_flush_walker(u64 addr, u64 end, u32 level, kvm_pte_t *ptep, > + enum kvm_pgtable_walk_flags flag, > + void * const arg) > +{ > + kvm_pte_t pte = *ptep; > + > + if (!kvm_pte_valid(pte) || !stage2_pte_cacheable(pte)) > + return 0; > + > + stage2_flush_dcache(kvm_pte_follow(pte), kvm_granule_size(level)); > + return 0; > +} > + > +int kvm_pgtable_stage2_flush(struct kvm_pgtable *pgt, u64 addr, u64 size) > +{ > + struct kvm_pgtable_walker walker = { > + .cb = stage2_flush_walker, > + .flags = KVM_PGTABLE_WALK_LEAF, > + }; > + > + if (cpus_have_const_cap(ARM64_HAS_STAGE2_FWB)) > + return 0; > + > + return kvm_pgtable_walk(pgt, addr, size, &walker); > +} > + > int kvm_pgtable_stage2_init(struct kvm_pgtable *pgt, struct kvm *kvm) > { > size_t pgd_sz; > Thanks, Gavin _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-09-03 4:51 UTC|newest] Thread overview: 172+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-08-25 9:39 [PATCH v3 00/21] KVM: arm64: Rewrite page-table code and fault handling Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-08-25 9:39 ` [PATCH v3 01/21] KVM: arm64: Remove kvm_mmu_free_memory_caches() Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-08-25 9:39 ` [PATCH v3 02/21] KVM: arm64: Add stand-alone page-table walker infrastructure Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-08-27 16:27 ` Alexandru Elisei 2020-08-27 16:27 ` Alexandru Elisei 2020-08-28 15:43 ` Alexandru Elisei 2020-08-28 15:43 ` Alexandru Elisei 2020-09-02 10:36 ` Will Deacon 2020-09-02 10:36 ` Will Deacon 2020-08-28 15:51 ` Alexandru Elisei 2020-08-28 15:51 ` Alexandru Elisei 2020-09-02 10:49 ` Will Deacon 2020-09-02 10:49 ` Will Deacon 2020-09-02 6:31 ` Gavin Shan 2020-09-02 6:31 ` Gavin Shan 2020-09-02 11:02 ` Will Deacon 2020-09-02 11:02 ` Will Deacon 2020-09-03 1:11 ` Gavin Shan 2020-09-03 1:11 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 03/21] KVM: arm64: Add support for creating kernel-agnostic stage-1 page tables Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-08-28 15:35 ` Alexandru Elisei 2020-08-28 15:35 ` Alexandru Elisei 2020-09-02 10:06 ` Will Deacon 2020-09-02 10:06 ` Will Deacon 2020-08-25 9:39 ` [PATCH v3 04/21] KVM: arm64: Use generic allocator for hyp stage-1 page-tables Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-08-28 16:32 ` Alexandru Elisei 2020-08-28 16:32 ` Alexandru Elisei 2020-09-02 11:35 ` Will Deacon 2020-09-02 11:35 ` Will Deacon 2020-09-02 14:48 ` Alexandru Elisei 2020-09-02 14:48 ` Alexandru Elisei 2020-08-25 9:39 ` [PATCH v3 05/21] KVM: arm64: Add support for creating kernel-agnostic stage-2 page tables Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-02 6:40 ` Gavin Shan 2020-09-02 6:40 ` Gavin Shan 2020-09-02 11:30 ` Will Deacon 2020-09-02 11:30 ` Will Deacon 2020-08-25 9:39 ` [PATCH v3 06/21] KVM: arm64: Add support for stage-2 map()/unmap() in generic page-table Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-01 16:24 ` Alexandru Elisei 2020-09-01 16:24 ` Alexandru Elisei 2020-09-02 11:46 ` Will Deacon 2020-09-02 11:46 ` Will Deacon 2020-09-03 2:57 ` Gavin Shan 2020-09-03 2:57 ` Gavin Shan 2020-09-03 5:27 ` Gavin Shan 2020-09-03 5:27 ` Gavin Shan 2020-09-03 11:18 ` Gavin Shan 2020-09-03 11:18 ` Gavin Shan 2020-09-03 12:30 ` Will Deacon 2020-09-03 12:30 ` Will Deacon 2020-09-03 16:15 ` Will Deacon 2020-09-03 16:15 ` Will Deacon 2020-09-04 0:47 ` Gavin Shan 2020-09-04 0:47 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 07/21] KVM: arm64: Convert kvm_phys_addr_ioremap() to generic page-table API Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-01 17:08 ` Alexandru Elisei 2020-09-01 17:08 ` Alexandru Elisei 2020-09-02 11:48 ` Will Deacon 2020-09-02 11:48 ` Will Deacon 2020-09-03 3:57 ` Gavin Shan 2020-09-03 3:57 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 08/21] KVM: arm64: Convert kvm_set_spte_hva() " Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-02 15:37 ` Alexandru Elisei 2020-09-02 15:37 ` Alexandru Elisei 2020-09-03 16:37 ` Will Deacon 2020-09-03 16:37 ` Will Deacon 2020-09-03 4:13 ` Gavin Shan 2020-09-03 4:13 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 09/21] KVM: arm64: Convert unmap_stage2_range() " Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-02 16:23 ` Alexandru Elisei 2020-09-02 16:23 ` Alexandru Elisei 2020-09-02 18:44 ` Alexandru Elisei 2020-09-02 18:44 ` Alexandru Elisei 2020-09-03 17:57 ` Will Deacon 2020-09-03 17:57 ` Will Deacon 2020-09-08 13:07 ` Alexandru Elisei 2020-09-08 13:07 ` Alexandru Elisei 2020-09-09 10:57 ` Alexandru Elisei 2020-09-09 10:57 ` Alexandru Elisei 2020-09-03 4:19 ` Gavin Shan 2020-09-03 4:19 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 10/21] KVM: arm64: Add support for stage-2 page-aging in generic page-table Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 4:33 ` Gavin Shan 2020-09-03 4:33 ` Gavin Shan 2020-09-03 16:48 ` Will Deacon 2020-09-03 16:48 ` Will Deacon 2020-09-04 1:01 ` Gavin Shan 2020-09-04 1:01 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 11/21] KVM: arm64: Convert page-aging and access faults to generic page-table API Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 4:37 ` Gavin Shan 2020-09-03 4:37 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 12/21] KVM: arm64: Add support for stage-2 write-protect in generic page-table Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 4:47 ` Gavin Shan 2020-09-03 4:47 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 13/21] KVM: arm64: Convert write-protect operation to generic page-table API Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 4:48 ` Gavin Shan 2020-09-03 4:48 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 14/21] KVM: arm64: Add support for stage-2 cache flushing in generic page-table Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 4:51 ` Gavin Shan [this message] 2020-09-03 4:51 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 15/21] KVM: arm64: Convert memslot cache-flushing code to generic page-table API Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 4:52 ` Gavin Shan 2020-09-03 4:52 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 16/21] KVM: arm64: Add support for relaxing stage-2 perms in generic page-table code Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 4:55 ` Gavin Shan 2020-09-03 4:55 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 17/21] KVM: arm64: Convert user_mem_abort() to generic page-table API Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 6:05 ` Gavin Shan 2020-09-03 6:05 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 18/21] KVM: arm64: Check the pgt instead of the pgd when modifying page-table Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 5:00 ` Gavin Shan 2020-09-03 5:00 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 19/21] KVM: arm64: Remove unused page-table code Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 6:02 ` Gavin Shan 2020-09-03 6:02 ` Gavin Shan 2020-08-25 9:39 ` [PATCH v3 20/21] KVM: arm64: Remove unused 'pgd' field from 'struct kvm_s2_mmu' Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 5:07 ` Gavin Shan 2020-09-03 5:07 ` Gavin Shan 2020-09-03 16:50 ` Will Deacon 2020-09-03 16:50 ` Will Deacon 2020-09-04 0:59 ` Gavin Shan 2020-09-04 0:59 ` Gavin Shan 2020-09-04 10:02 ` Marc Zyngier 2020-09-04 10:02 ` Marc Zyngier 2020-08-25 9:39 ` [PATCH v3 21/21] KVM: arm64: Don't constrain maximum IPA size based on host configuration Will Deacon 2020-08-25 9:39 ` Will Deacon 2020-09-03 5:09 ` Gavin Shan 2020-09-03 5:09 ` Gavin Shan 2020-08-27 16:26 ` [PATCH v3 00/21] KVM: arm64: Rewrite page-table code and fault handling Alexandru Elisei 2020-08-27 16:26 ` Alexandru Elisei 2020-09-01 16:15 ` Will Deacon 2020-09-01 16:15 ` Will Deacon 2020-09-03 7:34 ` Gavin Shan 2020-09-03 7:34 ` Gavin Shan 2020-09-03 11:13 ` Gavin Shan 2020-09-03 11:13 ` Gavin Shan 2020-09-03 11:48 ` Gavin Shan 2020-09-03 11:48 ` Gavin Shan 2020-09-03 12:16 ` Will Deacon 2020-09-03 12:16 ` Will Deacon 2020-09-04 0:51 ` Gavin Shan 2020-09-04 0:51 ` Gavin Shan 2020-09-04 10:07 ` Marc Zyngier 2020-09-04 10:07 ` Marc Zyngier 2020-09-05 3:56 ` Gavin Shan 2020-09-05 3:56 ` Gavin Shan 2020-09-05 9:33 ` Marc Zyngier 2020-09-05 9:33 ` Marc Zyngier 2020-09-07 9:27 ` Will Deacon 2020-09-07 9:27 ` Will Deacon 2020-09-03 18:52 ` Will Deacon 2020-09-03 18:52 ` Will Deacon
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=4198f2f9-abaa-2755-0113-38384150b595@redhat.com \ --to=gshan@redhat.com \ --cc=catalin.marinas@arm.com \ --cc=kernel-team@android.com \ --cc=kvmarm@lists.cs.columbia.edu \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=maz@kernel.org \ --cc=will@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.