[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/13] Use DMADirection type for dma_bdrv_io
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH 01/13] Use DMADirection type for dma_bdrv_io |
Date: |
Tue, 13 Mar 2012 16:53:32 +1100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Mar 09, 2012 at 11:01:16AM +0100, Paolo Bonzini wrote:
> Il 09/03/2012 06:01, David Gibson ha scritto:
> > Currently dma_bdrv_io() takes a 'to_dev' boolean parameter to
> > determine the direction of DMA it is emulating. We already have a
> > DMADirection enum designed specifically to encode DMA directions.
> > This patch uses it for dma_bdrv_io() as well. This involves removing
> > the DMADirection definition from the #ifdef it was inside, but since that
> > only existed to protect the definition of dma_addr_t from places where
> > config.h is not included, there wasn't any reason for it to be there in
> > the first place.
>
> Looks good, but please use this for dma_buf_* too.
Well, dma_buf_rw() is the only one that needs change, and it is
static, so the importance is not so great. However, more importantly,
dma_buf_rw() still uses cpu_physical_memory_rw() because I missed
switching it to the universal wrappers. I'll fix that up as well as
the DMADirection stuff in the next spin of the patch adding DMAContext
awareness to sglists and the rest.
--
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 05/13] iommu: Add universal DMA helper functions, (continued)
[Qemu-devel] [PATCH 04/13] Implement cpu_physical_memory_zero(), David Gibson, 2012/03/09
[Qemu-devel] [PATCH 12/13] iommu: Allow PCI to use IOMMU infrastructure, David Gibson, 2012/03/09
[Qemu-devel] [PATCH 07/13] iommu: Make sglists and dma_bdrv helpers use new universal DMA helpers, David Gibson, 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