linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Zach Pfeffer <zpfeffer@codeaurora.org>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>,
	FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
	linux-arch@vger.kernel.org, dwalker@codeaurora.org,
	mel@csn.ul.ie, linux-arm-msm@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	andi@firstfloor.org, linux-omap@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [RFC 1/3 v3] mm: iommu: An API to unify IOMMU, CPU and device memory management
Date: Thu, 22 Jul 2010 08:34:55 +0100	[thread overview]
Message-ID: <20100722073455.GB6802@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <20100722042528.GB22559@codeaurora.org>

On Wed, Jul 21, 2010 at 09:25:28PM -0700, Zach Pfeffer wrote:
> Yes it is a problem, as Russell has brought up, but there's something
> I probably haven't communicated well. I'll use the following example:
> 
> There are 3 devices: A CPU, a decoder and a video output device. All 3
> devices need to map the same 12 MB buffer at the same time.

Why do you need the same buffer mapped by the CPU?

Let's take your example of a video decoder and video output device.
Surely the CPU doesn't want to be writing to the same memory region
used for the output picture as the decoder is writing to.  So what's
the point of mapping that memory into the CPU's address space?

Surely the video output device doesn't need to see the input data to
the decoder either?

Surely, all you need is:

1. a mapping for the CPU for a chunk of memory to pass data to the
   decoder.
2. a mapping for the decoder to see the chunk of memory to receive data
   from the CPU.
3. a mapping for the decoder to see a chunk of memory used for the output
   video buffer.
4. a mapping for the output device to see the video buffer.

So I don't see why everything needs to be mapped by everything else.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2010-07-22  7:36 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-03  5:38 [RFC 1/3 v3] mm: iommu: An API to unify IOMMU, CPU and device memory management Zach Pfeffer
2010-07-03 19:06 ` Eric W. Biederman
2010-07-07 22:44   ` Zach Pfeffer
2010-07-07 23:07     ` Russell King - ARM Linux
2010-07-08 23:59       ` Zach Pfeffer
2010-07-12  1:25         ` FUJITA Tomonori
2010-07-13  5:57           ` Zach Pfeffer
2010-07-13  6:03             ` FUJITA Tomonori
2010-07-13 12:14               ` Zach Pfeffer
2010-07-14  1:59                 ` FUJITA Tomonori
2010-07-14 20:11                   ` Zach Pfeffer
2010-07-14 22:05                     ` Russell King - ARM Linux
2010-07-15  1:29                       ` Zach Pfeffer
2010-07-15  1:47                         ` Eric W. Biederman
2010-07-15  5:40                           ` Zach Pfeffer
2010-07-15  5:35                         ` Zach Pfeffer
2010-07-15  8:55                         ` Russell King - ARM Linux
2010-07-16  0:48                           ` Tim HRM
2010-07-16  7:58                             ` Russell King - ARM Linux
2010-07-17  0:01                               ` Larry Bassel
2010-07-19  9:21                                 ` Tim HRM
2010-07-21  0:44                                   ` Zach Pfeffer
2010-07-21  1:44                                     ` Timothy Meade
2010-07-22  4:06                                       ` Zach Pfeffer
2010-07-19 17:55                               ` Michael Bohan
2010-07-19 18:40                                 ` Russell King - ARM Linux
2010-07-20 22:02                                   ` stepanm
2010-07-20 22:29                                     ` Russell King - ARM Linux
2010-07-21  5:49                                       ` Shilimkar, Santosh
2010-07-21  7:28                                         ` Russell King - ARM Linux
2010-07-21  7:45                                           ` Shilimkar, Santosh
2010-07-21 18:04                                           ` stepanm
2010-07-20 20:45                               ` Zach Pfeffer
2010-07-20 20:54                                 ` Russell King - ARM Linux
2010-07-20 21:56                                   ` Zach Pfeffer
2010-07-19  6:52                           ` Zach Pfeffer
2010-07-19  7:44                             ` Eric W. Biederman
2010-07-22  4:25                               ` Zach Pfeffer
2010-07-22  7:34                                 ` Russell King - ARM Linux [this message]
2010-07-22 16:25                                   ` Zach Pfeffer
2010-07-14 23:07                     ` FUJITA Tomonori
2010-07-15  1:41                       ` Zach Pfeffer
2010-07-19  8:22                         ` Russell King - ARM Linux
2010-07-20 10:09                           ` FUJITA Tomonori
2010-07-20 22:20                           ` Zach Pfeffer
2010-07-21  1:44                             ` FUJITA Tomonori
2010-07-22  4:30                               ` Zach Pfeffer
2010-07-22  4:43                                 ` FUJITA Tomonori
2010-07-22 16:44                                   ` Zach Pfeffer
2010-07-22  7:39                                 ` Russell King - ARM Linux
2010-07-22 16:28                                   ` Zach Pfeffer
  -- strict thread matches above, loose matches on Subject: below --
2010-07-06 15:42 Zach Pfeffer
2010-07-21  5:18 stepanm

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=20100722073455.GB6802@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=andi@firstfloor.org \
    --cc=dwalker@codeaurora.org \
    --cc=ebiederm@xmission.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=mel@csn.ul.ie \
    --cc=zpfeffer@codeaurora.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).