Re: [PATCH 06/10] KVM: arm/arm64: Only clean the dcache on translation fault

2017-10-17 Thread Marc Zyngier
On 17/10/17 15:36, Christoffer Dall wrote: > On Tue, Oct 17, 2017 at 10:34:15AM +0100, Marc Zyngier wrote: >> On 16/10/17 21:08, Christoffer Dall wrote: >>> On Mon, Oct 09, 2017 at 04:20:28PM +0100, Marc Zyngier wrote: The only case where we actually need to perform a dache maintenance

Re: [PATCH 06/10] KVM: arm/arm64: Only clean the dcache on translation fault

2017-10-17 Thread Christoffer Dall
On Tue, Oct 17, 2017 at 10:34:15AM +0100, Marc Zyngier wrote: > On 16/10/17 21:08, Christoffer Dall wrote: > > On Mon, Oct 09, 2017 at 04:20:28PM +0100, Marc Zyngier wrote: > >> The only case where we actually need to perform a dache maintenance > >> is when we map the page for the first time, and

Re: [PATCH 06/10] KVM: arm/arm64: Only clean the dcache on translation fault

2017-10-17 Thread Marc Zyngier
On 16/10/17 21:08, Christoffer Dall wrote: > On Mon, Oct 09, 2017 at 04:20:28PM +0100, Marc Zyngier wrote: >> The only case where we actually need to perform a dache maintenance >> is when we map the page for the first time, and subsequent permission >> faults do not require cache maintenance.

Re: [PATCH 06/10] KVM: arm/arm64: Only clean the dcache on translation fault

2017-10-16 Thread Christoffer Dall
On Mon, Oct 09, 2017 at 04:20:28PM +0100, Marc Zyngier wrote: > The only case where we actually need to perform a dache maintenance > is when we map the page for the first time, and subsequent permission > faults do not require cache maintenance. Let's make it conditional > on not being a

[PATCH 06/10] KVM: arm/arm64: Only clean the dcache on translation fault

2017-10-09 Thread Marc Zyngier
The only case where we actually need to perform a dache maintenance is when we map the page for the first time, and subsequent permission faults do not require cache maintenance. Let's make it conditional on not being a permission fault (and thus a translation fault). Signed-off-by: Marc Zyngier