Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-30 Thread Quentin Perret
On Tuesday 27 Jul 2021 at 19:11:08 (+0100), Will Deacon wrote: > On Thu, Jul 15, 2021 at 05:31:47PM +0100, Marc Zyngier wrote: > > Introduce the infrastructure required to identify an IPA region > > that is expected to be used as an MMIO window. > > > > This include mapping, unmapping and

Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-30 Thread Marc Zyngier
On Fri, 30 Jul 2021 13:26:59 +0100, Will Deacon wrote: > > On Wed, Jul 28, 2021 at 10:57:30AM +0100, Marc Zyngier wrote: > > On Tue, 27 Jul 2021 19:11:08 +0100, > > Will Deacon wrote: > > > On Thu, Jul 15, 2021 at 05:31:47PM +0100, Marc Zyngier wrote: > > > > +bool

Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-30 Thread Will Deacon
On Wed, Jul 28, 2021 at 10:57:30AM +0100, Marc Zyngier wrote: > On Tue, 27 Jul 2021 19:11:08 +0100, > Will Deacon wrote: > > On Thu, Jul 15, 2021 at 05:31:47PM +0100, Marc Zyngier wrote: > > > +bool kvm_install_ioguard_page(struct kvm_vcpu *vcpu, gpa_t ipa) > > > +{ > > > + struct

Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-28 Thread Marc Zyngier
On Tue, 27 Jul 2021 19:11:08 +0100, Will Deacon wrote: > > On Thu, Jul 15, 2021 at 05:31:47PM +0100, Marc Zyngier wrote: > > Introduce the infrastructure required to identify an IPA region > > that is expected to be used as an MMIO window. > > > > This include mapping, unmapping and checking

Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-27 Thread Will Deacon
On Thu, Jul 15, 2021 at 05:31:47PM +0100, Marc Zyngier wrote: > Introduce the infrastructure required to identify an IPA region > that is expected to be used as an MMIO window. > > This include mapping, unmapping and checking the regions. Nothing > calls into it yet, so no expected functional

Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-23 Thread Quentin Perret
On Thursday 22 Jul 2021 at 19:04:59 (+0100), Marc Zyngier wrote: > So FWIW, I've now pushed out an updated series for the THP changes[1], > and you will find a similar patch at the base of the branch. Please > have a look and let me know what you think! I Like the look of it! I'll pull this patch

Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-22 Thread Marc Zyngier
On Tue, 20 Jul 2021 16:49:45 +0100, Quentin Perret wrote: > > On Tuesday 20 Jul 2021 at 14:15:56 (+0100), Marc Zyngier wrote: > > On Tue, 20 Jul 2021 12:13:20 +0100, > > Quentin Perret wrote: > > > > > > On Thursday 15 Jul 2021 at 17:31:47 (+0100), Marc Zyngier wrote: > > > > +struct

Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-20 Thread Quentin Perret
On Tuesday 20 Jul 2021 at 14:15:56 (+0100), Marc Zyngier wrote: > On Tue, 20 Jul 2021 12:13:20 +0100, > Quentin Perret wrote: > > > > On Thursday 15 Jul 2021 at 17:31:47 (+0100), Marc Zyngier wrote: > > > +struct s2_walk_data { > > > + kvm_pte_t pteval; > > > + u32 level; > > >

Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-20 Thread Quentin Perret
On Thursday 15 Jul 2021 at 17:31:47 (+0100), Marc Zyngier wrote: > +struct s2_walk_data { > + kvm_pte_t pteval; > + u32 level; > +}; > + > +static int s2_walker(u64 addr, u64 end, u32 level, kvm_pte_t *ptep, > + enum kvm_pgtable_walk_flags flag, void *

Re: [PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-20 Thread Marc Zyngier
On Tue, 20 Jul 2021 12:13:20 +0100, Quentin Perret wrote: > > On Thursday 15 Jul 2021 at 17:31:47 (+0100), Marc Zyngier wrote: > > +struct s2_walk_data { > > + kvm_pte_t pteval; > > + u32 level; > > +}; > > + > > +static int s2_walker(u64 addr, u64 end, u32 level, kvm_pte_t

[PATCH 04/16] KVM: arm64: Add MMIO checking infrastructure

2021-07-15 Thread Marc Zyngier
Introduce the infrastructure required to identify an IPA region that is expected to be used as an MMIO window. This include mapping, unmapping and checking the regions. Nothing calls into it yet, so no expected functional change. Signed-off-by: Marc Zyngier ---