public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Joerg Roedel <joro@8bytes.org>
To: "Duran, Leo" <leo.duran@amd.com>
Cc: Muli Ben-Yehuda <muli@il.ibm.com>, Adrian Bunk <bunk@kernel.org>,
	"Richter, Robert" <rrichter@elbe.amd.com>,
	linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org,
	Andi Kleen <andi@firstfloor.org>,
	"Biemueller, Sebastian" <Sebastian.Biemueller@amd.com>,
	tglx@linutronix.de, mingo@redhat.com, "Sarathy,
	Bhavna" <Bhavna.Sarathy@amd.com>
Subject: Re: [PATCH 01/34] AMD IOMMU: add Kconfig entry
Date: Sat, 28 Jun 2008 18:27:48 +0200	[thread overview]
Message-ID: <20080628162748.GA31029@8bytes.org> (raw)
In-Reply-To: <6453C3CB8E2B3646B0D020C1126132730120C518@sausexmb4.amd.com>

On Sat, Jun 28, 2008 at 09:40:57AM -0500, Duran, Leo wrote:
> On Saturday, June 28, 2008 6:04 AM, Joerg Roedel wrote:
> 
> > Yes, there is only one exclusion range per IOMMU. The problem is that
> > an
> > exclusion range from 64MB to the end may not be possible because there
> > is
> > an exclusion range already configured in the ACPI table. On my System
> > for
> > example the exclusion range is somewhere in the first megabyte of RAM.
> > In this case the direct mapping using page tables is needed. If there
> > is
> > no exclusion range defined in ACPI this idea would work of course.
> > 
> > Joerg
> > 
> 
> Direct 1:1 mappings have a couple of issues:
> 1) They don't provide a performance optimization (i.e., table-walk still
> required)

True. But the table walk is nothing which could be optimized in
software. The optimization here is that we don't need to map/unmap on
each dma_ops call. Mapping/Unmapping requires TLB flushing and
completion_wait. This is expensive for the CPU. But direct mapping
exclusion ranges inside the aperture and mark the area from 64mb to the
end as the exclusion range as you suggested could be a real option.

> 2) Your aperture would have to be large enough so that virtual==physical
> (i.e., lots of memory for page-tables)

The AMD IOMMU supports multiple page sizes from 4kb up to 4GB. I don't
think that we need too much memory for the page tables.

Joerg


  reply	other threads:[~2008-06-28 16:28 UTC|newest]

