From: Marco Gerards <mgerards@xs4all.nl>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: Grub for ia64
Date: Thu, 28 Sep 2006 16:29:00 +0200 [thread overview]
Message-ID: <87ac4k12vn.fsf@xs4all.nl> (raw)
In-Reply-To: <1159451113.451bd1e94a4d9@imp1-g19.free.fr> (tgingold@free.fr's message of "Thu, 28 Sep 2006 15:45:13 +0200")
tgingold@free.fr writes:
>> Can you please explain why you want this in detail? I know there are
>> issues with EFI to determine the prefix. My guess is that is why you
>> want to change this.
> Yes.
Please elaborate.
>> But I am afraid this might not be a generic
>> solution so I hope we can discuss the problem first.
> I suppose you can't use mixed case filename currently for FAT fs. This
> patch fixes that.
Well, I am afraid the same problem will show up when using HFS+, ext2
or whatever. So that is why I would like a detailed description of
this problem (I just know there is a problem, but never received a
detailed report).
>> > [I think most 64 bits issues have already been reported recently and
>> > independently by the mail grub2 64bit system compatible]
>>
>> Cool. I assume these patches are in CVS now?
> AFAIK not yet integrated.
Ok. Your patch relies on that? so would those have to go in first?
>> > * the current common code can't work on ia64 (on ia64 a function
>> > pointer is a descriptor and not the address of the first function
>> > instruction).
>>
>> Can you give some examples by using code? Certainly we would have to
>> aim for module loading at some point in time. :)
> Yes, cf kern/dl.c:
>
> if (grub_strcmp (name, "grub_mod_init") == 0)
> mod->init = (void (*) (grub_dl_t)) sym->st_value;
>
> This won't work on ia64 AFAIK.
Why not? In the internal working of filesystems there are also
function pointer being used. Same for almost every other part of
GRUB 2. So I am wondering what makes this different.
>> > * grub-emu doesn't have dynamic modules and could reuse this work to remove
>> > most of #ifdef/#endif GRUB_UTIL
>>
>> *Please* do not mess around too much with grub-emu. I can perfectly
>> understand why people want all kind of things changed in grub-emu and
>> I would certainly like to have module support there (IIRC there were
>> patches sent in, some of which I still have to review, etc :-/).
>>
>> It's important to know why I wrote grub-emu. It is mainly a debugging
>> tool. For example you don't want to implement a filesystem or work on
>> the commandline interface without such tool. Having module loading
>> only is just annoying so GRUB_UTIL can't and won't be removed.
From my point of view, GRUB_UTIL could be removed without adding
>modules to grub-emu. This would be slightly cleaner.
Well, it's use can certainly be brought back or changed. But adding a
module loader is not the way to go. If it bothers a lot of people I
could change functions like grub_dl_ref into a macro which does the
#ifdef.
>> Actually, it is technically almost impossible to do modules loading in
>> every case on every processor. For example take the PPC, some
>> relocations are almost impossible or very hard to implement in
>> userspace.
> Looks strange. How does ld.so works on these systems ?
For the PPC I once wrote a module loader (userspace). Some
relocations expect the symbols you point to from the relocated code,
to be within 4MB. The dynamic loader on the PPC perhaps has no
problems with that because it can load code about everywhere and
perhaps those relocations are normally not used, but I can be wrong.
--
Marco
next prev parent reply other threads:[~2006-09-28 14:21 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-09-28 11:41 Grub for ia64 tgingold
2006-09-28 13:06 ` bibo,mao
2006-09-28 13:12 ` tgingold
2006-09-28 13:33 ` Marco Gerards
2006-09-28 13:45 ` tgingold
2006-09-28 14:29 ` Marco Gerards [this message]
2006-09-28 14:49 ` tgingold
2006-09-29 5:10 ` Grub for ia64 - function descriptors Hollis Blanchard
2006-09-29 6:59 ` tgingold
2006-09-29 13:16 ` Hollis Blanchard
2006-10-02 6:31 ` tgingold
2006-10-03 2:17 ` Hollis Blanchard
2006-10-03 4:04 ` tgingold
2006-10-03 15:09 ` Hollis Blanchard
2006-09-28 13:47 ` Grub for ia64 Johan Rydberg
2006-09-28 13:35 ` tgingold
2006-09-28 14:15 ` Johan Rydberg
2006-09-28 13:38 ` Marco Gerards
2006-10-10 18:15 ` Johan Rydberg
2006-10-10 22:03 ` Jeff Bailey
2006-10-11 11:19 ` Johan Rydberg
2006-10-11 10:50 ` Tristan Gingold
-- strict thread matches above, loose matches on Subject: below --
2006-10-11 11:43 Mao, Bibo
2006-10-12 8:21 ` Tristan Gingold
2006-10-12 11:30 ` Johan Rydberg
2006-10-12 11:35 ` Tristan Gingold
2006-10-12 12:11 ` Johan Rydberg
2006-10-12 14:14 ` Tristan Gingold
2006-10-12 15:32 ` Johan Rydberg
2006-10-13 5:22 ` Tristan Gingold
2006-10-13 8:02 ` bibo,mao
2006-10-13 18:48 ` Yoshinori K. Okuji
2006-10-12 12:50 Mao, Bibo
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=87ac4k12vn.fsf@xs4all.nl \
--to=mgerards@xs4all.nl \
--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.