On 26/11/2019 08:42, Jan Beulich wrote: > On 25.11.2019 22:05, Igor Druzhinin wrote: >> --- a/xen/drivers/passthrough/amd/iommu_init.c >> +++ b/xen/drivers/passthrough/amd/iommu_init.c >> @@ -1279,7 +1279,7 @@ static int __init amd_iommu_setup_device_table( >> for ( bdf = 0, size /= sizeof(*dt); bdf < size; ++bdf ) >> dt[bdf] = (struct amd_iommu_dte){ >> .v = true, >> - .iv = true, >> + .iv = iommu_intremap, > This was very intentionally "true", and ignoring "iommu_intremap":
Deliberate or not, it is a regression from 4.12. Booting with iommu=no-intremap is a common debugging technique, and that means no interrupt remapping anywhere in the system, even for supposedly-unused DTEs. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel