From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] ARM: Fix bad SP address after relocating kernel Date: Wed, 27 Apr 2011 00:48:33 -0700 Message-ID: <20110427074833.GD3755@atomide.com> References: <20110421055945.GB13688@atomide.com> <20110421104954.GH13688@atomide.com> <20110422062843.GA841@atomide.com> <20110426085700.GJ16892@atomide.com> <20110426123746.GC3755@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-03-ewr.mailhop.org ([204.13.248.66]:55245 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751295Ab1D0Hsm (ORCPT ); Wed, 27 Apr 2011 03:48:42 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Nicolas Pitre Cc: Shawn Guo , linux-arm-kernel@lists.infradead.org, patches@linaro.org, Aaro Koskinen , linux-omap@vger.kernel.org * Nicolas Pitre [110426 14:28]: > On Tue, 26 Apr 2011, Tony Lindgren wrote: > > > Otherwise cache_clean_flush can overwrite some of the relocated > > area depending on where the kernel image gets loaded. This fixes > > booting on n900 after commit 6d7d0ae51574943bf571d269da3243257a2d15db > > (ARM: 6750/1: improvements to compressed/head.S). > > Gaaaah. Indeed. > > > Thanks to Aaro Koskinen for debugging > > the address of the relocated area that gets corrupted, and to > > Nicolas Pitre for the other uncompress > > related fixes. > > > > Signed-off-by: Tony Lindgren > > I think there could be a better fix yet. Could you test this patch: Cool that works too and avoids using the stack, so that's a better fix: Tested-by: Tony Lindgren