Re: normal mode chainloader

[Marco Gerards]

address@hidden (Tomas Ebenlendr) writes:

> Okay, here is the patch without dependency symbol. (I hope it is GCS
> compliant now.)

Great.

I have still some questions about the patch.  How do both loaders
(that is both in normal mode and rescue mode) find the generic
chainloader?  If I understand it correctly, the chain.mod depends on
_chain.mod, but how does this work?

Does it load _normal.mod and use those symbols?  Or does it really
share the code? (so there is not dependency on the rescue mode
chainloader at all, even when it is not part of core.mod).

Perhaps it would be better to just use one sourcecode file,
chainloader.c, like this:

#ifdef GRUB_NORMAL_MODE
  the normalmode code
#else
  rescue mode code
#endif

And just put -DGRUB_NORMAL_MODE in the CFLAGS when compiling the
normal mode module.  That will make it possible to use one sourcefile
for both modes.

Perhaps I just misunderstood your patch, if that is the case, please
tell me.  Next time I try to read your entire patch a bit earlier.
Same for the other patches, I try to apply them (after cleaning them
up a bit) this weekend.  Now I am still busy with school, it is taking
longer than I have expected.  I hope you understand.

Thanks,
Marco