[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/8] Add universal DMA helper functions
From: |
Eduard - Gabriel Munteanu |
Subject: |
Re: [Qemu-devel] [PATCH 4/8] Add universal DMA helper functions |
Date: |
Sun, 26 Feb 2012 23:09:38 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sun, Feb 26, 2012 at 12:04:49PM +0200, Michael S. Tsirkin wrote:
> On Fri, Feb 24, 2012 at 02:27:39PM +1100, David Gibson wrote:
> > Not that long ago, every device implementation using DMA directly
> > accessed guest memory using cpu_physical_memory_*(). This meant that
> > adding support for a guest visible IOMMU would require changing every
> > one of these devices to go through IOMMU translation.
> >
> > Shortly before qemu 1.0, I made a start on fixing this by providing
> > helper functions for PCI DMA. These are currently just stubs which
> > call the direct access functions, but mean that an IOMMU can be
> > implemented in one place, rather than for every PCI device.
> >
> > Clearly, this doesn't help for non PCI devices, which could also be
> > IOMMU translated on some platforms. It is also problematic for the
> > devices which have both PCI and non-PCI version (e.g. OHCI, AHCI) - we
> > cannot use the the pci_dma_*() functions, because they assume the
> > presence of a PCIDevice, but we don't want to have to check between
> > pci_dma_*() and cpu_physical_memory_*() every time we do a DMA in the
> > device code.
> >
> > This patch makes the first step on addressing both these problems, by
> > introducing new (stub) dma helper functions which can be used for any
> > DMA capable device.
> >
> > These dma functions take a DMAContext *, a new (currently empty)
> > variable describing the DMA address space in which the operation is to
> > take place. NULL indicates untranslated DMA directly into guest
> > physical address space. The intention is that in future non-NULL
> > values will given information about any necessary IOMMU translation.
> >
> > DMA using devices must obtain a DMAContext (or, potentially, contexts)
> > from their bus or platform. For now this patch just converts the PCI
> > wrappers to be implemented in terms of the universal wrappers,
> > converting other drivers can take place over time.
> >
> > Cc: Michael S. Tsirkin <address@hidden>
> > Cc: Joerg Rodel <address@hidden>
> > Cc: Eduard - Gabriel Munteanu <address@hidden>
> > Cc: Richard Henderson <address@hidden>
> >
> > Signed-off-by: David Gibson <address@hidden>
>
> I'm a bit confused with all the stubbing going on.
> Is this the final form of the pci_* functions or just
> a stub? If the final form, we probably should just
> open-code them - they don't buy us much.
> If not, let's add a comment?
>
It is a stub. The final form needs to deal with address translation,
permissions checking and invalidating memory maps.
[snip]
Eduard
[Qemu-devel] [PATCH 8/8] Make dma_addr_t 64 bit always, David Gibson, 2012/02/23
[Qemu-devel] [PATCH 5/8] usb-ohci: Use universal DMA helper functions, David Gibson, 2012/02/23
[Qemu-devel] [PATCH 6/8] Make sglists and dma_bdrv helpers use new universal DMA herlpers, David Gibson, 2012/02/23