Re: [Xen-devel] [PATCH] AMD-Vi: allocate root table on demand

2017-03-20 Thread Jan Beulich
>>> On 20.03.17 at 12:49, wrote: >> @@ -648,6 +647,13 @@ int amd_iommu_map_page(struct domain *d, >> >> spin_lock(>arch.mapping_lock); >> >> +rc = amd_iommu_alloc_root(hd); >> +if ( rc ) >> +{ >> +spin_unlock(>arch.mapping_lock); >> +

Re: [Xen-devel] [PATCH] AMD-Vi: allocate root table on demand

2017-03-20 Thread Suravee Suthikulpanit
Hi Jan, Sorry for delay. Please seem my comment below. On 3/3/17 21:29, Jan Beulich wrote: This was my originally intended fix for the AMD side of XSA-207: There's no need to unconditionally allocate the root table, and with that there's then also no way to leak it when a guest has no devices

[Xen-devel] [PATCH] AMD-Vi: allocate root table on demand

2017-03-03 Thread Jan Beulich
This was my originally intended fix for the AMD side of XSA-207: There's no need to unconditionally allocate the root table, and with that there's then also no way to leak it when a guest has no devices assigned. Signed-off-by: Jan Beulich ---