"Ruslan Nikolaev" <[EMAIL PROTECTED]> writes:

>> I agree. This is more readable.
>
> I can change it of course... But it is not so bad:
>
> 1. if elf class == 64 => using ELF64 validate & loading code
>
> 2. else:
>
> - validate header by grub_dl_check_header(). Function
> grub_dl_check_header() also will check that elf class is 32-bit.

The important thing to remember is that GRUB support more machines
than the PC and other loaders than just multiboot.  So having a ELF32
and ELF64 loader like Hollis suggested would be the right thing to
do.  It will mean more shared code and a clean solution.

Thanks,
Marco



_______________________________________________
Grub-devel mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to