[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/13] ide/ahci: Use universal DMA helper functi
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH 08/13] ide/ahci: Use universal DMA helper functions |
Date: |
Sat, 10 Mar 2012 16:30:25 +1100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Mar 09, 2012 at 10:48:20AM +0100, Kevin Wolf wrote:
> Am 09.03.2012 06:01, schrieb David Gibson:
> > The AHCI device can provide both PCI and SysBus AHCI device
> > emulations. For this reason, it wasn't previously converted to use
> > the pci_dma_*() helper functions. Now that we have universal DMA
> > helper functions, this converts AHCI to use them.
> >
> > The DMAContext is obtained from pci_dma_context() in the PCI case and
> > set to NULL in the SysBus case (i.e. we assume for now that a SysBus
> > AHCI has no IOMMU translation).
> >
> > Cc: Kevin Wolf <address@hidden>
> > Cc: Michael S. Tsirkin <address@hidden>
> >
> > Signed-off-by: David Gibson <address@hidden>
> > ---
> > hw/ide/ahci.c | 7 ++++---
> > hw/ide/ahci.h | 3 ++-
> > hw/ide/ich.c | 2 +-
> > 3 files changed, 7 insertions(+), 5 deletions(-)
> >
> > diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c
> > index 6a218b5..3d31179 100644
> > --- a/hw/ide/ahci.c
> > +++ b/hw/ide/ahci.c
> > @@ -668,7 +668,7 @@ static int ahci_populate_sglist(AHCIDevice *ad,
> > QEMUSGList *sglist)
> > AHCI_SG *tbl = (AHCI_SG *)prdt;
> >
> > /* FIXME: pass the correct DMAContext */
> > - qemu_sglist_init(sglist, sglist_alloc_hint, NULL);
> > + qemu_sglist_init(sglist, sglist_alloc_hint, ad->hba->dma);
>
> Does the FIXME still apply after this change?
Ah, no, I'll fix that in the next version. I've also spotted there
are still a bunch of cpu_physical_memory_*() calls in there that I
either missed, or which have been added since I did the conversion.
I'll tackle that next week.
--
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
- Re: [Qemu-devel] [PATCH 07/13] iommu: Make sglists and dma_bdrv helpers use new universal DMA helpers, (continued)
Re: [Qemu-devel] [PATCH 07/13] iommu: Make sglists and dma_bdrv helpers use new universal DMA helpers, Paolo Bonzini, 2012/03/09
[Qemu-devel] [PATCH 01/13] Use DMADirection type for dma_bdrv_io, David Gibson, 2012/03/09
[Qemu-devel] [PATCH 08/13] ide/ahci: Use universal DMA helper functions, David Gibson, 2012/03/09
[Qemu-devel] [PATCH 13/13] pseries: Implement IOMMU and DMA for PAPR PCI devices, David Gibson, 2012/03/09
[Qemu-devel] [PATCH 06/13] usb-ohci: Use universal DMA helper functions, David Gibson, 2012/03/09
[Qemu-devel] [PATCH 03/13] usb-xhci: Use PCI DMA helper functions, David Gibson, 2012/03/09
[Qemu-devel] [PATCH 10/13] iommu: Introduce IOMMU emulation infrastructure, David Gibson, 2012/03/09