Re: [Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Aurelien Jarno
On Sun, Mar 14, 2010 at 10:55:48PM +, Paul Brook wrote: > > Thanks for the review. > > > > On Sun, Mar 14, 2010 at 09:59:44PM +, Paul Brook wrote: > > > > The patch also convert all machines that have an addend, simplify the > > > > PowerPC kernel loading > > > > > > The new PPC code looks

Re: [Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Paul Brook
> Thanks for the review. > > On Sun, Mar 14, 2010 at 09:59:44PM +, Paul Brook wrote: > > > The patch also convert all machines that have an addend, simplify the > > > PowerPC kernel loading > > > > The new PPC code looks like it may break images > 16M in size. > > The idea was to get rid of t

Re: [Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Aurelien Jarno
Thanks for the review. On Sun, Mar 14, 2010 at 09:59:44PM +, Paul Brook wrote: > > The patch also convert all machines that have an addend, simplify the > > PowerPC kernel loading > > The new PPC code looks like it may break images > 16M in size. The idea was to get rid of the most signific

Re: [Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Paul Brook
> The patch also convert all machines that have an addend, simplify the > PowerPC kernel loading The new PPC code looks like it may break images > 16M in size. > and fix the MIPS kernel loading using this new > feature. I'm fairly sure the MIPS routines are not board specific, so it'd be nice i

[Qemu-devel] [PATCH] load_elf: replace the address addend by a translation function

2010-03-14 Thread Aurelien Jarno
A few machines need to translate the ELF header addresses into physical addresses. Currently the only possibility is to add a value to the addresses. This patch replaces the addend argument by and a translation function and an opaque passed to the function. A NULL function does not translate the a