On 28/11/14 10:29, Li, Liang Z wrote: >>> - if (!hvm_pse1gb_supported(d)) >>> + if (!hvm_pse1gb_supported(d) || paging_mode_shadow(d)) >>> *edx &= ~cpufeat_mask(X86_FEATURE_PAGE1GB); >> With >> >> #define hvm_pse1gb_supported(d) \ >> (cpu_has_page1gb && paging_mode_hap(d)) >> the change above is pointless. While, considering this, comments on >> v2 may have been misleading, you should have simply updated the patch >> description instead to clarify why the v2 change was okay even for the >> shadow mode case. > I checked the code and found that for a normal guest can only be in hap mode > or shadow mode before I sending the patch, but I am not share if it's true > for dom0. > > Liang
Dom0 may either be PV (in which case neither hap nor shadow, and cant use 1GB pages anyway), or experimentally PVH which is currently restricted to hap. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel