[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 00 of 14 V3] amd iommu: support ATS device passthru on IOMMUv2 systems

Am 11.01.2012 16:04, schrieb Jan Beulich:
On 10.01.12 at 18:07, Wei Wang<wei.wang2@xxxxxxx>  wrote:
Hi all, this is patch v3.
ATS devices with PRI and PASID capabilities can communicate with iommuv2 to
perform two level (nested) address translation and demand paging for DMA.
To passthru such devices, iommu driver has to been enabled in guest OS.
This patch set adds initial iommu emulation for hvm guests to support ATS
device passthru.
I would look into committing 1-6 and 10 (if that one is independent of
7-9), if you can confirm that those on their own provide meaningful
benefit (enabling the ppr log probably is what I'm after, but I'd still
like your confirmation - patch 3 in particular doesn't look very useful
without the later ones). So ideally the ones leading up to the ppr log
enabling would all be first (or even a separate series), and the guest
iommu ones would follow (as those make only sense when the tools
maintainers are okay with the changes too)
Hi Jan,
Thanks for doing this. It sounds great! Even without guest iommu
being enabled, patch 1-6 and 10 would still be useful for turn on ppr and
GT features. Patch 6 will call some functions from patch 3. If you want to
leave patch 3 behind, I could send a new version to move patch 6 ahead.


changes in v3:
* Use xenstore to receive guest iommu configuration instead of adding in a
new field in hvm_info_table.
* Support pci segment in vbdf to mbdf bind.
* Make hypercalls visible for non-x86 platforms.
* A few code cleanups according to comments from Jan and Ian.

Changes in v2:
* Do not use linked list to access guest iommu tables.
* Do not parse iommu parameter in libxl_device_model_info again.
* Fix incorrect logical calculation in patch 11.
* Fix hypercall definition for non-x86 systems.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.