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

["Jan Beulich" <jbeulich@xxxxxxxxxx>]

>>> "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