[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen staging] pvh/dom0: warn when dom0_mem is not set
commit fbaa0e58f39ae23263ad0935d88a7e61e0dd9ef1 Author: Roger Pau Monné <roger.pau@xxxxxxxxxx> AuthorDate: Mon Feb 18 13:44:53 2019 +0100 Commit: Jan Beulich <jbeulich@xxxxxxxx> CommitDate: Mon Feb 18 13:44:53 2019 +0100 pvh/dom0: warn when dom0_mem is not set There have been several reports of the dom0 builder running out of memory when building a PVH dom0 without having specified a dom0_mem value. Print a warning message if dom0_mem is not set when booting in PVH mode. This is a temporary workaround until accounting for internal memory required by Xen (ie: paging structures) is improved. Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx> Acked-by: Jan Beulich <jbeulich@xxxxxxxx> Reviewed-by: Wei Liu <wei.liu2@xxxxxxxxxx> Release-acked-by: Juergen Gross <jgross@xxxxxxxx> --- xen/arch/x86/dom0_build.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/xen/arch/x86/dom0_build.c b/xen/arch/x86/dom0_build.c index 2b4d9e9ea6..6ebe36766b 100644 --- a/xen/arch/x86/dom0_build.c +++ b/xen/arch/x86/dom0_build.c @@ -378,8 +378,18 @@ unsigned long __init dom0_compute_nr_pages( * maximum of 128MB. */ if ( !nr_pages ) + { nr_pages = avail - (pv_shim ? pv_shim_mem(avail) : min(avail / 16, 128UL << (20 - PAGE_SHIFT))); + if ( is_hvm_domain(d) && !need_paging ) + /* + * Temporary workaround message until internal (paging) memory + * accounting required to build a pvh dom0 is improved. + */ + printk("WARNING: PVH dom0 without dom0_mem set is still unstable. " + "If you get crashes during boot, try adding a dom0_mem parameter\n"); + } + /* Clamp according to min/max limits and available memory. */ nr_pages = max(nr_pages, min_pages); -- generated by git-patchbot for /home/xen/git/xen.git#staging _______________________________________________ Xen-changelog mailing list Xen-changelog@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/xen-changelog
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |