From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marc Zyngier Subject: Re: [PATCH v2] ARM/arm64: KVM: fix use of WnR bit in kvm_is_write_fault() Date: Tue, 09 Sep 2014 12:02:59 +0100 Message-ID: <540EDE63.6060904@arm.com> References: <1410258429-17090-1-git-send-email-ard.biesheuvel@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: 8BIT Cc: "linux-arm-kernel@lists.infradead.org" , "christoffer.dall@linaro.org" , "kvmarm@lists.cs.columbia.edu" , "kvm@vger.kernel.org" , "peter.maydell@linaro.org" , "lersek@redhat.com" To: Ard Biesheuvel Return-path: Received: from service87.mimecast.com ([91.220.42.44]:35413 "EHLO service87.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756423AbaIILDF convert rfc822-to-8bit (ORCPT ); Tue, 9 Sep 2014 07:03:05 -0400 In-Reply-To: <1410258429-17090-1-git-send-email-ard.biesheuvel@linaro.org> Sender: kvm-owner@vger.kernel.org List-ID: [resending, as ARM email server seems to be in some mood] On 09/09/14 11:27, Ard Biesheuvel wrote: > The ISS encoding for an exception from a Data Abort has a WnR > bit[6] that indicates whether the Data Abort was caused by a > read or a write instruction. While there are several fields > in the encoding that are only valid if the ISV bit[24] is set, > WnR is not one of them, so we can read it unconditionally. > > Instead of fixing both implementations of kvm_is_write_fault() > in place, reimplement it just once using kvm_vcpu_dabt_iswrite(), > which already does the right thing with respect to the WnR bit. > Also fix up the callers to pass 'vcpu' > > Acked-by: Laszlo Ersek > Signed-off-by: Ard Biesheuvel Because I like that kind of diffstat: Acked-by: Marc Zyngier Christoffer, if you too are happy with that, I'll queue it right away. Thanks, M. -- Jazz is not dead. It just smells funny...