From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:43405) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S9gqc-0008Ko-W3 for qemu-devel@nongnu.org; Mon, 19 Mar 2012 13:58:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S9gqb-0000LA-AW for qemu-devel@nongnu.org; Mon, 19 Mar 2012 13:58:22 -0400 Received: from mx1.redhat.com ([209.132.183.28]:24410) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S9gqb-0000L4-2h for qemu-devel@nongnu.org; Mon, 19 Mar 2012 13:58:21 -0400 Message-ID: <4F6773B4.2040209@redhat.com> Date: Mon, 19 Mar 2012 19:58:12 +0200 From: Avi Kivity MIME-Version: 1.0 References: <20120319155650.GA6430@redhat.com> In-Reply-To: <20120319155650.GA6430@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH RFC] virtio-pci: add MMIO property List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: Anthony Liguori , kvm@vger.kernel.org, Alexey Kardashevskiy , rusty@rustcorp.com.au, qemu-devel@nongnu.org, virtualization@lists.linux-foundation.org On 03/19/2012 05:56 PM, Michael S. Tsirkin wrote: > Currently virtio-pci is specified so that configuration of the device is > done through a PCI IO space (via BAR 0 of the virtual PCI device). > However, Linux guests happen to use ioread/iowrite/iomap primitives > for access, and these work uniformly across memory/io BARs. > > While PCI IO accesses are faster than MMIO on x86 kvm, > MMIO might be helpful on other systems which don't > implement PIO or where PIO is slower than MMIO. > > Add a property to make it possible to tweak the BAR type. > > Signed-off-by: Michael S. Tsirkin > > This is harmless by default but causes segfaults in memory.c > when enabled. Thus an RFC until I figure out what's wrong. > Should be done via an extra BAR (with the same layout, perhaps extended) so compatibility is preserved. -- error compiling committee.c: too many arguments to function