WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH 1/2][VTD] pci mmconfig support to be used for ATS

To: "Allen M Kay" <allen.m.kay@xxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH 1/2][VTD] pci mmconfig support to be used for ATS
From: "Jan Beulich" <jbeulich@xxxxxxxxxx>
Date: Thu, 20 Nov 2008 08:56:26 +0000
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>, Weidong Han <weidong.han@xxxxxxxxx>, Dexuan Cui <dexuan.cui@xxxxxxxxx>
Delivery-date: Thu, 20 Nov 2008 04:51:06 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <57C9024A16AD2D4C97DC78E552063EA35C4BC7A4@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <57C9024A16AD2D4C97DC78E552063EA35C4BC7A4@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
>>> "Kay, Allen M" <allen.m.kay@xxxxxxxxx> 20.11.08 02:42 >>>
>ATS (Address Translation Service) is a PCI specification used for 
>synchronizing iotlb translations between chipset iommu such as vt-d with iotlb 
>on board a PCI device.
>
>This patch enables PCI mmconfig for Intel64 systems.  Most of the code were 
>copied from Linux.  This functionality is need for parsing ATS capability in 
>PCIe extended configuration space.

If this is x86-64 only anyway, why do you need to use fixmap space for this 
(and limit things arbitrarily to 4 segments) rather than using a 256Mb block 
from one of the unused virtual address regions in hypervisor space?

Also, using obj-$(CONFIG_X86_64) in the Makefile changes would avoid building 
dead code for x86-32...

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel