From: Tim Deegan <tim@xen.org>
To: Santosh Jodh <Santosh.Jodh@citrix.com>
Cc: "xiantao.zhang@intel.com" <xiantao.zhang@intel.com>,
"Nakajima, Jun" <jun.nakajima@intel.com>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: Superpages for VT-D
Date: Fri, 3 Aug 2012 12:40:51 +0100 [thread overview]
Message-ID: <20120803114051.GC25286@ocelot.phlegethon.org> (raw)
In-Reply-To: <7914B38A4445B34AA16EB9F1352942F1012F0DE665E4@SJCPMAILBOX01.citrite.net>
At 08:56 -0700 on 02 Aug (1343897810), Santosh Jodh wrote:
> Thanks for confirming. BTW, would the IOMMU ever have entries above
> domains max mapped pfn?
I don't believe so, but potentially some grant-table-style operations
might want to map pages in for DMA that don't need to be mapped in for
CPU access.
But if you're writing a dump routine from scratch it should be easy
enough (and more efficient) to dump all the entries with a depth-first
pass over the trie, rather than individually querying all frames up to
max_mapped_pfn.
Tim.
> -----Original Message-----
> From: Tim Deegan [mailto:tim@xen.org]
> Sent: Thursday, August 02, 2012 4:11 AM
> To: Santosh Jodh
> Cc: xiantao.zhang@intel.com; Nakajima, Jun; xen-devel@lists.xen.org
> Subject: Re: [Xen-devel] Superpages for VT-D
>
> At 13:44 -0700 on 31 Jul (1343742260), Santosh Jodh wrote:
> > I am going to try to add this support.
> >
> > It looks like a new iommu_ops handler would be needed that would do
> > the actual work of dumping the entries - one for AMD and one for
> > Intel. Am I reading this correctly?
>
> I think that's correct.
>
> > Or is it better to get the root_table + paging_mode (for AMD) and
> > pgd_maddr + agaw (for Intel) and then do a generic dump?
>
> No; the iommu tabels are sufficiebntly arch-specific that it's best to add arch-specific dump routines.
>
> Cheers,
>
> Tim.
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
prev parent reply other threads:[~2012-08-03 11:40 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-13 16:18 Superpages for VT-D Santosh Jodh
2012-07-19 10:23 ` Tim Deegan
2012-07-19 14:01 ` Santosh Jodh
2012-07-19 14:53 ` Tim Deegan
2012-07-20 21:38 ` Nakajima, Jun
2012-07-23 17:56 ` Santosh Jodh
2012-07-24 19:41 ` Tim Deegan
2012-07-31 20:44 ` Santosh Jodh
2012-08-02 11:11 ` Tim Deegan
2012-08-02 15:56 ` Santosh Jodh
2012-08-03 11:40 ` Tim Deegan [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=20120803114051.GC25286@ocelot.phlegethon.org \
--to=tim@xen.org \
--cc=Santosh.Jodh@citrix.com \
--cc=jun.nakajima@intel.com \
--cc=xen-devel@lists.xen.org \
--cc=xiantao.zhang@intel.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 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).