Re: [PATCH] New x86_64 EFI patch

[Isaac Dupree <id@isaac.cedarswampstudios.org>]

Bean wrote:
> On Sat, Jun 21, 2008 at 1:04 AM, Bean <bean123ch@gmail.com> wrote:
>> Hi,
>>
>> This new patch add the following function to the original x86_64 EFI patch:
>>
>> 1, Fix menu drawing problem
>> It maps the unicode char to EFI char so that the rectangle box is
>> showed properly
>>
>> 2, Handle command line option for chainloader command
>>
>> 3, Add new command appleloader
>> The command is used to enable bios boot in apple's bootcamp, for example
>>
>> boot from legacy mbr:
>> appleloader HD
>> boot
>>
>> boot from legacy cd:
>> appleloader CD
>> boot
>>
>> boot from usb:
>> appleloader USB
>> boot
>>
>> The usb booting may not work in some machine.
> 
> Hi,
> 
> I also fix a few problem in the linux loader, now it can load both 686
> and x86_64 kernel from 64-bit EFI firmware.
> 
> Would someone test it in 32-bit EFI ?

with just that patch against CVS, for me

- linux then initrd doesn't run out of memory (that fix is still there). 
  On the other hand, 'linux' with wrong parameters so I repeated the 
command, did give me such an error.  In any case, trying again perfectly 
and booting, gave me the same (as my last test with your EFI patches) 
hanging on the GRUB terminal, not booting successfully.

- appleloader always tells me "Model : Core Duo/Solo" and nothing else 
(no matter whether I give it a parameter or not).  If I say "boot" 
nothing happens, it just stays in GRUB prompt and I can enter more 
commands.  (admittedly, I think I didn't *have* anything to boot in my 
CD drive, USB or MBR)

-Isaac