All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Roskin <proski@gnu.org>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: State of GRUB on PowerPC
Date: Wed, 28 Jan 2009 21:57:47 -0500	[thread overview]
Message-ID: <1233197867.2727.39.camel@dv> (raw)
In-Reply-To: <1233040781.5108.98.camel@thor.local>

On Tue, 2009-01-27 at 08:19 +0100, Michel Dänzer wrote:
> I was able to reproduce Jordi's findings on my PowerBook G4. (Well,
> except device.map seems to get generated correctly and the search
> command seems to work for me, maybe this is due to differences between
> our OF device trees or something like that)
> 
> After some printf-style debugging over the weekend, the failure to load
> some modules indeed turns out to be an hfs.mod bug: the problem is that
> strncasecmp() doesn't match the HFS B-tree sort order, which in
> particular breaks lookup of files with an underscore in their name. The
> first attached patch fixes this using a lookup table from Linux
> fs/hfs/string.c.

Actually, the return value of grub_strncasecmp() was incorrect until
recently.  Maybe the current version would work for you?

I'm not against your patch, but I'd like to understand how important it
is for GRUB.

Please write a ChangeLog entry for the patch.

> The failure to auto-load some modules like search was also caused by
> this, the auto-loading process aborts after failing to load a module. It
> might be better to continue auto-loading other modules anyway.

Patches are welcome.  With explanations, please.

> BTW, I also need the second attached patch to be able to boot my
> self-built 32 bit kernels configured to support 2GB lowmem.
> elf->ehdr.ehdr32.e_entry ends up as 0x70000000.

Strange.  The original mask should ensure that elf->ehdr.ehdr32.e_entry
is less than 0x40000000.

-- 
Regards,
Pavel Roskin



  parent reply	other threads:[~2009-01-29  2:57 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-27  7:19 State of GRUB on PowerPC Michel Dänzer
2009-01-27 16:54 ` Vesa Jääskeläinen
2009-01-27 17:15   ` Michel Dänzer
2009-01-29  2:57 ` Pavel Roskin [this message]
2009-01-29 16:18   ` Michel Dänzer
2009-02-07 19:48 ` hfs patch (Re: State of GRUB on PowerPC) Robert Millan
2009-02-08  4:38   ` Pavel Roskin
2009-02-09 14:19     ` Robert Millan
2009-02-10  9:55       ` Michel Dänzer
2009-02-10  9:54     ` Michel Dänzer
2009-02-10 10:50       ` Jordi Mallach
2009-02-10 12:06         ` Jordi Mallach
2009-02-10 12:12           ` Michel Dänzer
2009-02-21 12:46         ` Robert Millan
2009-02-10 14:47       ` Robert Millan
2009-02-11  9:24         ` Michel Dänzer
2009-02-21 13:01           ` Robert Millan
  -- strict thread matches above, loose matches on Subject: below --
2008-12-19  0:00 State of GRUB on PowerPC Jordi Mallach
2008-12-19 11:47 ` Manoel Rebelo Abranches
2008-12-21 18:37 ` Jordi Mallach

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=1233197867.2727.39.camel@dv \
    --to=proski@gnu.org \
    --cc=grub-devel@gnu.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.