From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albin Tonnerre Subject: Re: [PATCH 4/6] Add support for LZO-compressed kernels for ARM Date: Thu, 13 Aug 2009 11:30:36 +0200 Message-ID: <20090813093036.GG4051@pc-ras4041.res.insa> References: <4A7C14EB.1060701@parrot.com> <20090807130130.GE4455@laptop> <4A7C2BA4.7030408@parrot.com> <20090807135524.GF4455@laptop> <20090807200001.GE31543@n2100.arm.linux.org.uk> <20090807210816.GA29797@merkur.ravnborg.org> <20090811094418.GI9704@pc-ras4041.res.insa> <4A819642.10700@zytor.com> <20090811162730.GM9704@pc-ras4041.res.insa> <4A819CDD.1000700@zytor.com> Mime-Version: 1.0 Return-path: Content-Disposition: inline In-Reply-To: <4A819CDD.1000700@zytor.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "H. Peter Anvin" , Russell King - ARM Linux Cc: Sam Ravnborg , Matthieu CASTET , Alain Knaff , "linux-kernel@vger.kernel.org" , "linux-embedded@vger.kernel.org" , "akpm@linux-foundation.org" On Tue, Aug 11, 2009 at 09:31:25AM -0700, H. Peter Anvin wrote : > On 08/11/2009 09:27 AM, Albin Tonnerre wrote: > > > > So I guess the only options left are either define a dummy raise() function, or > > get rid of the divisions like Alain Knaff did in his patch ? > > > > Define a dummy raise, get rid of the divisions, or provide your own > division function (there is probably one in the kernel already...) Yes, there's such a function in arch/arm/lib/lib1funcs.S, which in turns requires the __div0 symbol, defined in arch/arm/kernel/traps.c, and this one cannot be used as it brings plenty of symbols that aren't available in the pre-boot environment. So anyway, we have to define our own symbols there. Either we use libgcc and define raise(), or lib1funcs and define __div0(). Russell, what's your take on this ? Regards, -- Albin Tonnerre, Free Electrons Kernel, drivers and embedded Linux development, consulting, training and support. http://free-electrons.com