[Xen-devel] Re: [PATCH][VTD] enabling PCI ACS P2P upstream forwarding

["Jan Beulich" <JBeulich@xxxxxxxxxx>]

I'm pretty certain a call to pci_enable_acs() would also be needed from
pci_add_device_ext()...

Jan

PS: Generating patches with -p makes it easier for reviewers to identfiy
where hunks belong.

>>> "Kay, Allen M" <allen.m.kay@xxxxxxxxx> 17.11.09 22:49 >>>
This patch enables P2P upstream forwarding in ACS capable PCIe switches.  The 
enabling is conditioned on iommu_enabled variable.  This code solves two 
potential problems in virtualization environment where a PCIe device is 
assigned to a guest domain using a HW iommu such as VT-d:

1) Unintentional failure caused by guest physical address programmed into the 
device's DMA that happens to match the memory address range of other downstream 
ports in the same PCIe switch.  This causes the PCI transaction to go to the 
matching downstream port instead of go to the root complex to get translated by 
VT-d as it should be.

2) Malicious guest software intentionally attacks another downstream PCIe 
device by programming the DMA address into the assigned device that matches 
memory address range of the downstream PCIe port.

Corresponding ACS filtering code is already in upstream control panel code that 
do not allow PCI device passthrough to guests if it is behind a PCIe switch 
that does not have ACS capability or with ACS capability but is not enabled. 

Signed-off-by: Allen Kay allen.m.kay@xxxxxxxxx


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