From patchwork Wed Aug 21 09:59:07 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ian Campbell X-Patchwork-Id: 2847633 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 8D15CBF546 for ; Wed, 21 Aug 2013 09:59:58 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 7305D204EC for ; Wed, 21 Aug 2013 09:59:57 +0000 (UTC) Received: from casper.infradead.org (casper.infradead.org [85.118.1.10]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id E62D2204EA for ; Wed, 21 Aug 2013 09:59:55 +0000 (UTC) Received: from merlin.infradead.org ([2001:4978:20e::2]) by casper.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VC5Cg-0003Tb-RX; Wed, 21 Aug 2013 09:59:51 +0000 Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1VC5Ce-0005iI-JA; Wed, 21 Aug 2013 09:59:48 +0000 Received: from smtp02.citrix.com ([66.165.176.63]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VC5CS-0005g8-O3 for linux-arm-kernel@lists.infradead.org; Wed, 21 Aug 2013 09:59:46 +0000 X-IronPort-AV: E=Sophos;i="4.89,927,1367971200"; d="scan'208";a="43910984" Received: from accessns.citrite.net (HELO FTLPEX01CL01.citrite.net) ([10.9.154.239]) by FTLPIPO02.CITRIX.COM with ESMTP; 21 Aug 2013 09:59:10 +0000 Received: from [127.0.0.1] (10.80.16.47) by smtprelay.citrix.com (10.13.107.78) with Microsoft SMTP Server id 14.2.342.4; Wed, 21 Aug 2013 05:59:09 -0400 Message-ID: <1377079147.31937.37.camel@hastur.hellion.org.uk> Subject: Re: [PATCH v2] arm: update advice on kernel, initramfs and FDT load address. From: Ian Campbell To: Will Deacon Date: Wed, 21 Aug 2013 10:59:07 +0100 In-Reply-To: <20130808215429.GG19383@mudshark.cambridge.arm.com> References: <1375974974-31603-1-git-send-email-ian.campbell@citrix.com> <20130808215429.GG19383@mudshark.cambridge.arm.com> X-Mailer: Evolution 3.4.4-3 MIME-Version: 1.0 X-DLP: MIA2 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20130821_055937_462041_ED65E04B X-CRM114-Status: GOOD ( 24.20 ) X-Spam-Score: -5.4 (-----) Cc: Nicolas Pitre , Roy Franz , Dave Martin , Russell King , Grant Likely , "linux-arm-kernel@lists.infradead.org" X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-7.0 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Thu, 2013-08-08 at 22:54 +0100, Will Deacon wrote: > On Thu, Aug 08, 2013 at 04:16:14PM +0100, Ian Campbell wrote: > > diff --git a/Documentation/arm/Booting b/Documentation/arm/Booting > > +overwrite it while also with the region which will be covered by the > > +kernel's low-memory mapping. > > s/with/within/ ? Yep, ETOOMANYREWRITES trying to get it sounding right... > > +If an initramfs is in use then, as with the dtb, it must be placed in > > +a region of memory where the kernel decompressor will not overwrite it > > +while also with the region which will be covered by the kernel's > > Hmm, same thing here. Maybe `...will not overwrite it, whilst remaining in the > region...'? Also made this change. Thanks. Unless there are any more comments on the correctness of the advice I suppose this should go via Russell's PTS? Now to dredge up the 8 year old memories of how that stuff works... Thanks, Ian. 8<---------------------- From 43f0e4eb33e1e8895666b5faffe1d1b6080d23b2 Mon Sep 17 00:00:00 2001 From: Ian Campbell Date: Mon, 29 Jul 2013 13:44:42 +0100 Subject: [PATCH] arm: update advice on kernel, initramfs and FDT load address. - Recommend that the kernel be placed under 128MiB, this is required if CONFIG_AUTO_ZRELADDR=y and good (or at least not bad) advice even if CONFIG_AUTO_ZRELADDR=n. - Recommend that a zImage kernel be placed above 32MiB, this avoids the need to relocate prior to decompression, which can speed up boot. - Add basic info on the requirements when loading a raw (non-zImage) kernel which are stricter than the zImage requirements. - Recommend that the DTB be placed after the 128MiB boundary, avoiding any potential conflict with the kernel decompressor, and within the lowmem region. In practice it could follow the kernel loaded after 32MiB, assuming the kernel decompesses to less than 32MiB, but the 128MiB recommendation is simple and unambiguous. - Add similar recommendation regarding initramfs. Signed-off-by: Ian Campbell Cc: Russell King Cc: Nicolas Pitre Cc: Will Deacon Cc: Dave Martin Cc: Grant Likely Cc: Roy Franz Cc: linux-arm-kernel@lists.infradead.org --- v3: Wording fixed from Will. v2: Update based upon discussion with Russell, Nico, Will. Unconditionally recommend loading the kernel below 128MiB, but above 32MiB. Recommend loading DTB at 128MiB and initramfs right after. Try to focus on concrete recommendations without getting into to much into the details of why or other alternatives/corner cases. --- Documentation/arm/Booting | 42 ++++++++++++++++++++++++++++++++---------- 1 files changed, 32 insertions(+), 10 deletions(-) diff --git a/Documentation/arm/Booting b/Documentation/arm/Booting index 0c1f475..371814a 100644 --- a/Documentation/arm/Booting +++ b/Documentation/arm/Booting @@ -18,7 +18,8 @@ following: 2. Initialise one serial port. 3. Detect the machine type. 4. Setup the kernel tagged list. -5. Call the kernel image. +5. Load initramfs. +6. Call the kernel image. 1. Setup and initialise RAM @@ -120,12 +121,27 @@ tagged list. The boot loader must pass at a minimum the size and location of the system memory, and the root filesystem location. The dtb must be placed in a region of memory where the kernel decompressor will not -overwrite it. The recommended placement is in the first 16KiB of RAM -with the caveat that it may not be located at physical address 0 since -the kernel interprets a value of 0 in r2 to mean neither a tagged list -nor a dtb were passed. +overwrite it, whilst remaining within the region which will be covered +by the kernel's low-memory mapping. -5. Calling the kernel image +A safe location is just above the 128MiB boundary from start of RAM. + +5. Load initramfs. +------------------ + +Existing boot loaders: OPTIONAL +New boot loaders: OPTIONAL + +If an initramfs is in use then, as with the dtb, it must be placed in +a region of memory where the kernel decompressor will not overwrite it +while also with the region which will be covered by the kernel's +low-memory mapping. + +A safe location is just above the device tree blob which itself will +be loaded just above the 128MiB boundary from the start of RAM as +recommended above. + +6. Calling the kernel image --------------------------- Existing boot loaders: MANDATORY @@ -136,11 +152,17 @@ is stored in flash, and is linked correctly to be run from flash, then it is legal for the boot loader to call the zImage in flash directly. -The zImage may also be placed in system RAM (at any location) and -called there. Note that the kernel uses 16K of RAM below the image -to store page tables. The recommended placement is 32KiB into RAM. +The zImage may also be placed in system RAM and called there. The +kernel should be placed in the first 128MiB of RAM. It is recommended +that it is loaded above 32MiB in order to avoid the need to relocate +prior to decompression, which will make the boot process slightly +faster. + +When booting a raw (non-zImage) kernel the constraints are tighter. +In this case the kernel must be loaded at an offset into system equal +to TEXT_OFFSET - PAGE_OFFSET. -In either case, the following conditions must be met: +In any case, the following conditions must be met: - Quiesce all DMA capable devices so that memory does not get corrupted by bogus network packets or disk data. This will save