From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v2] tools: implement initial ramdisk support for ARM. Date: Wed, 09 Apr 2014 09:17:37 +0100 Message-ID: <53450221.4060408@linaro.org> References: <1396618125-1623-1-git-send-email-ian.campbell@citrix.com> <21310.51440.15560.997315@mariner.uk.xensource.com> <1396971362.22845.266.camel@kazak.uk.xensource.com> <21316.10617.170497.479163@mariner.uk.xensource.com> <534431E3.7080207@linaro.org> <53443DED.8060901@linaro.org> <1397031125.22845.280.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1397031125.22845.280.camel@kazak.uk.xensource.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 Cc: tim@xen.org, stefano.stabellini@eu.citrix.com, Ian Jackson , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On 09/04/14 09:12, Ian Campbell wrote: >> This small changes fix boot of the guest with RAM < 128Mb: >> >> diff --git a/tools/libxc/xc_dom_arm.c b/tools/libxc/xc_dom_arm.c >> index f051515..2228ba5 100644 >> --- a/tools/libxc/xc_dom_arm.c >> +++ b/tools/libxc/xc_dom_arm.c >> @@ -300,7 +300,7 @@ int arch_setup_meminit(struct xc_dom_image *dom) >> if ( ramend >= ram128mb + modsize && kernend < ram128mb ) >> modbase = ram128mb; >> else if ( ramend >= kernend + modsize ) >> - modbase = kernend; >> + modbase = ramend - modsize; >> else >> return -1; >> >> I guess this is because the kernel is extracting on it. I think we >> should follow the same "algorithm" as Xen (see place_modules) to decide >> where the modules should be loaded. > > Yes, this fix is correct. The existing code is just bogus, it makes no > sense to place the modules exactly at the end of RAM since they will > spin over the end. I'm not sure to understand why you are talking about RAM... currently the modules are placed just after the kernel (which in this case is always before ramend). > Can you resubmit with a changelog and an S-o-b please. Sure. Regards, -- Julien Grall