Re: [PATCH] ELF64 and changelog and 4Gb

[Marco Gerards]

"Ruslan Nikolaev" <address@hidden> 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