Subject: Re: MIPS shared linker running at virtual address 0!
To: Matthias Drochner <M.Drochner@fz-juelich.de>
From: Simon Burge <email@example.com>
Date: 10/07/2005 18:24:46
On Thu, Oct 06, 2005 at 07:20:38PM +0200, Matthias Drochner wrote:
> firstname.lastname@example.org said:
> > The p_vaddr check is just something approximating "is this a low
> > address". It's not really MIPS specific.
> That's still a hack for a mips specific problem, and only
> for the COMPAT_16 case. We shouldn't bloat other kernels
> with this, and also visually separate it in the source code.
> An #ifdef ELF_INTERP_NON_RELOCATABLE would be fine imo.
> > I'd like
> > to propose that we use the current patch for a couple of reasons
> > [...]
> > It's been well tested
> I concur that it is better to use your tested patch for 3.x.
> In theory it would still be conceivable that some linker change
> causes a non-PT_LOAD section with a higher (junk) vaddr, undermining
> your check, but this is unlikely in the release branch.
> For HEAD, we should make sure to find a cleaner fix.
I'll commit it as-is to HEAD then done one more build test on 3.0 before
submitting the pull-up. After that I'll add a note to the comment block
in HEAD about the potential problems of a non-PT_LOAD section first up.
Simon Burge <email@example.com>
NetBSD Development, Support and Service: http://www.wasabisystems.com/