Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-04 Thread Marc Zyngier
On Wed, 3 Mar 2021 10:42:25 +0800, Jia He wrote: > If the start addr is not aligned with the granule size of that level. > loop step size should be adjusted to boundary instead of simple > kvm_granual_size(level) increment. Otherwise, some mmu entries might miss > the chance to be walked through. >

Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-04 Thread Will Deacon
On Thu, Mar 04, 2021 at 09:16:25AM +, Marc Zyngier wrote: > On 2021-03-04 00:46, Justin He wrote: > > > On Wed, Mar 03, 2021 at 07:07:37PM +, Marc Zyngier wrote: > > > > From e0524b41a71e0f17d6dc8f197e421e677d584e72 Mon Sep 17 00:00:00 2001 > > > > From: Jia He > > > > Date: Wed, 3 Mar 202

Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-04 Thread Marc Zyngier
; Quentin Perret ; linux-arm- ker...@lists.infradead.org; linux-kernel@vger.kernel.org Subject: Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking On Wed, Mar 03, 2021 at 07:07:37PM +, Marc Zyngier wrote: > From e0524b41a71e0f17d6dc8f197e421e677d584e72 Mon Sep 17 00:00:00 2001 >

RE: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-03 Thread Justin He
Quentin Perret ; linux-arm- > ker...@lists.infradead.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking > > On Wed, Mar 03, 2021 at 07:07:37PM +, Marc Zyngier wrote: > > From e0524b41a71e0f17d6dc8f197e421e677d584e72 Mon Se

RE: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-03 Thread Justin He
9 PM > To: Marc Zyngier > Cc: Justin He ; kvm...@lists.cs.columbia.edu; James > Morse ; Julien Thierry ; > Suzuki Poulose ; Catalin Marinas > ; Will Deacon ; Gavin Shan > ; Yanan Wang ; linux-arm- > ker...@lists.infradead.org; linux-kernel@vger.kernel.org > Subject: Re: [PATC

Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-03 Thread Will Deacon
On Wed, Mar 03, 2021 at 07:07:37PM +, Marc Zyngier wrote: > From e0524b41a71e0f17d6dc8f197e421e677d584e72 Mon Sep 17 00:00:00 2001 > From: Jia He > Date: Wed, 3 Mar 2021 10:42:25 +0800 > Subject: [PATCH] KVM: arm64: Fix range alignment when walking page tables > > When walking the page tables

Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-03 Thread Marc Zyngier
On Wed, 03 Mar 2021 11:29:34 +, Will Deacon wrote: > > On Wed, Mar 03, 2021 at 10:42:25AM +0800, Jia He wrote: > > If the start addr is not aligned with the granule size of that level. > > loop step size should be adjusted to boundary instead of simple > > kvm_granual_size(level) increment. O

Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-03 Thread Will Deacon
On Wed, Mar 03, 2021 at 09:54:25AM +, Marc Zyngier wrote: > Hi Jia, > > On Wed, 03 Mar 2021 02:42:25 +, > Jia He wrote: > > > > If the start addr is not aligned with the granule size of that level. > > loop step size should be adjusted to boundary instead of simple > > kvm_granual_size(l

Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-03 Thread Will Deacon
On Wed, Mar 03, 2021 at 10:42:25AM +0800, Jia He wrote: > If the start addr is not aligned with the granule size of that level. > loop step size should be adjusted to boundary instead of simple > kvm_granual_size(level) increment. Otherwise, some mmu entries might miss > the chance to be walked thr

Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-03 Thread Quentin Perret
On Wednesday 03 Mar 2021 at 09:54:25 (+), Marc Zyngier wrote: > Hi Jia, > > On Wed, 03 Mar 2021 02:42:25 +, > Jia He wrote: > > > > If the start addr is not aligned with the granule size of that level. > > loop step size should be adjusted to boundary instead of simple > > kvm_granual_si

Re: [PATCH] KVM: arm64: Fix unaligned addr case in mmu walking

2021-03-03 Thread Marc Zyngier
Hi Jia, On Wed, 03 Mar 2021 02:42:25 +, Jia He wrote: > > If the start addr is not aligned with the granule size of that level. > loop step size should be adjusted to boundary instead of simple > kvm_granual_size(level) increment. Otherwise, some mmu entries might miss > the chance to be wal