From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753939AbZHMJak (ORCPT ); Thu, 13 Aug 2009 05:30:40 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753904AbZHMJaj (ORCPT ); Thu, 13 Aug 2009 05:30:39 -0400 Received: from mail.free-electrons.com ([88.191.76.200]:50125 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753907AbZHMJai (ORCPT ); Thu, 13 Aug 2009 05:30:38 -0400 Date: Thu, 13 Aug 2009 11:30:36 +0200 From: Albin Tonnerre 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" Subject: Re: [PATCH 4/6] Add support for LZO-compressed kernels for ARM 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 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4A819CDD.1000700@zytor.com> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.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