From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH] xen: arm: configure correct dom0_gnttab_start/size Date: Tue, 04 Nov 2014 12:46:52 +0000 Message-ID: <5458CABC.9010607@linaro.org> References: <1410448889-18731-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: <1410448889-18731-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, stefano.stabellini@eu.citrix.com List-Id: xen-devel@lists.xenproject.org On 09/11/2014 04:21 PM, Ian Campbell wrote: > Vexpress is currently failing to boot for me with: > ------------[ cut here ]------------ > WARNING: CPU: 0 PID: 1 at arch/arm/mm/ioremap.c:301 __arm_ioremap_pfn_caller+0x118/0x1a4() > CPU: 0 PID: 1 Comm: swapper Tainted: G W 3.16.0-arm-native+ #276 > [] (unwind_backtrace) from [] (show_stack+0x10/0x14) > [] (show_stack) from [] (warn_slowpath_common+0x5c/0x7c) > [] (warn_slowpath_common) from [] (warn_slowpath_null+0x18/0x20) > [] (warn_slowpath_null) from [] (__arm_ioremap_pfn_caller+0x118/0x1a4) > [] (__arm_ioremap_pfn_caller) from [] (__arm_ioremap+0x14/0x20) > [] (__arm_ioremap) from [] (gnttab_setup_auto_xlat_frames+0x30/0xdc) > [] (gnttab_setup_auto_xlat_frames) from [] (xen_guest_init+0x19c/0x2d4) > [] (xen_guest_init) from [] (do_one_initcall+0xfc/0x1a4) > [] (do_one_initcall) from [] (kernel_init_freeable+0x58/0x1b4) > [] (kernel_init_freeable) from [] (kernel_init+0x8/0xe4) > [] (kernel_init) from [] (ret_from_fork+0x14/0x3c) > ---[ end trace 3406ff24bd97382f ]--- > xen:grant_table: Failed to ioremap gnttab share frames (addr=0x00000000b0000000)! > > which is: > /* > * Don't allow RAM to be mapped - this causes problems with ARMv6+ > */ > if (WARN_ON(pfn_valid(pfn))) > return NULL; > > This makes sense since the gnttab defaults to 0xb000000 and my dom0 > is being allocated a 1:1 mapping at 0xa0000000-0xc0000000. > > I suspect this broke around the time we stopped forcing dom0 memory to be > allocated as low as possible which happened to prevent the default dom0_gnttab > region overlapping RAM. > > This patch specifies an explicit dom0_gnttab base which is explicitly unused > according to the FVP model docs (although it correpsonds to CS5 this isn't NIT: corresponds > wired up to anything). > > Signed-off-by: Ian Campbell Reviewed-by: Julien Grall Regards, -- Julien Grall