In my tests, the patch with better error handling seems to work without a problem.<br><br>My modified elilo and esxi5 bootloader both made it through boot with command line arguments intact and everything.<br><br>Thanks.<br>
<div class="gmail_quote">On Mon, Feb 13, 2012 at 8:12 AM, Michael Brown <span dir="ltr"><<a href="mailto:mbrown@fensystems.co.uk">mbrown@fensystems.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Saturday 11 Feb 2012 20:41:36 Jarrod Johnson wrote:<br>
> In the same way iPXE exports its capabilities under BIOS in a way that<br>
> pxelinux.0 exploits to get better protocols than tftp, export capabilities<br>
> as a UEFI protocol so that other efi binaries may use it instead of tftp.<br>
> Can be used together with a patch submitted to elilo for one example:<br>
> <a href="http://comments.gmane.org/gmane.comp.boot-loaders.elilo.general/147" target="_blank">http://comments.gmane.org/gmane.comp.boot-loaders.elilo.general/147</a><br>
><br>
> <a href="https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/d748ebf72206dcde1379b0" target="_blank">https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/d748ebf72206dcde1379b0</a><br>
> 63fd9b533f7572caa5<br>
> <a href="https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/e7b41890bc67350a9f5bc" target="_blank">https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/e7b41890bc67350a9f5bc</a><br>
> f7291ec859fa2174e26<br>
> <a href="https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/45a51f99fd51b231a388b" target="_blank">https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/45a51f99fd51b231a388b</a><br>
> 1db00dfc4b18a5e9e4f<br>
> <a href="https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/a84739023c391fa4d5e3b" target="_blank">https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/a84739023c391fa4d5e3b</a><br>
> 65a487834bf92552895<br>
> <a href="https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/23aea6c209965546c3956" target="_blank">https://git.ipxe.org/vendor/xcat/ipxe.git/commitdiff/23aea6c209965546c3956</a><br>
> 9a770823eabf1358a71<br>
<br>
</div></div>I've merged these into a single patch (not yet merged):<br>
<br>
  <a href="http://git.ipxe.org/people/mcb30/ipxe.git/commitdiff/1730726" target="_blank">http://git.ipxe.org/people/mcb30/ipxe.git/commitdiff/1730726</a><br>
<br>
This is produced by taking your commits, converting initial spaces to tabs<br>
where applicable, and changing "gPXE" to "iPXE" where applicable.<br>
<br>
I also tidied up what looked like some dubious error handling in efi_image.c:<br>
errors were being ignored, and nothing ever freed the memory allocated by<br>
efi_image_make_cmdline().  I've put in what I think should be correct code, but<br>
I haven't tested it in any way.<br>
<br>
Please let me know if you think this modified patch is OK to merge.<br>
<br>
Thanks,<br>
<br>
Michael<br>
</blockquote></div><br>