From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:51461) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S9hlo-0006AH-5w for qemu-devel@nongnu.org; Mon, 19 Mar 2012 14:57:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S9hlX-0003u5-PO for qemu-devel@nongnu.org; Mon, 19 Mar 2012 14:57:27 -0400 Received: from mx1.redhat.com ([209.132.183.28]:14752) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S9hlX-0003tO-Fe for qemu-devel@nongnu.org; Mon, 19 Mar 2012 14:57:11 -0400 Date: Mon, 19 Mar 2012 20:57:23 +0200 From: "Michael S. Tsirkin" Message-ID: <20120319185723.GA8366@redhat.com> References: <20120319155650.GA6430@redhat.com> <4F6773B4.2040209@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4F6773B4.2040209@redhat.com> Subject: Re: [Qemu-devel] [PATCH RFC] virtio-pci: add MMIO property List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avi Kivity Cc: Anthony Liguori , kvm@vger.kernel.org, Alexey Kardashevskiy , rusty@rustcorp.com.au, qemu-devel@nongnu.org, virtualization@lists.linux-foundation.org On Mon, Mar 19, 2012 at 07:58:12PM +0200, Avi Kivity wrote: > 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. No, that would need guest changes to be of use. The point of this hack is to make things work for Linux guests where PIO does not work. > -- > error compiling committee.c: too many arguments to function