From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marc Zyngier Subject: Re: [PATCH v3 09/55] KVM: arm/arm64: Fix MMIO emulation data handling Date: Tue, 10 May 2016 09:57:03 +0100 Message-ID: <5731A25F.4020203@arm.com> References: <1462531568-9799-1-git-send-email-andre.przywara@arm.com> <1462531568-9799-10-git-send-email-andre.przywara@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: Eric Auger , kvmarm@lists.cs.columbia.edu, kvm@vger.kernel.org, linux-arm-kernel@lists.infradead.org To: Andre Przywara , Christoffer Dall Return-path: Received: from foss.arm.com ([217.140.101.70]:42852 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750974AbcEJI5I (ORCPT ); Tue, 10 May 2016 04:57:08 -0400 In-Reply-To: <1462531568-9799-10-git-send-email-andre.przywara@arm.com> Sender: kvm-owner@vger.kernel.org List-ID: On 06/05/16 11:45, Andre Przywara wrote: > From: Christoffer Dall > > When the kernel was handling a guest MMIO read access internally, we > need to copy the emulation result into the run->mmio structure in order > for the kvm_handle_mmio_return() function to pick it up and inject the > result back into the guest. > > Currently the only user of kvm_io_bus for ARM is the VGIC, which did > this copying itself, so this was not causing issues so far. > > But with the upcoming new vgic implementation we need this done > properly. > > Update the kvm_handle_mmio_return description and cleanup the code to > only perform a single copying when needed. > > Code and commit message inspired by Andre Przywara. > > Reported-by: Andre Przywara > Signed-off-by: Christoffer Dall > Signed-off-by: Andre Przywara Reviewed-by: Marc Zyngier M. -- Jazz is not dead. It just smells funny...