From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v3] xen: arm32: reduce default size of the xenheap Date: Thu, 19 Feb 2015 18:57:35 +0000 Message-ID: <54E6321F.4000402@linaro.org> References: <1424367594-2880-1-git-send-email-ian.campbell@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1424367594-2880-1-git-send-email-ian.campbell@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell , xen-devel@lists.xen.org Cc: tim@xen.org, Jintack Lim , Jan Beulich , stefano.stabellini@eu.citrix.com List-Id: xen-devel@lists.xenproject.org Hi Ian, On 19/02/15 17:39, 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 Campbell Reviewed-by: Julien Grall > Cc: Jintack Lim > Cc: Jan Beulich > --- > v3: > - Clarify doc and comments > - Enforce new minimum of 32M, not old 128M > - Only allocate exact amount if user gives a value, rather than > searching for the next smallest power of two we can find. > v2: > - Use xenheap_megabytes as the option, which is what older x86 Xen > used. > > I'd like to backport at least the command line option to 4.5. Reducing > the default heap size is a bit border line but I'm inclined to take > it. I'm fine with the backport. Regards, -- Julien Grall