[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH] xen/arm: increase default dom0_mem to 512M
Hi Stefano, On 15/02/2017 23:05, Stefano Stabellini wrote:
The default dom0_mem is 128M which is not sufficient to boot a Ubuntu based Dom0. Increase it to 512M. Signed-off-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
I am not a big fan of increasing the default value. 128M is plenty enough if you use a small DOM0 (e.g buildroot or yocto) and people may rely on it because it is the default value in the documentation
(see docs/misc/xen-command-line.markdown).Also, 512M may boot Ubuntu for you but it might not be the case in all the configuration. There is no perfect default value, but I think the smaller is better. Looking at the documentation, it looks like x86 is using 128MB or 1/16 of the memory (whichever is smaller).
But to be fair, I am not even sure why there is a default value, it is quite easy to specify the amount of memory used by DOM0 on the command line.
diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c index c97a1f5..f4612a2 100644 --- a/xen/arch/arm/domain_build.c +++ b/xen/arch/arm/domain_build.c @@ -31,7 +31,7 @@ integer_param("dom0_max_vcpus", opt_dom0_max_vcpus); int dom0_11_mapping = 1; -#define DOM0_MEM_DEFAULT 0x8000000 /* 128 MiB */ +#define DOM0_MEM_DEFAULT 0x20000000 /* 512 MiB */
I would use the MB(..) macro here to make the code more readable.
static u64 __initdata dom0_mem = DOM0_MEM_DEFAULT; static void __init parse_dom0_mem(const char *s)
Cheers, -- Julien Grall _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
Lists.xenproject.org is hosted with RackSpace, monitoring our