qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: David Gibson <david@gibson.dropbear.id.au>
To: mst@redhat.com, anthony@codemonkey.ws
Cc: joerg.roedel@amd.com, qemu-devel@nongnu.org, agraf@suse.de,
	avi@redhat.com, eduard.munteanu@linux360.ro, rth@twiddle.net
Subject: [Qemu-devel] [PATCH 02/14] Use dma_addr_t type for scatter/gather code
Date: Mon, 31 Oct 2011 17:06:46 +1100	[thread overview]
Message-ID: <1320041218-30487-3-git-send-email-david@gibson.dropbear.id.au> (raw)
In-Reply-To: <1320041218-30487-1-git-send-email-david@gibson.dropbear.id.au>

This patch uses the newly created dma_addr_t type throughout the
scatter/gather handling code in dma-helpers.c whenever we need to
represent a dma bus address.  This makes a better distinction as to
what is a bus address and what is a cpu physical address.  Since we
don't support IOMMUs yet, they can't be very different for now, but
that will change in future, and this preliminary helps clarify what's
going on.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---
 dma-helpers.c |    5 ++---
 dma.h         |    9 ++++-----
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/dma-helpers.c b/dma-helpers.c
index 86d2d0a..bdcd38c 100644
--- a/dma-helpers.c
+++ b/dma-helpers.c
@@ -18,8 +18,7 @@ void qemu_sglist_init(QEMUSGList *qsg, int alloc_hint)
     qsg->size = 0;
 }
 
-void qemu_sglist_add(QEMUSGList *qsg, target_phys_addr_t base,
-                     target_phys_addr_t len)
+void qemu_sglist_add(QEMUSGList *qsg, dma_addr_t base, dma_addr_t len)
 {
     if (qsg->nsg == qsg->nalloc) {
         qsg->nalloc = 2 * qsg->nalloc + 1;
@@ -45,7 +44,7 @@ typedef struct {
     bool to_dev;
     bool in_cancel;
     int sg_cur_index;
-    target_phys_addr_t sg_cur_byte;
+    dma_addr_t sg_cur_byte;
     QEMUIOVector iov;
     QEMUBH *bh;
     DMAIOFunc *io_func;
diff --git a/dma.h b/dma.h
index 56e163a..a13209d 100644
--- a/dma.h
+++ b/dma.h
@@ -28,20 +28,19 @@ typedef enum {
 } DMADirection;
 
 struct ScatterGatherEntry {
-    target_phys_addr_t base;
-    target_phys_addr_t len;
+    dma_addr_t base;
+    dma_addr_t len;
 };
 
 struct QEMUSGList {
     ScatterGatherEntry *sg;
     int nsg;
     int nalloc;
-    target_phys_addr_t size;
+    dma_addr_t size;
 };
 
 void qemu_sglist_init(QEMUSGList *qsg, int alloc_hint);
-void qemu_sglist_add(QEMUSGList *qsg, target_phys_addr_t base,
-                     target_phys_addr_t len);
+void qemu_sglist_add(QEMUSGList *qsg, dma_addr_t base, dma_addr_t len);
 void qemu_sglist_destroy(QEMUSGList *qsg);
 #endif
 
-- 
1.7.7

  parent reply	other threads:[~2011-10-31  6:07 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-31  6:06 [Qemu-devel] [0/14] Preliminary work for IOMMU emulation support (v3) David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 01/14] Define DMA address and direction types David Gibson
2011-11-01 22:21   ` Anthony Liguori
2011-10-31  6:06 ` David Gibson [this message]
2011-10-31  6:06 ` [Qemu-devel] [PATCH 03/14] Add stub functions for PCI device models to do PCI DMA David Gibson
2011-11-02  7:17   ` Michael S. Tsirkin
2011-11-02 11:47     ` David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 04/14] rtl8139: Use PCI DMA stub functions David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 05/14] eepro100: " David Gibson
2011-10-31 16:45   ` Stefan Weil
2011-11-01 20:24     ` Anthony Liguori
2011-11-02  7:40       ` Michael S. Tsirkin
2011-11-02 12:46         ` Anthony Liguori
2011-11-02 17:56         ` Alexander Graf
2011-11-02 17:19           ` Anthony Liguori
2011-11-02 17:28             ` Alexander Graf
2011-11-02 19:11           ` Michael S. Tsirkin
2011-11-03  0:52             ` David Gibson
2011-11-02 11:01       ` Michael S. Tsirkin
2011-11-02  7:16   ` Michael S. Tsirkin
2011-11-03  5:16     ` David Gibson
2011-11-03 12:25       ` Michael S. Tsirkin
2011-11-04  0:28         ` David Gibson
2011-11-05  8:32         ` Stefan Weil
2011-10-31  6:06 ` [Qemu-devel] [PATCH 06/14] ac97: " David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 07/14] es1370: " David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 08/14] e1000: " David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 09/14] lsi53c895a: " David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 10/14] pcnet-pci: " David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 11/14] intel-hda: " David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 12/14] PCI IDE: " David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 13/14] usb-ehci: " David Gibson
2011-10-31  6:06 ` [Qemu-devel] [PATCH 14/14] usb-uhci: " David Gibson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1320041218-30487-3-git-send-email-david@gibson.dropbear.id.au \
    --to=david@gibson.dropbear.id.au \
    --cc=agraf@suse.de \
    --cc=anthony@codemonkey.ws \
    --cc=avi@redhat.com \
    --cc=eduard.munteanu@linux360.ro \
    --cc=joerg.roedel@amd.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).