xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
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

      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).