[syslinux] Re: Patch to add Symbols/segments loader for mboot.c

[Tim Deegan]

Hi Ram,

Thanks for the patch.  I've tidied it up a little and I'm testing it
now.  I have a question about the code to load additional sections out
of the section table.  Why do you use place_module_section() for this?
Since the sections come with their virtual addresses specified in the
ELF binary, shouldn't you be using place_kernel_section()?

Cheers,

Tim.

On Fri, Jan 13, 2006 at 05:45:54PM -0800, Ram Yalamanchili wrote:
> Hi Tim!
> 
> I use your mboot.c32 to load multiboot kernels over pxelinux. I appreciate
> the work you have done on this. However, some kernels require the symbol
> table and other information in the section header table to be able to load
> properly. This functionality is not present in mboot.c now, so I added
> support for it. I haven't touched a.out however.
> 
> The patch is attached. It would be great if you can push this into the next
> release of syslinux so we dont have to distribute patched versions of
> mboot.c32.
> 
> The patch is diff'ed against syslinux-3.11. I wasn't sure who is maintaining
> this, so i'll send a copy of this message to the syslinux mailing list too,
> so Peter Anvin can have a look.
> 
> thanks,
> Ram


-- 
Tim Deegan                           (My opinions, not the University's)
Systems Research Group
University of Cambridge Computer Laboratory