From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44326) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W3P84-0005KT-Ew for qemu-devel@nongnu.org; Wed, 15 Jan 2014 06:59:34 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W3P7y-0004JS-7p for qemu-devel@nongnu.org; Wed, 15 Jan 2014 06:59:28 -0500 Received: from mail-qc0-x22f.google.com ([2607:f8b0:400d:c01::22f]:45217) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W3P7y-0004JE-4E for qemu-devel@nongnu.org; Wed, 15 Jan 2014 06:59:22 -0500 Received: by mail-qc0-f175.google.com with SMTP id x13so826032qcv.6 for ; Wed, 15 Jan 2014 03:59:21 -0800 (PST) Sender: Paolo Bonzini Message-ID: <52D67815.4020205@redhat.com> Date: Wed, 15 Jan 2014 12:59:17 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1389338418-26244-1-git-send-email-aik@ozlabs.ru> In-Reply-To: <1389338418-26244-1-git-send-email-aik@ozlabs.ru> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] KVM: fix addr type for KVM_IOEVENTFD List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexey Kardashevskiy Cc: "Michael S. Tsirkin" , qemu-devel@nongnu.org, Alexander Graf Il 10/01/2014 08:20, Alexey Kardashevskiy ha scritto: > The @addr here is a guest physical address and can easily be bigger > than 4G. > > This changes uint32_t to hwaddr. > > Cc: Michael S. Tsirkin > Signed-off-by: Alexey Kardashevskiy > --- > > This was found during investigation why vhost is not always working > on PPC64-spapr. It turned out that KVM_IOEVENTFD was called with a 32bit address > while the actual guest physycal address for that PCI host bus was 0x100.0000.0000 > and __kvm_io_bus_write() in the host kernel would never succeed. > > --- > kvm-all.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/kvm-all.c b/kvm-all.c > index 3937754..dcd7cdd 100644 > --- a/kvm-all.c > +++ b/kvm-all.c > @@ -519,7 +519,7 @@ int kvm_check_extension(KVMState *s, unsigned int extension) > return ret; > } > > -static int kvm_set_ioeventfd_mmio(int fd, uint32_t addr, uint32_t val, > +static int kvm_set_ioeventfd_mmio(int fd, hwaddr addr, uint32_t val, > bool assign, uint32_t size, bool datamatch) > { > int ret; > Applied to uq/master, thanks. Paolo