All of lore.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: Albin Tonnerre <albin.tonnerre@free-electrons.com>
Cc: Russell King - ARM Linux <linux@arm.linux.org.uk>,
	Sam Ravnborg <sam@ravnborg.org>,
	Matthieu CASTET <matthieu.castet@parrot.com>,
	Alain Knaff <alain@knaff.lu>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-embedded@vger.kernel.org" <linux-embedded@vger.kernel.org>,
	"akpm@linux-foundation.org" <akpm@linux-foundation.org>
Subject: Re: [PATCH 4/6] Add support for LZO-compressed kernels for ARM
Date: Thu, 13 Aug 2009 07:07:00 -0700	[thread overview]
Message-ID: <4A841E04.8070108@zytor.com> (raw)
In-Reply-To: <20090813093036.GG4051@pc-ras4041.res.insa>

On 08/13/2009 02:30 AM, Albin Tonnerre wrote:
> 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().
> 

I would go with the variant in the kernel, as being a known quantity.

Your __div0 function can just loop forever.

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.

  reply	other threads:[~2009-08-13 14:07 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-22 14:01 [PATCH 1/5] lib/decompress_*: only include <linux/slab.h> if STATIC is not defined Albin Tonnerre
2009-07-22 14:01 ` [PATCH 2/5] include/linux/unaligned/{l,b}e_byteshift.h: Fix usage for compressed kernels Albin Tonnerre
2009-07-22 14:01   ` [PATCH 3/5] Add support for LZO-compressed kernels Albin Tonnerre
2009-07-22 14:01     ` [PATCH 4/5] Add support for LZO-compressed kernels for ARM Albin Tonnerre
2009-07-22 14:01       ` [PATCH 5/5] Add support for LZO-compressed kernels on x86 Albin Tonnerre
2009-07-29 15:37         ` [PATCH] Add LZO compression support for initramfs and old-style initrd Albin Tonnerre
2009-07-29 20:00         ` [PATCH 5/5] Add support for LZO-compressed kernels on x86 H. Peter Anvin
2009-07-29 21:02           ` Sam Ravnborg
2009-07-31  7:51             ` Albin Tonnerre
2009-07-31  9:31               ` Sam Ravnborg
2009-08-03 14:58                 ` [PATCH 1/6] lib/decompress_*: only include <linux/slab.h> if STATIC is not defined Albin Tonnerre
2009-08-03 14:58                   ` [PATCH 2/6] include/linux/unaligned/{l,b}e_byteshift.h: Fix usage for compressed kernels Albin Tonnerre
2009-08-03 14:58                     ` [PATCH 3/6] Add support for LZO-compressed kernels Albin Tonnerre
2009-08-03 14:58                       ` [PATCH 4/6] Add support for LZO-compressed kernels for ARM Albin Tonnerre
2009-08-03 14:58                         ` [PATCH 5/6] Add support for LZO-compressed kernels on x86 Albin Tonnerre
2009-08-03 14:58                           ` [PATCH 6/6] Add LZO compression support for initramfs and old-style initrd Albin Tonnerre
2009-08-03 15:12                             ` H. Peter Anvin
2009-08-03 16:05                               ` Albin Tonnerre
2009-08-03 15:11                           ` [PATCH 5/6] Add support for LZO-compressed kernels on x86 H. Peter Anvin
2009-08-06 22:40                         ` [PATCH 4/6] Add support for LZO-compressed kernels for ARM Russell King - ARM Linux
2009-08-07  9:24                           ` Albin Tonnerre
2009-08-07  9:34                             ` Russell King - ARM Linux
2009-08-07  9:36                             ` Alain Knaff
2009-08-07 10:21                               ` Albin Tonnerre
2009-08-07 11:29                                 ` Alain Knaff
2009-08-07 11:50                                 ` Matthieu CASTET
2009-08-07 13:01                                   ` Albin Tonnerre
2009-08-07 13:27                                     ` Matthieu CASTET
2009-08-07 13:55                                       ` Albin Tonnerre
2009-08-07 20:00                                         ` Russell King - ARM Linux
2009-08-07 20:21                                           ` H. Peter Anvin
2009-08-07 21:08                                           ` Sam Ravnborg
2009-08-11  9:44                                             ` Albin Tonnerre
2009-08-11 13:39                                               ` Sam Ravnborg
2009-08-11 14:17                                                 ` Albin Tonnerre
2009-08-11 13:51                                               ` Matthieu CASTET
2009-08-11 16:03                                               ` H. Peter Anvin
2009-08-11 16:27                                                 ` Albin Tonnerre
2009-08-11 16:31                                                   ` H. Peter Anvin
2009-08-13  9:30                                                     ` Albin Tonnerre
2009-08-13 14:07                                                       ` H. Peter Anvin [this message]
2009-08-13 12:25                           ` Albin Tonnerre
2009-08-14 10:08                           ` [PATCH 4/6 v2] " Albin Tonnerre
2009-09-04 15:31                             ` Albin Tonnerre
2009-08-04 23:00                       ` [PATCH 3/6] Add support for LZO-compressed kernels Andrew Morton
2009-08-04 23:00                         ` Andrew Morton
2009-08-05  1:36                         ` H. Peter Anvin
2009-08-05  9:19                         ` Albin Tonnerre
2009-08-04 23:04                       ` Andrew Morton
2009-08-04 23:04                         ` Andrew Morton
2009-08-04 22:55                     ` [PATCH 2/6] include/linux/unaligned/{l,b}e_byteshift.h: Fix usage for compressed kernels Andrew Morton
2009-08-04 22:55                       ` Andrew Morton
2009-08-05  9:29                       ` Albin Tonnerre
2009-08-05 21:19                         ` Russell King - ARM Linux
2009-08-04 22:55                   ` [PATCH 1/6] lib/decompress_*: only include <linux/slab.h> if STATIC is not defined Andrew Morton
2009-08-04 22:55                     ` Andrew Morton
2009-08-05  0:47                     ` Phillip Lougher
2009-08-05  0:57                       ` H. Peter Anvin
2009-08-05  1:32                         ` Phillip Lougher
2009-08-05  1:08                       ` Andrew Morton
2009-08-05  2:06                         ` Phillip Lougher
2009-07-23 17:11       ` [PATCH 4/5] Add support for LZO-compressed kernels for ARM Albin Tonnerre
     [not found]     ` <0022152d7fe9b6dbcf046f4d04a6@google.com>
2009-07-22 16:08       ` [PATCH 3/5] Add support for LZO-compressed kernels H. Peter Anvin
2009-07-22 16:50       ` Albin Tonnerre
2009-07-29 13:51     ` [PATCH 3/5 v2] " Albin Tonnerre

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A841E04.8070108@zytor.com \
    --to=hpa@zytor.com \
    --cc=akpm@linux-foundation.org \
    --cc=alain@knaff.lu \
    --cc=albin.tonnerre@free-electrons.com \
    --cc=linux-embedded@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=matthieu.castet@parrot.com \
    --cc=sam@ravnborg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.