qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: malc <av1474@comtv.ru>
To: qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] Re: PATCH, RFC: Generic DMA framework
Date: Thu, 16 Aug 2007 23:58:59 +0400 (MSD)	[thread overview]
Message-ID: <Pine.LNX.4.64.0708162353140.3886@linmac.oyster.ru> (raw)
In-Reply-To: <f43fc5580708161118w531a5124s6e0bc2aa3e89e9bb@mail.gmail.com>

On Thu, 16 Aug 2007, Blue Swirl wrote:

> On 8/14/07, Blue Swirl <blauwirbel@gmail.com> wrote:
>> Would the framework need any changes to support other targets? Comments welcome.
>
> Replying to myself: Yes, changes may be needed. Some of the DMA
> controllers move the data outside CPU loop, but that does not make
> much difference.
>
> Background: I want to use the framework for at least devices that
> Sparc32/64 use. For Sparc32 the reason is that on Sun4c (Sparcstation
> 1, 2, IPX etc.) there is no IOMMU, but instead the CPU MMU is used for
> address translation. The DMA framework makes it possible to remove the
> IOMMU without changing the devices.
>
> On Sparc64 an IOMMU needs to be inserted between PCI devices and RAM
> without disturbing other targets.
>
> About the devices: Users of PC ISA DMA controller (SB16, FDC) pass the
> DMA position parameter to controller. I'm not sure this can be removed
> easily. Of course a real DMA controller does not get any position data
> from target. For Sparc32/64 I would not need to touch the PC ISA DMA
> devices, except maybe for FDC. On Sparc32, the FDC DMA is not even
> used. I have to think about this part.

Very long time ago i changed the ISA DMA API to address some of the
critique that Fabrice expressed, i can't remember offhand if that
included removal of explicit position passing or not (the patch is on
some off-line HDD so it's not easy to check whether it's in fact so)

http://www.mail-archive.com/qemu-devel@nongnu.org/msg06594.html

If needed i can try to locate the patch but the FDC problem still needs
to be addressed by someone.

> PCI DMA-like devices (eepro100, pcnet, rtl8139, ide) as well as PXA
> use cpu_physical_memory_rw to transfer data (eepro100 also uses
> ldl_phys, which looks very suspicious). These could be converted to
> generic DMA easily.
>
> OMAP DMA is strange, but fortunately I'm not interested in those devices.
>
>

-- 
vale

  reply	other threads:[~2007-08-16 20:00 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-14 19:48 [Qemu-devel] PATCH, RFC: Generic DMA framework Blue Swirl
2007-08-16 18:18 ` [Qemu-devel] " Blue Swirl
2007-08-16 19:58   ` malc [this message]
2007-08-19 17:46     ` Blue Swirl
2007-08-24 19:40 ` Blue Swirl
2007-08-24 20:18   ` Paul Brook
2007-08-24 23:33     ` Fabrice Bellard
2007-08-25  0:29       ` Paul Brook
2007-08-26 11:30         ` Fabrice Bellard
2007-08-26 17:54           ` Blue Swirl
2007-08-28 19:03             ` Blue Swirl
2007-08-28 19:43               ` Paul Brook
2007-08-29 17:00                 ` Blue Swirl
2007-08-29 20:39                   ` Paul Brook
2007-08-29 21:18                     ` Paul Brook
2007-09-08 14:07                       ` Blue Swirl
2007-09-08 14:31                         ` Paul Brook
2007-09-08 14:53                           ` Blue Swirl
2007-09-08 16:03                             ` Paul Brook
2007-09-15 16:16                               ` Blue Swirl

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=Pine.LNX.4.64.0708162353140.3886@linmac.oyster.ru \
    --to=av1474@comtv.ru \
    --cc=qemu-devel@nongnu.org \
    /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).