Re: [Xen-devel] [PATCH v2] xen: arm32: reduce default size of the xenheap

2015-02-08 Thread Julien Grall
Hi Ian, On 05/02/2015 23:34, Ian Campbell wrote: On Thu, 2015-02-05 at 22:01 +0800, Julien Grall wrote: If the user requests a xenheap of 0MB, we will use the default size, right? It may be worth to explain this case. I think it's pretty generally understood that 0 generally means the default

Re: [Xen-devel] [PATCH v2] xen: arm32: reduce default size of the xenheap

2015-02-05 Thread Ian Campbell
On Thu, 2015-02-05 at 22:01 +0800, Julien Grall wrote: > If the user requests a xenheap of 0MB, we will use the default size, > right? It may be worth to explain this case. I think it's pretty generally understood that 0 generally means the default, unless otherwise specified. > Also with the al

Re: [Xen-devel] [PATCH v2] xen: arm32: reduce default size of the xenheap

2015-02-05 Thread Julien Grall
Hi Ian, On 05/02/2015 18:52, Ian Campbell wrote: ... and make it tunable via the command line. 1/8 of RAM is 128M on a 1GB system and 256M on a 2GB system etc, which is a lot. 1/32 of RAM seems more reasonable. Also drop the minimum to 32M. Leave the maximum at 1GB. Signed-off-by: Ian Campbel

Re: [Xen-devel] [PATCH v2] xen: arm32: reduce default size of the xenheap

2015-02-05 Thread Jan Beulich
>>> On 05.02.15 at 11:52, wrote: > ... and make it tunable via the command line. > > 1/8 of RAM is 128M on a 1GB system and 256M on a 2GB system etc, > which is a lot. 1/32 of RAM seems more reasonable. Also drop the > minimum to 32M. > > Leave the maximum at 1GB. > > Signed-off-by: Ian Campbel

[Xen-devel] [PATCH v2] xen: arm32: reduce default size of the xenheap

2015-02-05 Thread Ian Campbell
... and make it tunable via the command line. 1/8 of RAM is 128M on a 1GB system and 256M on a 2GB system etc, which is a lot. 1/32 of RAM seems more reasonable. Also drop the minimum to 32M. Leave the maximum at 1GB. Signed-off-by: Ian Campbell Cc: Jintack Lim Cc: Jan Beulich --- v2: - Use