From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.linuxfoundation.org ([140.211.169.12]) by bombadil.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1S2Bo4-0002N1-CW for kexec@lists.infradead.org; Tue, 28 Feb 2012 01:24:45 +0000 Date: Mon, 27 Feb 2012 17:25:55 -0800 From: Andrew Morton Subject: Re: [PATCH] kexec: crash: don't save swapper_pg_dir for !CONFIG_MMU configurations Message-Id: <20120227172555.46bffedf.akpm@linux-foundation.org> In-Reply-To: <20120228005130.GE11325@verge.net.au> References: <1330094447-11900-1-git-send-email-will.deacon@arm.com> <20120225023748.GE21429@verge.net.au> <20120226225842.GA23028@mudshark.cambridge.arm.com> <20120227003727.GA30831@verge.net.au> <20120227193054.GB24605@mudshark.cambridge.arm.com> <20120227235235.GE24159@verge.net.au> <20120227155643.cafb1c6d.akpm@linux-foundation.org> <20120228001928.GB11325@verge.net.au> <20120227162631.55cf014e.akpm@linux-foundation.org> <20120228005130.GE11325@verge.net.au> Mime-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: kexec-bounces@lists.infradead.org Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: Simon Horman Cc: "kexec@lists.infradead.org" , Will Deacon On Tue, 28 Feb 2012 09:51:30 +0900 Simon Horman wrote: > On Mon, Feb 27, 2012 at 04:26:31PM -0800, Andrew Morton wrote: > > On Tue, 28 Feb 2012 09:19:28 +0900 > > Simon Horman wrote: > > > > > > > --- a/kernel/kexec.c > > > > > +++ b/kernel/kexec.c > > > > > @@ -1462,7 +1462,9 @@ static int __init crash_save_vmcoreinfo_init(void) > > > > > > > > > > VMCOREINFO_SYMBOL(init_uts_ns); > > > > > VMCOREINFO_SYMBOL(node_online_map); > > > > > +#ifdef CONFIG_MMU > > > > > VMCOREINFO_SYMBOL(swapper_pg_dir); > > > > > +#endif > > > > > VMCOREINFO_SYMBOL(_stext); > > > > > VMCOREINFO_SYMBOL(vmlist); > > > > > > > > Well, what might be the effects of this patch? nommu crashfiles will > > > > no longer have the swapper_pg_dir string? What are the chances that > > > > someone's (badly written!) downstream tool will crash and burn if this > > > > is absent? > > > > > > My understanding is that up until this patch creating a dump > > > for nonmmu platform wouldn't work. > > > > Surprised. From reading the code I expect it would have emitted > > > > SYMBOL(swapper_pg_dir)=0 > > Hi Andrew, > > My understanding from discussion with Will earlier in this thread is that if > CONFIG_MMU is not defined then swapper_pg_dir is NULL and the current code > doesn't compile. > > > The only case where it's interesting is when you have CONFIG_MMU > enabled - otherwise it's always NULL. If it's #defined as NULL, the > current code will fail at build time so simply omitting it from the > dump seems like the best bet to me (the alternative being to add a > NULL entry explicitly, but I don't see what the gains us). > OK. That's because everything is all screwed up ;) swapper_pg_dir is normally an array. But on nommu it is a pointer. VMCOREINFO_SYMBOL() wants to take its address (unnecessary on an array) and this blows up when fed a pointer. Still, you didn't answer my question! What effect will the absence of SYMBOL(swapper_pg_dir)= have upon downstream tools? If "none" then sure, let's remove it. If "explosion" then we should emit a dummy SYMBOL(swapper_pg_dir)=0 if CONFIG_NOMMU. _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec