From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-pa0-x22d.google.com ([2607:f8b0:400e:c03::22d]) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1bUONs-00078b-59 for kexec@lists.infradead.org; Tue, 02 Aug 2016 01:20:40 +0000 Received: by mail-pa0-x22d.google.com with SMTP id b2so5076889pat.0 for ; Mon, 01 Aug 2016 18:20:19 -0700 (PDT) Date: Tue, 2 Aug 2016 10:25:54 +0900 From: AKASHI Takahiro Subject: Re: [PATCH] arm64:kexec: Memstart should not be before the kernel start address Message-ID: <20160802012551.GF7995@linaro.org> References: <1461708860-23034-1-git-send-email-sgoel@codeaurora.org> <1461715120.27026.4.camel@infradead.org> <3624bf19-6569-b631-92f9-cc9fc91b40cd@codeaurora.org> <1465576876.3105.54.camel@infradead.org> <569f36b7-e153-742f-b7fe-c3a1d7b1febc@codeaurora.org> <1470079910.16559.49.camel@infradead.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1470079910.16559.49.camel@infradead.org> 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" Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: Geoff Levand Cc: "Goel, Sameer" , timur@codeaurora.org, kexec@lists.infradead.org, rruigrok@codeaurora.org, shankerd@codeaurora.org On Mon, Aug 01, 2016 at 12:31:50PM -0700, Geoff Levand wrote: > Hi, > > On Fri, 2016-07-29 at 16:58 -0600, Goel, Sameer wrote: > > In our setup we cannot place the kernel starting at 0 address. So, it it > > placed at the first valid aligned address after placing the reserved > > regions in memory. > > > > The code from the change (https://patchwork.kernel.org/patch/8325221/) > > adds the reserved regions to iomem ranges. So, when the kexec tool is > > run it picks up the first reserved region as the start address for > > physical ram. When the kernel is loaded at this address, it fails to boot. > > > > So, I have put in a filter in the code that would pick up the right > > address of the primary kernel after this change was introduced. You're using Image, not vmlinux, right? I don't think that it's a good idea to add this sort of restriction because it is very system-specific. You'd better use "--mem-min=" parameter in your case. (In fact, this option doesn't work on the current arm64 port of kexec-tools, though.) Thanks, -Takahiro AKASHI > > Thanks, > > Sameer > > > Your patch no longer applies. Please rebase it to my > for-merge-arm64-v2 branch and re-post. > > -Geoff > _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec