From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:53456) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RDpEu-00054E-Ps for qemu-devel@nongnu.org; Tue, 11 Oct 2011 23:12:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RDpEt-0004xo-2H for qemu-devel@nongnu.org; Tue, 11 Oct 2011 23:12:16 -0400 Received: from e23smtp02.au.ibm.com ([202.81.31.144]:57801) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RDpEs-0004uK-HE for qemu-devel@nongnu.org; Tue, 11 Oct 2011 23:12:15 -0400 Received: from d23relay05.au.ibm.com (d23relay05.au.ibm.com [202.81.31.247]) by e23smtp02.au.ibm.com (8.14.4/8.13.1) with ESMTP id p9C35CK2026058 for ; Wed, 12 Oct 2011 14:05:12 +1100 Received: from d23av03.au.ibm.com (d23av03.au.ibm.com [9.190.234.97]) by d23relay05.au.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id p9C39SoS1384492 for ; Wed, 12 Oct 2011 14:09:28 +1100 Received: from d23av03.au.ibm.com (loopback [127.0.0.1]) by d23av03.au.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id p9C3BoLm032576 for ; Wed, 12 Oct 2011 14:11:51 +1100 Date: Wed, 12 Oct 2011 14:11:37 +1100 From: David Gibson Message-ID: <20111012031137.GQ4849@truffala.fritz.box> References: <1315197304-22469-1-git-send-email-david@gibson.dropbear.id.au> <1315197304-22469-2-git-send-email-david@gibson.dropbear.id.au> <20111002102547.GC30747@redhat.com> <4E883CF4.6060606@redhat.com> <20111002105238.GE30747@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20111002105238.GE30747@redhat.com> Subject: Re: [Qemu-devel] [PATCH 1/9] Add stub functions for PCI device models to do PCI DMA List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: aliguori@us.ibm.com, kraxel@redhat.com, joerg.roedel@amd.com, agraf@suse.de, qemu-devel@nongnu.org, Avi Kivity , eduard.munteanu@linux360.ro, rth@twiddle.net On Sun, Oct 02, 2011 at 12:52:39PM +0200, Michael S. Tsirkin wrote: > On Sun, Oct 02, 2011 at 12:29:08PM +0200, Avi Kivity wrote: > > On 10/02/2011 12:25 PM, Michael S. Tsirkin wrote: > > >On Mon, Sep 05, 2011 at 02:34:56PM +1000, David Gibson wrote: > > >> This patch adds functions to pci.[ch] to perform PCI DMA operations. At > > >> present, these are just stubs which perform directly cpu physical memory > > >> accesses. > > >> > > >> Using these stubs, however, distinguishes PCI device DMA transactions from > > >> other accesses to physical memory, which will allow PCI IOMMU support to > > >> be added in one place, rather than updating every PCI driver at that time. > > >> > > >> That is, it allows us to update individual PCI drivers to support an IOMMU > > >> without having yet determined the details of how the IOMMU emulation will > > >> operate. This will let us remove the most bitrot-sensitive part of an > > >> IOMMU patch in advance. > > >> > > >> Signed-off-by: David Gibson > > > > > >So something I just thought about: > > > > > >all wrappers now go through cpu_physical_memory_rw. > > >This is a problem as e.g. virtio assumes that > > >accesses such as stw are atomic. cpu_physical_memory_rw > > >is a memcpy which makes no such guarantees. > > > > > > > Let's change cpu_physical_memory_rw() to provide that guarantee for > > aligned two and four byte accesses. Having separate paths just for > > that is not maintainable. > > Well, we also have stX_phys convert to target native endian-ness > (nop for KVM but not necessarily for qemu). Yes.. as do the stX_pci_dma() helpers. They assume LE, rather than having two variants, because PCI is an LE spec, and all normal PCI devices work in LE. If we need to model some perverse BE PCI device, it can reswap itself. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson