Re: [PATCH v2 3/3] kvm/arm: Align the VMID allocation with the arm64 ASID one

2021-07-23 Thread Vladimir Murzin
Hi Will, On 7/22/21 4:38 PM, Will Deacon wrote: > Hi Vladimir, > > On Thu, Jul 22, 2021 at 04:22:26PM +0100, Vladimir Murzin wrote: >> On 7/22/21 10:50 AM, Will Deacon wrote: >>> As an aside: I'm more and more inclined to rip out the CnP stuff given >>> that it doesn't appear to being any

Re: [PATCH v2 3/3] kvm/arm: Align the VMID allocation with the arm64 ASID one

2021-07-23 Thread Marco Elver
On Thu, Jul 22, 2021 at 10:11AM +0100, Quentin Perret wrote: > On Thursday 22 Jul 2021 at 06:45:14 (+), Shameerali Kolothum Thodi wrote: > > > From: Will Deacon [mailto:w...@kernel.org] > > > > diff --git a/arch/arm64/kvm/hyp/nvhe/mem_protect.c > > > b/arch/arm64/kvm/hyp/nvhe/mem_protect.c > >

Re: [PATCH v2 3/3] kvm/arm: Align the VMID allocation with the arm64 ASID one

2021-07-22 Thread Will Deacon
Hi Vladimir, On Thu, Jul 22, 2021 at 04:22:26PM +0100, Vladimir Murzin wrote: > On 7/22/21 10:50 AM, Will Deacon wrote: > > As an aside: I'm more and more inclined to rip out the CnP stuff given > > that it doesn't appear to being any benefits, but does have some clear > > downsides. Perhaps

Re: [PATCH v2 3/3] kvm/arm: Align the VMID allocation with the arm64 ASID one

2021-07-22 Thread Vladimir Murzin
On 7/22/21 10:50 AM, Will Deacon wrote: > As an aside: I'm more and more inclined to rip out the CnP stuff given > that it doesn't appear to being any benefits, but does have some clear > downsides. Perhaps something for next week. Can you please clarify what do you mean by "it doesn't appear to

Re: [PATCH v2 3/3] kvm/arm: Align the VMID allocation with the arm64 ASID one

2021-07-22 Thread Quentin Perret
On Thursday 22 Jul 2021 at 06:45:14 (+), Shameerali Kolothum Thodi wrote: > > From: Will Deacon [mailto:w...@kernel.org] > > > diff --git a/arch/arm64/kvm/hyp/nvhe/mem_protect.c > > b/arch/arm64/kvm/hyp/nvhe/mem_protect.c > > > index 4b60c0056c04..a02c4877a055 100644 > > > ---

Re: [PATCH v2 3/3] kvm/arm: Align the VMID allocation with the arm64 ASID one

2021-07-22 Thread Will Deacon
On Thu, Jul 22, 2021 at 06:45:14AM +, Shameerali Kolothum Thodi wrote: > > > diff --git a/arch/arm64/kvm/hyp/nvhe/tlb.c > > b/arch/arm64/kvm/hyp/nvhe/tlb.c > > > index 83dc3b271bc5..42df9931ed9a 100644 > > > --- a/arch/arm64/kvm/hyp/nvhe/tlb.c > > > +++ b/arch/arm64/kvm/hyp/nvhe/tlb.c > > > @@

RE: [PATCH v2 3/3] kvm/arm: Align the VMID allocation with the arm64 ASID one

2021-07-22 Thread Shameerali Kolothum Thodi
rm.com; > james.mo...@arm.com; julien.thierry.k...@gmail.com; > suzuki.poul...@arm.com; jean-phili...@linaro.org; > alexandru.eli...@arm.com; Linuxarm ; > qper...@google.com > Subject: Re: [PATCH v2 3/3] kvm/arm: Align the VMID allocation with the > arm64 ASID one > > [+Quenti

Re: [PATCH v2 3/3] kvm/arm: Align the VMID allocation with the arm64 ASID one

2021-07-21 Thread Will Deacon
[+Quentin] On Wed, Jun 16, 2021 at 04:56:06PM +0100, Shameer Kolothum wrote: > From: Julien Grall > > At the moment, the VMID algorithm will send an SGI to all the CPUs to > force an exit and then broadcast a full TLB flush and I-Cache > invalidation. > > This patch use the new VMID allocator.