From: Joanna Rutkowska <joanna@invisiblethingslab.com>
To: Keir Fraser <keir.fraser@eu.citrix.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
Rafal Wojtczuk <rafal@invisiblethingslab.com>
Subject: Re: Does xc_map_foreign_pages() update IOMMU tables?
Date: Thu, 13 May 2010 14:39:53 +0200 [thread overview]
Message-ID: <4BEBF319.3020202@invisiblethingslab.com> (raw)
In-Reply-To: <C811AEC8.14295%keir.fraser@eu.citrix.com>
[-- Attachment #1.1: Type: text/plain, Size: 1006 bytes --]
On 05/13/2010 02:27 PM, Keir Fraser wrote:
> On 13/05/2010 13:09, "Joanna Rutkowska" <joanna@invisiblethingslab.com>
> wrote:
>
>>> Seeing as how xc_map_foreign_pages() maps pages into a particular
>>> guest-process virtual address space, whereas IOMMU tables are a mapping from
>>> guest-physical address space, it's clear that x_m_f_p() cannot sensibly be
>>> having any effect on the IOMMU.
>>>
>> Interesting. That suggests the graphics drivers we tested do not use DMA
>> for XShmPutImage()...
>
> Ah, actually dom0 is special in that it gets a 1:1 mapping of all of RAM
> (except for some private Xen areas). See
> drivers/passthrough/vtd/x86/vtd.c:iommu_set_dom0_mapping(). I forgot about
> that when I wrote my previous reply!
>
Ah, indeed! In fact Rafal and I even had this very fragment of code in
our slides 2 years ago and we both forgot about it :) [Back then it was
in drivers/passthrough/vtd/iommu.c:intel_iommu_domain_init().
Thanks for clarifying!
joanna.
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 226 bytes --]
[-- Attachment #2: Type: text/plain, Size: 138 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
prev parent reply other threads:[~2010-05-13 12:39 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-13 11:48 Does xc_map_foreign_pages() update IOMMU tables? Joanna Rutkowska
2010-05-13 12:00 ` Keir Fraser
2010-05-13 12:09 ` Joanna Rutkowska
2010-05-13 12:27 ` Keir Fraser
2010-05-13 12:39 ` Joanna Rutkowska [this message]
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=4BEBF319.3020202@invisiblethingslab.com \
--to=joanna@invisiblethingslab.com \
--cc=keir.fraser@eu.citrix.com \
--cc=rafal@invisiblethingslab.com \
--cc=xen-devel@lists.xensource.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.