Thread overview: 124+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-26 19:27 [PATCH 00/34] AMD IOMMU driver Joerg Roedel
2008-06-26 19:27 ` [PATCH 01/34] AMD IOMMU: add Kconfig entry Joerg Roedel
2008-06-27 14:25   ` Adrian Bunk
2008-06-27 14:47     ` Andi Kleen
2008-06-27 16:39       ` Muli Ben-Yehuda
2008-06-27 16:54         ` Joerg Roedel
2008-06-27 16:59           ` Muli Ben-Yehuda
2008-06-27 17:05             ` Joerg Roedel
2008-06-27 17:12               ` Muli Ben-Yehuda
2008-06-27 17:20                 ` Joerg Roedel
2008-06-27 17:31                   ` Muli Ben-Yehuda
2008-06-27 17:40                     ` Joerg Roedel
2008-06-27 17:44                       ` Muli Ben-Yehuda
2008-06-27 17:52                         ` Joerg Roedel
2008-07-02  5:45                       ` Amit Shah
2008-07-02  8:12                         ` Alan Cox
2008-06-27 20:39               ` Duran, Leo
2008-06-27 22:29                 ` Muli Ben-Yehuda
2008-06-27 22:47                   ` Duran, Leo
2008-06-28 11:04                     ` Joerg Roedel
2008-06-28 14:40                       ` Duran, Leo
2008-06-28 16:27                         ` Joerg Roedel [this message]
2008-06-28 14:58                       ` Duran, Leo
2008-06-27 18:54           ` Andi Kleen
2008-06-28 10:52             ` Joerg Roedel
2008-06-27 16:40     ` Joerg Roedel
2008-07-08 12:45   ` Pavel Machek
2008-07-09  8:48     ` Ingo Molnar
2008-07-10  0:50       ` FUJITA Tomonori
2008-07-10  8:44         ` Ingo Molnar
2008-06-26 19:27 ` [PATCH 02/34] AMD IOMMU: add header file for driver data structures and defines Joerg Roedel
2008-06-29 15:07   ` FUJITA Tomonori
2008-06-29 15:14     ` Joerg Roedel
2008-06-29 23:11       ` FUJITA Tomonori
2008-06-30 12:22         ` Ingo Molnar
2008-07-10  1:38   ` Andrew Morton
2008-07-10  1:50     ` Arjan van de Ven
2008-07-10  2:36       ` Andrew Morton
2008-07-10 12:12         ` Joerg Roedel
2008-07-10 12:59           ` Andrew Morton
2008-06-26 19:27 ` [PATCH 03/34] AMD IOMMU: add defines and structures for ACPI scanning code Joerg Roedel
2008-07-10  1:41   ` Andrew Morton
2008-07-10 12:17     ` Joerg Roedel
2008-06-26 19:27 ` [PATCH 04/34] AMD IOMMU: add data structures to manage the IOMMUs in the system Joerg Roedel
2008-07-10  1:43   ` Andrew Morton
2008-07-10 12:25     ` Joerg Roedel
2008-06-26 19:27 ` [PATCH 05/34] AMD IOMMU: add functions to find last possible PCI device for IOMMU Joerg Roedel
2008-06-26 19:27 ` [PATCH 06/34] AMD IOMMU: add amd_iommu_init.c to Makefile Joerg Roedel
2008-06-26 19:27 ` [PATCH 07/34] AMD IOMMU: add functions for mapping/unmapping the MMIO space Joerg Roedel
2008-06-26 19:27 ` [PATCH 08/34] AMD IOMMU: add functions for programming IOMMU " Joerg Roedel
2008-06-26 19:27 ` [PATCH 09/34] AMD IOMMU: add command buffer (de-)allocation Joerg Roedel
2008-07-10  1:47   ` Andrew Morton
2008-06-26 19:27 ` [PATCH 10/34] AMD IOMMU: add device table initialization functions Joerg Roedel
2008-06-26 19:27 ` [PATCH 11/34] AMD IOMMU: add functions for IOMMU hardware initialization from ACPI Joerg Roedel
2008-07-10  1:49   ` Andrew Morton
2008-06-26 19:27 ` [PATCH 12/34] AMD IOMMU: add detect code for AMD IOMMU hardware Joerg Roedel
2008-07-10  1:51   ` Andrew Morton
2008-06-26 19:27 ` [PATCH 13/34] AMD IOMMU: add functions to parse IOMMU memory mapping requirements for devices Joerg Roedel
2008-07-10  1:51   ` Andrew Morton
2008-06-26 19:27 ` [PATCH 14/34] AMD IOMMU: clue initialization code together Joerg Roedel
2008-07-10  1:55   ` Andrew Morton
2008-07-10 12:37     ` Joerg Roedel
2008-07-10 13:03       ` Andrew Morton
2008-07-10 13:31         ` Joerg Roedel
2008-07-10 18:37           ` Joerg Roedel
2008-06-26 19:27 ` [PATCH 15/34] AMD IOMMU: add early detection code Joerg Roedel
2008-06-26 19:27 ` [PATCH 16/34] AMD IOMMU: add kernel command line parameters for AMD IOMMU Joerg Roedel
2008-07-10  1:56   ` Andrew Morton
2008-07-10  4:25     ` FUJITA Tomonori
2008-07-10  4:39       ` Andrew Morton
2008-07-10  6:26         ` Ingo Molnar
2008-07-10 20:42           ` Randy Dunlap
2008-07-14 23:56           ` FUJITA Tomonori
2008-07-10  7:04   ` Yinghai Lu
2008-07-10 12:41     ` Joerg Roedel
2008-06-26 19:27 ` [PATCH 17/34] AMD IOMMU: add generic defines and structures for mapping code Joerg Roedel
2008-07-10  2:01   ` Andrew Morton
2008-07-10  2:38     ` Andrew Morton
2008-07-10  4:25     ` FUJITA Tomonori
2008-07-10 12:44       ` Joerg Roedel
2008-06-26 19:27 ` [PATCH 18/34] AMD IOMMU: add amd_iommu.c to Makefile Joerg Roedel
2008-06-26 19:27 ` [PATCH 19/34] AMD IOMMU: add functions to send IOMMU commands Joerg Roedel
2008-07-10  2:04   ` Andrew Morton
2008-07-10 12:53     ` Joerg Roedel
2008-06-26 19:27 ` [PATCH 20/34] AMD IOMMU: add functions to initialize unity mappings Joerg Roedel
2008-06-26 19:27 ` [PATCH 21/34] AMD IOMMU: add address allocation and deallocation functions Joerg Roedel
2008-06-29 15:07   ` FUJITA Tomonori
2008-06-29 15:17     ` Joerg Roedel
2008-06-29 23:11       ` FUJITA Tomonori
2008-06-26 19:27 ` [PATCH 22/34] AMD IOMMU: add domain " Joerg Roedel
2008-07-10  2:14   ` Andrew Morton
2008-07-10 12:54     ` Joerg Roedel
2008-06-26 19:27 ` [PATCH 23/34] AMD IOMMU: add functions to find IOMMU device resources Joerg Roedel
2008-07-10  2:18   ` Andrew Morton
2008-07-10 16:46     ` Joerg Roedel
2008-07-10 23:59       ` Andrew Morton
2008-06-26 19:28 ` [PATCH 24/34] AMD IOMMU: add generic dma_ops mapping functions Joerg Roedel
2008-06-26 19:28 ` [PATCH 25/34] AMD IOMMU: add dma_ops mapping functions for single mappings Joerg Roedel
2008-07-10  2:26   ` Andrew Morton
2008-07-10 13:20     ` Joerg Roedel
2008-06-26 19:28 ` [PATCH 26/34] AMD IOMMU: add mapping functions for scatter gather lists Joerg Roedel
2008-06-29 15:07   ` FUJITA Tomonori
2008-06-30 13:25     ` Joerg Roedel
2008-06-26 19:28 ` [PATCH 27/34] AMD IOMMU: add mapping functions for coherent mappings Joerg Roedel
2008-06-26 19:28 ` [PATCH 28/34] AMD IOMMU: add pre-allocation of protection domains Joerg Roedel
2008-06-26 19:28 ` [PATCH 29/34] AMD IOMMU: add dma_ops initialization function Joerg Roedel
2008-06-26 19:28 ` [PATCH 30/34] AMD IOMMU: add amd_iommu.h to export functions to the generic x86 dma code Joerg Roedel
2008-06-26 19:28 ` [PATCH 31/34] AMD IOMMU: initialize dma_ops from IOMMU initialization and enable IOMMUs Joerg Roedel
2008-06-26 19:28 ` [PATCH 32/34] AMD_IOMMU: call detect and initialization functions from dma code Joerg Roedel
2008-06-26 19:28 ` [PATCH 33/34] AMD IOMMU: add MAINTAINERS entry Joerg Roedel
2008-06-26 19:28 ` [PATCH 34/34] AMD IOMMU: add documentation for kernel parameters Joerg Roedel
2008-06-29 15:07   ` FUJITA Tomonori
2008-06-30 12:25     ` Ingo Molnar
2008-06-26 20:37 ` [PATCH 00/34] AMD IOMMU driver Rafael J. Wysocki
2008-06-26 20:37   ` Joerg Roedel
2008-06-26 21:02     ` Rafael J. Wysocki
2008-06-27  8:18 ` Ingo Molnar
2008-06-27 10:07   ` Ingo Molnar
2008-06-27 10:15     ` Joerg Roedel
2008-06-27 10:59   ` Joerg Roedel
2008-07-11 10:22 ` Eric W. Biederman
2008-07-11 14:11   ` Joerg Roedel
2008-07-11 16:23   ` Duran, Leo
2008-07-11 17:20     ` Eric W. Biederman

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=20080628162748.GA31029@8bytes.org \
    --to=joro@8bytes.org \
    --cc=Bhavna.Sarathy@amd.com \
    --cc=Sebastian.Biemueller@amd.com \
    --cc=andi@firstfloor.org \
    --cc=bunk@kernel.org \
    --cc=iommu@lists.linux-foundation.org \
    --cc=leo.duran@amd.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=muli@il.ibm.com \
    --cc=rrichter@elbe.amd.com \
    --cc=tglx@linutronix.de \
    /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