From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757363AbZHGJ6S (ORCPT ); Fri, 7 Aug 2009 05:58:18 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755985AbZHGJ6R (ORCPT ); Fri, 7 Aug 2009 05:58:17 -0400 Received: from crmm.lgl.lu ([158.64.72.228]:36157 "EHLO lll.lu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755410AbZHGJ6Q (ORCPT ); Fri, 7 Aug 2009 05:58:16 -0400 X-Greylist: delayed 1235 seconds by postgrey-1.27 at vger.kernel.org; Fri, 07 Aug 2009 05:58:15 EDT Message-ID: <4A7BF5B8.4030907@knaff.lu> Date: Fri, 07 Aug 2009 11:36:56 +0200 From: Alain Knaff User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3pre) Gecko/20090513 Fedora/3.0-2.3.beta2.fc11 Thunderbird/3.0b2 MIME-Version: 1.0 To: Albin Tonnerre CC: Russell King - ARM Linux , sam@ravnborg.org, hpa@zytor.com, 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 References: <20090731093107.GA29704@merkur.ravnborg.org> <1249311501-23102-1-git-send-email-albin.tonnerre@free-electrons.com> <1249311501-23102-2-git-send-email-albin.tonnerre@free-electrons.com> <1249311501-23102-3-git-send-email-albin.tonnerre@free-electrons.com> <1249311501-23102-4-git-send-email-albin.tonnerre@free-electrons.com> <20090806224055.GH31579@n2100.arm.linux.org.uk> <20090807092423.GA4455@laptop> In-Reply-To: <20090807092423.GA4455@laptop> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/07/09 11:24, Albin Tonnerre wrote: > On Thu, Aug 06, 2009 at 11:40:55PM +0100, Russell King - ARM Linux wrote : >> On Mon, Aug 03, 2009 at 04:58:19PM +0200, Albin Tonnerre wrote: >>> This is the second part of patch. This part includes: >>> - changes to ach/arch/boot/Makefile to make it easier to add new >>> compression types >>> - new piggy.lzo.S necessary for lzo compression >>> - changes in arch/arm/boot/compressed/misc.c to allow the use of lzo or >>> gzip, depending on the config >>> - Kconfig support > >> FYI, with these patches applied and selecting GZIP method, I get >> linker errors. I've been unable to track down what's going on, but >> it appears to be a libgcc issue. > >> In spite of the decompressor being built as an EABI object, gcc seems >> to be issuing calls to __umodsi3, which isn't in the EABI libgcc >> (they're called something different - don't ask.) > > I also happen to have issues related to the linker not finding __aeabi_uidivmod, > but only when compiling with CONFIG_CC_OPTIMIZE_FOR_SIZE. Is it similar for you, > or does it also fail when compiling with -O2 ? > >> So I think these patches need further testing and evaluation on ARM >> before they can be merged. Moreover, I'd like to see some comparisons >> between the _current_ gzip method, the new gzip method and the lzo >> method on ARM. > > I don't have my ARM platforms at hand, but I'll so these comparisons in a couple > days when I access to them again. > > Regards, Could it be that the patches that remove division (zutil.h and inflate.c) have somehow not been applied? Regards, Alain