On Tue, Sep 27, 2016 at 06:11:59AM -0400, Paolo Bonzini wrote: > > > I think we should keep it as it is, because Jason's patchset will only > > support intel-iommu, not amd-iommu. For now, it won't have problem > > (just like Intel IOMMU one). But after Jason's patch is merged, people > > will be able to boot a guest with vhost and amd-iommu (which we > > actually do not support yet), and that might be problematic. > > Let's fix Jason's patch instead. :)
Then I think Jason will need to add vhost DMAR support for AMD as well, which I believe is not in his current plan. :) Anyway, I think we can do it either way. I can hardly believe if someone will like to run vhost with AMD IOMMU. So IMHO it's not a big thing for all cases. Thanks, -- peterx
