From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tim Deegan <tim@xen.org>
Subject: Re: Superpages for VT-D
Date: Thu, 2 Aug 2012 12:11:02 +0100
Message-ID: <20120802111102.GE11437@ocelot.phlegethon.org>
References: <7914B38A4445B34AA16EB9F1352942F1012F0CDD8A71@SJCPMAILBOX01.citrite.net>
	<20120719102326.GC75169@ocelot.phlegethon.org>
	<7914B38A4445B34AA16EB9F1352942F1012F0CDD9588@SJCPMAILBOX01.citrite.net>
	<20120719145318.GA78502@ocelot.phlegethon.org>
	<CAL54oT3yqLtDpEHe7n20K=i-hz27P-TYK11tm+VNL2BXMs+hMg@mail.gmail.com>
	<7914B38A4445B34AA16EB9F1352942F1012F0D841328@SJCPMAILBOX01.citrite.net>
	<20120724194150.GA68065@ocelot.phlegethon.org>
	<7914B38A4445B34AA16EB9F1352942F1012F0DE66157@SJCPMAILBOX01.citrite.net>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Content-Disposition: inline
In-Reply-To: <7914B38A4445B34AA16EB9F1352942F1012F0DE66157@SJCPMAILBOX01.citrite.net>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Santosh Jodh <Santosh.Jodh@citrix.com>
Cc: "Nakajima, Jun" <jun.nakajima@intel.com>, "xiantao.zhang@intel.com" <xiantao.zhang@intel.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

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.