>>> On 27.09.17 at 16:16, <roger....@citrix.com> wrote:
> --- a/xen/arch/x86/dom0_build.c
> +++ b/xen/arch/x86/dom0_build.c
> @@ -263,8 +263,7 @@ unsigned long __init dom0_compute_nr_pages(
>              avail -= max_pdx >> s;
>      }
>  
> -    need_paging = is_hvm_domain(d) &&
> -        (!iommu_hap_pt_share || !paging_mode_hap(d));
> +    need_paging = is_hvm_domain(d);
>      for ( ; ; need_paging = false )
>      {
>          nr_pages = dom0_nrpages;

Still in context above is the calculation for IOMMU page tables
When iommu_hap_pt_share, why do we need to reserve extra
space? If the IOMMU calculation isn't precise enough, perhaps
that's what wants changing? Quite possibly it would suffice to
simply double the value dom0_paging_pages() returns in that
case.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Reply via email to